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 2007/02/28 00:40:41 UTC

svn commit: r512496 - /spamassassin/trunk/sa-compile.raw

Author: jm
Date: Tue Feb 27 15:40:40 2007
New Revision: 512496

URL: http://svn.apache.org/viewvc?view=rev&rev=512496
Log:
oops, another point where sudo is required

Modified:
    spamassassin/trunk/sa-compile.raw

Modified: spamassassin/trunk/sa-compile.raw
URL: http://svn.apache.org/viewvc/spamassassin/trunk/sa-compile.raw?view=diff&rev=512496&r1=512495&r2=512496
==============================================================================
--- spamassassin/trunk/sa-compile.raw (original)
+++ spamassassin/trunk/sa-compile.raw Tue Feb 27 15:40:40 2007
@@ -196,6 +196,9 @@
   my $dirpath = Mail::SpamAssassin::Util::secure_tmpdir();
   die "secure_tmpdir failed" unless $dirpath && -w $dirpath;
 
+  my $sudo = '';
+  $opt{sudo} and $sudo = 'sudo ';
+
   foreach my $ruletype (sort keys %{$conf->{base_orig}})
   {
     open OUT, ">$dirpath/bases.in"
@@ -209,13 +212,11 @@
     run(get_perl()." Makefile.PL ".
           "PREFIX=$dirpath/ignored INSTALLSITEARCH=$installdir");
 
-    my $sudo = '';
-    $opt{sudo} and $sudo = 'sudo ';
     run($sudo."make install");	# into $installdir
   }
 
   if (!$opt{'keep-tmps'}) {
-    run("rm -rf $dirpath");       # cleanup
+    run($sudo."rm -rf $dirpath");       # cleanup
   }
   else {
     print "temporary dir left due to --keep-tmps: $dirpath\n";