You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2007/01/25 21:27:04 UTC

[Bug 5309] New: Config parse errors should show the file the config comes from

http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5309

           Summary: Config parse errors should show the file the config
                    comes from
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: minor
          Priority: P5
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: felicity@apache.org


It's come up on the list that it's not too easy to find where a config line
parse error comes from.  Generally speaking, we tell people to run a message
through with -D, see the list of configs read, then grep each of those to find
the bad line.  It'd be easier if we said "the following line in file
/foo/bar/baz.cf has an error: this is an invalid line" or something.  We already
do that for conditionals in config files via $self->{currentfile}, so it should
be pretty simple to use that for other error messages as well.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 5309] Config parse errors should show the file the config comes from

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5309


felicity@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.2.0






------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 5309] Config parse errors should show the file the config comes from

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5309





------- Additional Comments From jm@jmason.org  2007-01-25 15:29 -------
I think I've already implemented this for 3.2.0, btw.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 5309] Config parse errors should show the file the config comes from

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5309


felicity@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WORKSFORME




------- Additional Comments From felicity@apache.org  2007-01-25 16:11 -------
(In reply to comment #1)
> I think I've already implemented this for 3.2.0, btw.

Hrm.  Indeed, it appears that r489437 includes that stuff in Conf::Parser.  Ok,
nm. :)



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.