backportpackage: Make pylint happier.

This commit is contained in:
Benjamin Drung 2010-12-27 20:32:07 +01:00
parent 7b7c84a9fe
commit f6cb127479
2 changed files with 80 additions and 72 deletions

View File

@ -96,7 +96,7 @@ def find_main(cache, pack):
break break
if got_src: if got_src:
# pylint: disable=E1101 # pylint: disable=E1101
for deptype, all_deps in src_records.build_depends.iteritems(): for _, all_deps in src_records.build_depends.iteritems():
# pylint: enable=E1101 # pylint: enable=E1101
for or_deps in all_deps: for or_deps in all_deps:
base_deps = [] base_deps = []
@ -107,9 +107,9 @@ def find_main(cache, pack):
process_deps(cache, deps) process_deps(cache, deps)
def usage(exit): def usage(exit_code):
print 'Usage: %s <package name> [<distribution>]' % sys.argv[0] print 'Usage: %s <package name> [<distribution>]' % sys.argv[0]
sys.exit(exit) sys.exit(exit_code)
def main(): def main():

View File

@ -48,76 +48,84 @@ def check_call(cmd, *args, **kwargs):
def parse(args): def parse(args):
usage = 'Usage: %prog [options] <source package name or .dsc URL/file>' usage = 'Usage: %prog [options] <source package name or .dsc URL/file>'
p = optparse.OptionParser(usage) parser = optparse.OptionParser(usage)
p.add_option('-d', '--destination', parser.add_option('-d', '--destination',
dest='dest_releases', dest='dest_releases',
default=[], default=[],
action='append', action='append',
help='Backport to DEST release (default: current release)', help='Backport to DEST release '
metavar='DEST') '(default: current release)',
p.add_option('-s', '--source', metavar='DEST')
dest='source_release', parser.add_option('-s', '--source',
default=None, dest='source_release',
help='Backport from SOURCE release (default: devel release)', default=None,
metavar='SOURCE') help='Backport from SOURCE release '
p.add_option('-S', '--suffix', '(default: devel release)',
dest='suffix', metavar='SOURCE')
default=None, parser.add_option('-S', '--suffix',
help='Suffix to append to version number (default: ~ppa1)', dest='suffix',
metavar='SUFFIX') default=None,
p.add_option('-b', '--build', help='Suffix to append to version number '
dest='build', '(default: ~ppa1)',
default=False, metavar='SUFFIX')
action='store_true', parser.add_option('-b', '--build',
help='Build the package before uploading (default: %default)') dest='build',
p.add_option('-B', '--builder', default=False,
dest='builder', action='store_true',
default=None, help='Build the package before uploading '
help='Specify the package builder (default: pbuilder)', '(default: %default)')
metavar='BUILDER') parser.add_option('-B', '--builder',
p.add_option('-U', '--update', dest='builder',
dest='update', default=None,
default=False, help='Specify the package builder (default: pbuilder)',
action='store_true', metavar='BUILDER')
help='Update the build environment before attempting to build') parser.add_option('-U', '--update',
p.add_option('-u', '--upload', dest='update',
dest='upload', default=False,
help='Specify an upload destination', action='store_true',
metavar='UPLOAD') help='Update the build environment before '
p.add_option('-y', '--yes', 'attempting to build')
dest='prompt', parser.add_option('-u', '--upload',
default=True, dest='upload',
action='store_false', help='Specify an upload destination',
help='Do not prompt before uploading to a PPA') metavar='UPLOAD')
p.add_option('-v', '--version', parser.add_option('-y', '--yes',
dest='version', dest='prompt',
default=None, default=True,
help='Package version to backport (or verify)', action='store_false',
metavar='VERSION') help='Do not prompt before uploading to a PPA')
p.add_option('-w', '--workdir', parser.add_option('-v', '--version',
dest='workdir', dest='version',
default=None, default=None,
help='Specify a working directory (default: temporary dir)', help='Package version to backport (or verify)',
metavar='WORKDIR') metavar='VERSION')
p.add_option('-m', '--mirror', parser.add_option('-w', '--workdir',
dest='ubuntu_mirror', dest='workdir',
default=None, default=None,
help='Preferred Ubuntu mirror (default: Launchpad)', help='Specify a working directory '
metavar='INSTANCE') '(default: temporary dir)',
p.add_option('-l', '--lpinstance', metavar='WORKDIR')
dest='lpinstance', parser.add_option('-m', '--mirror',
default=None, dest='ubuntu_mirror',
help='Launchpad instance to connect to (default: production)', default=None,
metavar='INSTANCE') help='Preferred Ubuntu mirror (default: Launchpad)',
p.add_option('--no-conf', metavar='INSTANCE')
dest='no_conf', parser.add_option('-l', '--lpinstance',
default=False, dest='lpinstance',
help="Don't read config files or environment variables", default=None,
action='store_true') help='Launchpad instance to connect to '
'(default: production)',
metavar='INSTANCE')
parser.add_option('--no-conf',
dest='no_conf',
default=False,
help="Don't read config files or environment variables",
action='store_true')
opts, args = p.parse_args(args) opts, args = parser.parse_args(args)
if len(args) != 1: if len(args) != 1:
p.error('You must specify a single source package or a .dsc URL/path.') parser.error('You must specify a single source package or a .dsc '
'URL/path.')
config = UDTConfig(opts.no_conf) config = UDTConfig(opts.no_conf)
if opts.builder is None: if opts.builder is None:
opts.builder = config.get_value('BUILDER') opts.builder = config.get_value('BUILDER')
@ -130,7 +138,7 @@ def parse(args):
if opts.ubuntu_mirror is None: if opts.ubuntu_mirror is None:
opts.ubuntu_mirror = config.get_value('UBUNTU_MIRROR') opts.ubuntu_mirror = config.get_value('UBUNTU_MIRROR')
if not opts.upload and not opts.workdir: if not opts.upload and not opts.workdir:
p.error('Please specify either a working dir or an upload target!') parser.error('Please specify either a working dir or an upload target!')
return opts, args return opts, args