diff options
| author | Mike Frysinger <vapier@google.com> | 2019-09-30 23:47:03 -0400 |
|---|---|---|
| committer | Mike Frysinger <vapier@google.com> | 2019-10-01 05:43:51 +0000 |
| commit | dc1b59d2c0a7ee00b7e6b111285360c4cff32d2b (patch) | |
| tree | 504d0452e5a31556bbcc3b0f1a0e821001d22938 /subcmds | |
| parent | 71b0f312b15b597ab54d4d3bd6629efdcf188884 (diff) | |
| download | git-repo-dc1b59d2c0a7ee00b7e6b111285360c4cff32d2b.tar.gz | |
forall: exit 1 if we skip any repos
If a repo doesn't exist (e.g. an incomplete sync), make sure we exit
non-zero when they get skipped.
Change-Id: Ifff711e374416b1e6b9b8da4fdc6f14b27ced450
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/239235
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Tested-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'subcmds')
| -rw-r--r-- | subcmds/forall.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/forall.py b/subcmds/forall.py index 0be8d3bc..c9de26bc 100644 --- a/subcmds/forall.py +++ b/subcmds/forall.py | |||
| @@ -323,10 +323,10 @@ def DoWork(project, mirror, opt, cmd, shell, cnt, config): | |||
| 323 | cwd = project['worktree'] | 323 | cwd = project['worktree'] |
| 324 | 324 | ||
| 325 | if not os.path.exists(cwd): | 325 | if not os.path.exists(cwd): |
| 326 | if (opt.project_header and opt.verbose) \ | 326 | if ((opt.project_header and opt.verbose) |
| 327 | or not opt.project_header: | 327 | or not opt.project_header): |
| 328 | print('skipping %s/' % project['relpath'], file=sys.stderr) | 328 | print('skipping %s/' % project['relpath'], file=sys.stderr) |
| 329 | return | 329 | return 1 |
| 330 | 330 | ||
| 331 | if opt.project_header: | 331 | if opt.project_header: |
| 332 | stdin = subprocess.PIPE | 332 | stdin = subprocess.PIPE |