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 2006/08/14 15:36:11 UTC
svn commit: r431344 -
/spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision
Author: jm
Date: Mon Aug 14 06:36:11 2006
New Revision: 431344
URL: http://svn.apache.org/viewvc?rev=431344&view=rev
Log:
ensure /tmp/rulemd dir is always removed
Modified:
spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision
Modified: spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision
URL: http://svn.apache.org/viewvc/spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision?rev=431344&r1=431343&r2=431344&view=diff
==============================================================================
--- spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision (original)
+++ spamassassin/trunk/masses/rule-qa/get-rulemetadata-for-revision Mon Aug 14 06:36:11 2006
@@ -54,11 +54,28 @@
print "creating rule metadata file: $outfile\n";
my $tmpdir = "/tmp/rulemd.$$";
-svn_and_build();
-copy_md($outfile);
-system ("rm -rf $tmpdir");
+
+$SIG{TERM} = $SIG{INT} = sub {
+ cleanup_tmpdir(); die "killed";
+};
+
+eval {
+ svn_and_build();
+ copy_md($outfile);
+};
+if ($@) {
+ my $err = $@;
+ cleanup_tmpdir();
+ die $err;
+}
+
+cleanup_tmpdir();
exit;
+
+sub cleanup_tmpdir {
+ system ("rm -rf $tmpdir");
+}
sub configure {