svn commit: r1061373 - in /perl/Apache-Test/trunk: Changes lib/Apache/TestRun.pm

[ph...@apache.org]

Author: phred
Date: Thu Jan 20 16:31:28 2011
New Revision: 1061373

URL: http://svn.apache.org/viewvc?rev=1061373&view=rev
Log:
Return shell 0 on exit, instead of exit_perl(0), which is shell non-zero.

Thanks to Michael Schout for the spot.

Modified:
    perl/Apache-Test/trunk/Changes
    perl/Apache-Test/trunk/lib/Apache/TestRun.pm

Modified: perl/Apache-Test/trunk/Changes
URL: http://svn.apache.org/viewvc/perl/Apache-Test/trunk/Changes?rev=1061373&r1=1061372&r2=1061373&view=diff
==============================================================================
--- perl/Apache-Test/trunk/Changes (original)
+++ perl/Apache-Test/trunk/Changes Thu Jan 20 16:31:28 2011
@@ -8,6 +8,9 @@ Changes - Apache::Test change logfile
 
 =item 1.35-dev
 
+Return value on running tests as root should be 0, not 1.  Thanks to Michael Schout for the spot.
+[Fred Moyer]
+
 Add support for RFC2253 DN string format to dn_oneline in Apache::TestSSLCA
 [Stefan Fritsch]
 

Modified: perl/Apache-Test/trunk/lib/Apache/TestRun.pm
URL: http://svn.apache.org/viewvc/perl/Apache-Test/trunk/lib/Apache/TestRun.pm?rev=1061373&r1=1061372&r2=1061373&view=diff
==============================================================================
--- perl/Apache-Test/trunk/lib/Apache/TestRun.pm (original)
+++ perl/Apache-Test/trunk/lib/Apache/TestRun.pm Thu Jan 20 16:31:28 2011
@@ -947,7 +947,7 @@ sub check_runtime_user {
 
     if ($user eq 'root') {
         error "Apache cannot spawn child processes as root, therefore the test suite must be run as a non-privileged user.";
-        exit_perl(0);
+        exit_perl(1);
     }
 
     return 1;