You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@spamassassin.apache.org by jm...@apache.org on 2008/08/13 23:52:58 UTC

svn commit: r685690 - /spamassassin/trunk/Makefile.PL

Author: jm
Date: Wed Aug 13 14:52:57 2008
New Revision: 685690

URL: http://svn.apache.org/viewvc?rev=685690&view=rev
Log:
bug 5908: exit 0 instead of 1 if a required module is missing; this will avoid broken cpan-tester mails

Modified:
    spamassassin/trunk/Makefile.PL

Modified: spamassassin/trunk/Makefile.PL
URL: http://svn.apache.org/viewvc/spamassassin/trunk/Makefile.PL?rev=685690&r1=685689&r2=685690&view=diff
==============================================================================
--- spamassassin/trunk/Makefile.PL (original)
+++ spamassassin/trunk/Makefile.PL Wed Aug 13 14:52:57 2008
@@ -391,7 +391,9 @@
 use Mail::SpamAssassin::Util::DependencyInfo;
 if (Mail::SpamAssassin::Util::DependencyInfo::long_diagnostics() != 0) {
   # missing required module?  die!
-  exit 1;
+  # bug 5908: http://cpantest.grango.org/wiki/CPANAuthorNotes says
+  # we should exit with a status of 0, but without creating Makefile
+  exit 0;
 }
 
 foreach my $file (@FILES_THAT_MUST_EXIST) {