Over in https://github.com/microsoft/git/issues/623, it was pointed out that scheduled maintenance will error out when it encounters a missing repository. The scheduled maintenance should exit with an error, all right, but what about the remaining repositories for which maintenance was scheduled, and that may not be missing? This patch series addresses this by introducing a new for-each-repo option and then using it in the command that is run via scheduled maintenance. Changes since v1 (thanks Eric!): * Changed the option's documentation to reflect the current state (instead of the original design) * Fixed grammar issues Johannes Schindelin (2): for-each-repo: optionally keep going on an error maintenance: running maintenance should not stop on errors Documentation/git-for-each-repo.txt | 4 ++++ builtin/for-each-repo.c | 8 ++++++-- builtin/gc.c | 7 ++++--- t/t0068-for-each-repo.sh | 16 ++++++++++++++++ t/t7900-maintenance.sh | 6 +++--- 5 files changed, 33 insertions(+), 8 deletions(-) base-commit: 3c2a3fdc388747b9eaf4a4a4f2035c1c9ddb26d0 Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1719%2Fdscho%2Ffor-each-repo-stop-on-error-2.44-v2 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1719/dscho/for-each-repo-stop-on-error-2.44-v2 Pull-Request: https://github.com/gitgitgadget/git/pull/1719 Range-diff vs v1: 1: 6721e3ada5d ! 1: abd796894c8 for-each-repo: optionally keep going on an error @@ Commit message This is undesirable, and points out a gap in the design of `git for-each-repo`: We need a mode where that command does not stop on an - error, but continues to try the running the specified command with the - other repositories. + error, but continues to try running the specified command with the other + repositories. Imitating the `--keep-going` option of GNU make, this commit teaches `for-each-repo` the same trick: to continue with the operation on all the remaining repositories in case there was a problem with one repository, still setting the exit code to indicate an error occurred. + Helped-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx> Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> ## Documentation/git-for-each-repo.txt ## @@ builtin/for-each-repo.c: int cmd_for_each_repo(int argc, const char **argv, cons OPT_STRING(0, "config", &config_key, N_("config"), N_("config key storing a list of repository paths")), + OPT_BOOL(0, "keep-going", &keep_going, -+ N_("stop at the first repository where the operation failed")), ++ N_("keep going even if command fails in a repository")), OPT_END() }; 2: a86bcf2e1a0 = 2: 1ae11553052 maintenance: running maintenance should not stop on errors -- gitgitgadget
