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...@issues.apache.org on 2010/07/28 10:33:51 UTC

[Bug 6470] New: win32: cannot determine exit status of child in Util.pm

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

           Summary: win32: cannot determine exit status of child in
                    Util.pm
           Product: Spamassassin
           Version: 3.3.1
          Platform: PC
        OS/Version: Windows 7
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: lemke@jam-software.com


Created an attachment (id=4792)
 --> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4792)
bugfix for posix exit status issue under Windows in Util.pm

Again, several Posix commands may cause problems when Spamd is used in a
Windows environment. If a child terminates and there is a exit status set, a
posix command will be called which causes Spamd to hang up. 

Proposed bugfix attached.

Daniel

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

[Bug 6470] win32: cannot determine exit status of child in Util.pm

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

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

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

--- Comment #2 from Daniel Lemke <le...@jam-software.com> 2010-07-29 06:32:51 EDT ---
(In reply to comment #1)
> I haven't tested this but as with similar patches, it is clearly logic that
> affects only windows and I'm +1 for submitting it to trunk and branch.  Though
> for consistency, I'd likely change the patch to:
> 
> +    $str = 'exit (running under Windows, cannot determine exit status)';
> 
> BTW, didn't you send in a CLA because it doesn't appear in the Submitter/CLA
> Status above?
> 
> KAM

Indeed, but I think it was mentioned that the cla status in bugzilla has to be
stated explicitly.

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

[Bug 6470] [review] win32: cannot determine exit status of child in Util.pm

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.3.2
            Summary|win32: cannot determine     |[review] win32: cannot
                   |exit status of child in     |determine exit status of
                   |Util.pm                     |child in Util.pm
  Status Whiteboard|                            |needs 1 vote for 3.3.2

--- Comment #3 from Mark Martinec <Ma...@ijs.si> 2011-05-19 13:38:36 UTC ---
This is very much a no-brainer, affecting Windows only.
+1 for 3.3.2

trunk:
  Bug 6470: win32 cannot determine exit status of child in Util.pm
  Sending lib/Mail/SpamAssassin/Util.pm
Committed revision 1124766.

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

[Bug 6470] [review] win32: cannot determine exit status of child in Util.pm

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
  Status Whiteboard|needs 1 vote for 3.3.2      |rtc

--- Comment #5 from Kevin A. McGrail <km...@pccc.com> 2011-05-27 21:39:56 UTC ---
Explicitly stating my +1 and committing to 3.3.2.

Committed to 3.3.2

Author: kmcgrail
Date: Fri May 27 21:27:36 2011
New Revision: 1128465

URL: http://svn.apache.org/viewvc?rev=1128465&view=rev
Log:
Small change in windows to exit stating that the exit status is unknown. 
Thanks to Daniel Lemke for many of these small win32 patches.  Bug #6470.

Modified:
    spamassassin/branches/3.3/META.yml
    spamassassin/branches/3.3/lib/Mail/SpamAssassin/Util.pm


Also reversed the submission of META.yml by accident but changed the revision
to reflect 3.3.2.

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

[Bug 6470] win32: cannot determine exit status of child in Util.pm

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

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

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

--- Comment #1 from Kevin A. McGrail <km...@pccc.com> 2010-07-28 13:03:06 EDT ---
I haven't tested this but as with similar patches, it is clearly logic that
affects only windows and I'm +1 for submitting it to trunk and branch.  Though
for consistency, I'd likely change the patch to:

+    $str = 'exit (running under Windows, cannot determine exit status)';

BTW, didn't you send in a CLA because it doesn't appear in the Submitter/CLA
Status above?

KAM

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

[Bug 6470] [review] win32: cannot determine exit status of child in Util.pm

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

Justin Mason <jm...@jmason.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jm@jmason.org

--- Comment #4 from Justin Mason <jm...@jmason.org> 2011-05-26 21:24:22 UTC ---
+1

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