|
SA Bugzilla – Full Text Bug Listing |
Summary: | spamc: learning exit codes documentation wrong, possible uncaught error | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Karsten Bräckelmann <guenther> |
Component: | spamc/spamd | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | billcole |
Priority: | P5 | ||
Version: | 3.2.5 | ||
Target Milestone: | Undefined | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: |
Description
Karsten Bräckelmann
2009-02-18 14:22:43 UTC
The fix should be quite trivial, adding 2 lines to set the ret value accordingly. Now I just wonder which one is actually wrong -- the code or the POD... Got the little bugger! :) rev 158134, looks like this was changed intentional, almost 4 years ago. Alas, the documentation hasn't been adjusted. Hmm, so successful learning, unavailable (aka previously learned) and even *service unavailable* all are expected to return 0, success? Seriously? (In reply to comment #2) > rev 158134 http://svn.apache.org/viewvc?view=rev&revision=158134 Hrm, I was hoping for a link. Sorry for the noise. (In reply to comment #2) > rev 158134, looks like this was changed intentional, almost 4 years ago. Alas, > the documentation hasn't been adjusted. > > Hmm, so successful learning, unavailable (aka previously learned) and even > *service unavailable* all are expected to return 0, success? Seriously? More digging. Turns out all of this is bug 1201, revision 158029 and some fixes after that. According to bug 1201 comment 47, 48 and 49 returning EX_OK indeed is the desired behavior. Thus, concerning "successful learned" and "already learned" this is a documentation bug. Still unsure about the removed EX_UNAVAILABLE from revision 158134. Correcting Summary. And hoping the rev hints get linkified this time. |