You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by si...@apache.org on 2007/04/21 01:58:58 UTC

svn commit: r530950 - /spamassassin/trunk/t/mkrules.t

Author: sidney
Date: Fri Apr 20 16:58:58 2007
New Revision: 530950

URL: http://svn.apache.org/viewvc?view=rev&rev=530950
Log:
replace nonportable use of system mv with File::Copy::move

Modified:
    spamassassin/trunk/t/mkrules.t

Modified: spamassassin/trunk/t/mkrules.t
URL: http://svn.apache.org/viewvc/spamassassin/trunk/t/mkrules.t?view=diff&rev=530950&r1=530949&r2=530950
==============================================================================
--- spamassassin/trunk/t/mkrules.t (original)
+++ spamassassin/trunk/t/mkrules.t Fri Apr 20 16:58:58 2007
@@ -3,6 +3,7 @@
 use lib '.'; use lib 't';
 use SATest; sa_t_init("mkrules");
 use Test; BEGIN { plan tests => 81 };
+use File::Copy;
 use File::Path;
 
 # ---------------------------------------------------------------------------
@@ -398,8 +399,7 @@
 
 sub save_tdir {
   rmtree("$tdir.${Test::ntest}");
-  system("mv $tdir $tdir.${Test::ntest}");
-  if ($? >> 8 == 0) {
+  if (move( "$tdir", "$tdir.${Test::ntest}")) {
     print "\ttest output tree copied to $tdir.${Test::ntest}\n";
   }
 }