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

Mark Martinec <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #4622|0                           |1
        is obsolete|                            |

--- Comment #4 from Mark Martinec <[email protected]> 2010-01-07 17:24:16 
UTC ---
Created an attachment (id=4629)
 --> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4629)
proposed patch, v2

> + print "Update was available, but not installed installed in checkonly
> mode\n";
> drop one of those 'installed's.

Thanks, done.

> +  -v, --verbose           Be more verbose, like print updated channel names
> Does the new patch still do that?  (it probably should)

Does it still print updated channel names? Yes it does (like in the
original patch), but prefixed by a text 'Update available for channel '.

This only applies to network downloads. An equivalent print for
updates from a file is commented out, it's probably redundant.


> also, I don't think it needs to exclude the --debug case; if --verbose is
> indeed used too, the user wants both the verbose msgs and the debugs too IMO.

Agreed. It also allows print lines to fit into a single line, nicer!

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to