From: Joey Hess <joey@kitenet.net>
Date: Thu, 16 Dec 2010 16:53:30 +0000 (-0400)
Subject: Do not return a nonzero exit status when all repositories were skipped. Closes: ... 
X-Git-Url: https://git.madduck.net/code/myrepos.git/commitdiff_plain/92d07e8691d464a3676ac6289c578d6217776633?ds=sidebyside

Do not return a nonzero exit status when all repositories were skipped. Closes: #607287
---

diff --git a/debian/changelog b/debian/changelog
index 19edc74..1e1f549 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,8 @@ mr (0.51) UNRELEASED; urgency=low
 
   * Fix display when absolute directories are configured in mrconfig.
   * Add push to manpage synopsis. Closes: #603029
+  * Do not return a nonzero exit status when all repositories were skipped.
+    Closes: #607287
 
  -- Joey Hess <joeyh@debian.org>  Thu, 28 Oct 2010 18:01:27 -0400
 
diff --git a/mr b/mr
index 642aa6f..5a877fe 100755
--- a/mr
+++ b/mr
@@ -427,6 +427,10 @@ mr can be extended to support things such as unison and git-svn. Some
 files providing such extensions are available in /usr/share/mr/. See
 the documentation in the files for details about using them.
 
+=head1 EXIT STATUS
+
+mr returns nonzero if a command failed in any of the repositories.
+
 =head1 AUTHOR
 
 Copyright 2007-2010 Joey Hess <joey@kitenet.net>
@@ -1548,9 +1552,6 @@ sub exitstats {
 	if (@failed) {
 		exit 1;
 	}
-	elsif (! @ok && @skipped) {
-		exit 1;
-	}
 	else {
 		exit 0;
 	}