diff options
| author | Che-Liang Chiou <clchiou@google.com> | 2012-10-25 13:44:11 -0700 |
|---|---|---|
| committer | Che-Liang Chiou <clchiou@google.com> | 2012-10-25 13:55:49 -0700 |
| commit | ab8f911a6721424ddc7cda7ebd2a07270a5909b1 (patch) | |
| tree | cf4b0cfab756dc78747894189ef918a262811d48 /subcmds/sync.py | |
| parent | 608aff7f624e35348ff9fab74bad1d6921944238 (diff) | |
| download | git-repo-ab8f911a6721424ddc7cda7ebd2a07270a5909b1.tar.gz | |
Fix pylint warnings introduced by the submodule patch
"69998b0 Represent git-submodule as nested projects" has introduced a
few pylint warnings.
W0612:1439,8:Project._GetSubmodules.get_submodules: Unused variable 'sub_gitdir'
W0613:1424,36:Project._GetSubmodules.get_submodules: Unused argument 'path'
W0612:1450,25:Project._GetSubmodules.parse_gitmodules: Unused variable 'e'
W0622:516,8:Sync.Execute: Redefining built-in 'all'
Change-Id: I84378e2832ed1b5ab023e394d53b22dcea799ba4
Diffstat (limited to 'subcmds/sync.py')
| -rw-r--r-- | subcmds/sync.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 28c154a0..64ed38e7 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
| @@ -513,9 +513,9 @@ uncommitted changes are present' % project.relpath | |||
| 513 | previously_missing_set = set() | 513 | previously_missing_set = set() |
| 514 | while True: | 514 | while True: |
| 515 | self.manifest._Unload() | 515 | self.manifest._Unload() |
| 516 | all = self.GetProjects(args, missing_ok=True) | 516 | all_projects = self.GetProjects(args, missing_ok=True) |
| 517 | missing = [] | 517 | missing = [] |
| 518 | for project in all: | 518 | for project in all_projects: |
| 519 | if project.gitdir not in fetched: | 519 | if project.gitdir not in fetched: |
| 520 | missing.append(project) | 520 | missing.append(project) |
| 521 | if not missing: | 521 | if not missing: |