all skipped should be a nonzero exit

[code/myrepos.git] / mr

diff --git a/mr b/mr
index 6a525cdff7bf10618b9108f347ab35588b3b36fa..f769eff461ef8241a09679c18682540412a225f3 100755 (executable)
--- a/mr
+++ b/mr
@@ -263,7 +263,13 @@ print "\nmr $action: finished (".join("; ",
        showstat($#failed+1, "failed", "failed"),
        showstat($#skipped+1, "skipped", "skipped"),
 ).")\n";
-exit @failed ? 1 : 0;
+if (@failed) {
+       exit 1;
+}
+elsif (! @successful && @skipped) {
+       exit 1;
+}
+exit 0;
 
 my %loaded;
 sub loadconfig {
@@ -376,8 +382,7 @@ commit = \
        if [ -d .svn ]; then \
                svn commit "$@"; \
        elif [ -d .git ]; then \
-               echo git commit -a "$@" && echo git push --all \
-               git commit -a "$@" && git push --all \
+               git commit -a "$@" && git push --all; \
        else \
                echo "mr commit: unknown repo type"; \
                exit 1; \