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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|-e exit code option not     |[review] -e exit code option
                   |working in 3.1.2 on Windows |not working in 3.1.2 on
                   |                            |Windows
  Status Whiteboard|                            |needs 2 votes




------- Additional Comments From [EMAIL PROTECTED]  2006-06-05 18:59 -------
Hrm.  This is caused by the fix for bug 3754.  Basically we needed to kluge in 
a return value when files 
can't be read, etc.  Interestingly, the problem seems to be related to a perl 
"feature".  Specifically, it 
looks like:

eval { $exitvalue ||= !$iter->run(@targets); };

the decision to set $exitvalue (ala "||=") is made first (because $exitvalue is 
undefined), then it's set to 
the return from run() -- however, the wanted() function (called from run()) 
sets the global variable, so 
we get:

$exitvalue = undef; # start
$exitvalue = 5; # set in wanted() via run()
$exitvalue = 1; # set in the eval {}

$ perl -e '$t ||= foo(); sub foo { $t = 5; return 1; } print "$t\n"'
1

So, interesting.  Patch coming shortly.



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

Reply via email to