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 {