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";