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 2012/05/31 06:23:08 UTC

[Bug 6800] New: X-Spam-Checker-Version should also show what sa-update we are at

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6800

          Priority: P2
            Bug ID: 6800
          Assignee: dev@spamassassin.apache.org
           Summary: X-Spam-Checker-Version should also show what sa-update
                    we are at
          Severity: enhancement
    Classification: Unclassified
                OS: All
          Reporter: jidanni@jidanni.org
          Hardware: All
            Status: NEW
           Version: SVN Trunk (Latest Devel Version)
         Component: spamassassin
           Product: Spamassassin

The X-Spam-Checker-Version header should also show what sa-update we are at.

Indeed the value of the latter is often more signifigant that that of the
software itself.

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

--- Comment #6 from Mark Martinec <Ma...@ijs.si> ---
Does somebody feel like changing the 10_default_prefs.cf too
to extend the 'add_header all Checker-Version', or do we leave it
as an option to site managers?

I'd prefer to keep it as it is, i.e. with the new code offering a
macro RULESVERSION, but with unchanged default X-Spam-Checker-Version
header field.

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

Mark Martinec <Ma...@ijs.si> changed:

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

--- Comment #9 from Mark Martinec <Ma...@ijs.si> ---
So this is done, closing.

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

Kevin A. McGrail <km...@pccc.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@pccc.com

--- Comment #1 from Kevin A. McGrail <km...@pccc.com> ---
A good idea IMO.  Any chance you can submit a patch?

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

--- Comment #3 from Mark Martinec <Ma...@ijs.si> ---
Created attachment 5074
  --> https://issues.apache.org/SpamAssassin/attachment.cgi?id=5074&action=edit
Collect rules versions and make them available through _RULESVERSION_ macro

See the attached patch. It collects ruleset versions from
their '# UPDATE version' lines, and makes them available
through a macro _RULESVERSION_, which can be used in an
add_header directive, e.g. (single line, mind the wraps) :

add_header all Checker-Version SpamAssassin _VERSION_ (_SUBVERSION_)
(_RULESVERSION_) on _HOSTNAME_


trunk:
  Bug 6800: X-Spam-Checker-Version should also show
  what sa-update we are at'      
Sending lib/Mail/SpamAssassin/Conf/Parser.pm
Sending lib/Mail/SpamAssassin/Conf.pm
Sending lib/Mail/SpamAssassin/PerMsgStatus.pm
Committed revision 1346448.

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

--- Comment #4 from Kevin A. McGrail <km...@pccc.com> ---
Brilliant!

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

--- Comment #2 from jidanni@jidanni.org ---
Thanks but "grandma lost the ability to sew long ago",

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

--- Comment #8 from Kevin A. McGrail <km...@pccc.com> ---
(In reply to comment #7)
> I'd call it a 'nice to have' so no need to change the default.

Concur +1

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

Daniel Lemke <le...@jam-software.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |lemke@jam-software.com

--- Comment #7 from Daniel Lemke <le...@jam-software.com> ---
I'd call it a 'nice to have' so no need to change the default.

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

jidanni@jidanni.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://article.gmane.org/gm
                   |                            |ane.mail.spam.spamassassin.
                   |                            |general/137380
                 CC|                            |jidanni@jidanni.org

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

[Bug 6800] X-Spam-Checker-Version should also show what sa-update we are at

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

Mark Martinec <Ma...@ijs.si> changed:

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

--- Comment #5 from Mark Martinec <Ma...@ijs.si> ---
I just realized the grep is now redundant (a leftover from a version where I
tried to correlate versions with filenames or restrict to a main channel).
Removing it:

trunk:
  Bug 6800 (removed a redundant grep)
  Sending lib/Mail/SpamAssassin/PerMsgStatus.pm
Committed revision 1346458.

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