You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by gb...@apache.org on 2020/01/19 14:15:35 UTC

svn commit: r1872979 - /spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm

Author: gbechis
Date: Sun Jan 19 14:15:35 2020
New Revision: 1872979

URL: http://svn.apache.org/viewvc?rev=1872979&view=rev
Log:
print filename where the wrong rule is located in lint mode
makes debug easier

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm?rev=1872979&r1=1872978&r2=1872979&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm Sun Jan 19 14:15:35 2020
@@ -420,21 +420,21 @@ sub parse {
       {
         $parse_error = "config: SpamAssassin failed to parse line, ".
                         "\"$value\" is not valid for \"$key\", ".
-                        "skipping: $line";
+                        "skipping: \"$line\" in $self->{currentfile}";
         goto failed_line;
       }
       elsif ($ret eq $Mail::SpamAssassin::Conf::INVALID_HEADER_FIELD_NAME)
       {
         $parse_error = "config: SpamAssassin failed to parse line, ".
                        "it does not specify a valid header field name, ".
-                       "skipping: $line";
+                       "skipping: \"$line\" in $self->{currentfile}";
         goto failed_line;
       }
       elsif ($ret eq $Mail::SpamAssassin::Conf::MISSING_REQUIRED_VALUE)
       {
         $parse_error = "config: SpamAssassin failed to parse line, ".
                         "no value provided for \"$key\", ".
-                        "skipping: $line";
+                        "skipping: \"$line\" in $self->{currentfile}";
         goto failed_line;
       }
       else {