svn commit: r396117 - /spamassassin/trunk/build/mkrules

[jm...@apache.org]

Author: jm
Date: Sat Apr 22 05:42:05 2006
New Revision: 396117

URL: http://svn.apache.org/viewcvs?rev=396117&view=rev
Log:
don't issue a warning about the T_ prefix thing all the time, it's too noisy

Modified:
    spamassassin/trunk/build/mkrules

Modified: spamassassin/trunk/build/mkrules
URL: http://svn.apache.org/viewcvs/spamassassin/trunk/build/mkrules?rev=396117&r1=396116&r2=396117&view=diff
==============================================================================
--- spamassassin/trunk/build/mkrules (original)
+++ spamassassin/trunk/build/mkrules Sat Apr 22 05:42:05 2006
@@ -745,6 +745,7 @@
   my ($rule, $path) = @_;
   my $new;
   my $newreason;
+  my $dowarn = 0;
 
   return $rule if $opt_listpromotable;
   return $rule if $active_rules->{$rule};
@@ -764,11 +765,14 @@
     $new =~ s/_$//;
     $new = $rule.'_'.$new;
     $newreason = "collision with existing rule";
+    $dowarn = 1;
   }
 
   if (!$renamed_rules->{$new}) {
     $renamed_rules->{$new} = $rule;
-    warn "WARNING: $rule: renamed as $new due to $newreason\n";
+    if ($dowarn) {
+      warn "WARNING: $rule: renamed as $new due to $newreason\n";
+    }
   }
 
   return $new;