all skipped should be a nonzero exit

[code/myrepos.git] / mr

diff --git a/mr b/mr
index 8d90af34ca4ac89f801efcbd63476cf06f4a08ed..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,7 +382,7 @@ commit = \
        if [ -d .svn ]; then \
                svn commit "$@"; \
        elif [ -d .git ]; then \
-               git commit -a "$@" && git push --all \
+               git commit -a "$@" && git push --all; \
        else \
                echo "mr commit: unknown repo type"; \
                exit 1; \