diff options
author | Guido Günther <agx@sigxcpu.org> | 2015-08-13 09:51:46 +0200 |
---|---|---|
committer | Guido Günther <agx@sigxcpu.org> | 2015-08-13 09:51:46 +0200 |
commit | e1deb46a0dc5ded356f7ed9084f6180c932b5f49 (patch) | |
tree | 5322abc1c4d3b8428fac1e9c782fb5ab48a62570 | |
parent | 97c7a32d82631dba910d8632478fc093cbe2bcda (diff) |
pull: make pylint happy
Gbp-Dch: Ignore
-rwxr-xr-x | gbp/scripts/pull.py | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/gbp/scripts/pull.py b/gbp/scripts/pull.py index e0063940..67b2ab1c 100755 --- a/gbp/scripts/pull.py +++ b/gbp/scripts/pull.py @@ -21,7 +21,8 @@ from six.moves import configparser import sys -import os, os.path +import os +import os.path from gbp.command_wrappers import (Command, CommandExecFailed) from gbp.config import (GbpOptionParser, GbpOptionGroup) from gbp.errors import GbpError @@ -29,6 +30,7 @@ from gbp.git import GitRepositoryError from gbp.deb.git import DebianGitRepository import gbp.log + def fast_forward_branch(branch, repo, options): """ update branch to its remote branch, fail on non fast forward updates @@ -45,7 +47,7 @@ def fast_forward_branch(branch, repo, options): can_fast_forward, up_to_date = repo.is_fast_forward(branch, remote) - if up_to_date: # Great, we're done + if up_to_date: # Great, we're done gbp.log.info("Branch '%s' is already up to date." % branch) return True @@ -72,18 +74,18 @@ def fast_forward_branch(branch, repo, options): def build_parser(name): try: parser = GbpOptionParser(command=os.path.basename(name), prefix='', - usage='%prog [options] - safely update a repository from remote') + usage='%prog [options] - safely update a repository from remote') except configparser.ParsingError as err: gbp.log.err(err) return None branch_group = GbpOptionGroup(parser, "branch options", "branch update and layout options") parser.add_option_group(branch_group) - branch_group.add_boolean_config_file_option(option_name = "ignore-branch", dest="ignore_branch") + branch_group.add_boolean_config_file_option(option_name="ignore-branch", dest="ignore_branch") branch_group.add_option("--force", action="store_true", dest="force", default=False, - help="force a branch update even if it can't be fast forwarded") + help="force a branch update even if it can't be fast forwarded") branch_group.add_option("--redo-pq", action="store_true", dest="redo_pq", default=False, - help="redo the patch queue branch after a pull. Warning: this drops the old patch-queue branch") + help="redo the patch queue branch after a pull. Warning: this drops the old patch-queue branch") branch_group.add_config_file_option(option_name="upstream-branch", dest="upstream_branch") branch_group.add_config_file_option(option_name="debian-branch", dest="debian_branch") branch_group.add_boolean_config_file_option(option_name="pristine-tar", dest="pristine_tar") @@ -126,18 +128,18 @@ def main(argv): current = repo.get_branch() except GitRepositoryError: # Not being on any branch is o.k. with --git-ignore-branch - if options.ignore_branch: + if options.ignore_branch: current = repo.head gbp.log.info("Found detached head at '%s'" % current) else: raise - for branch in [ options.debian_branch, options.upstream_branch ]: + for branch in [options.debian_branch, options.upstream_branch]: if repo.has_branch(branch): - branches += [ branch ] + branches += [branch] if repo.has_pristine_tar_branch() and options.pristine_tar: - branches += [ repo.pristine_tar_branch ] + branches += [repo.pristine_tar_branch] (ret, out) = repo.is_clean() if not ret: |