[Patches] [ python-Patches-713428 ] Fixes for 'commands' module on win32

SourceForge.net noreply@sourceforge.net
Thu, 24 Apr 2003 09:07:45 -0700


Patches item #713428, was opened at 2003-04-01 12:51
Message generated for change (Comment added) made by warnes
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=713428&group_id=5470

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Gregory Warnes (warnes)
Assigned to: Martin v. L÷wis (loewis)
Summary: Fixes for 'commands' module on win32

Initial Comment:
Currently the 'commands' module is provided with python
on Win32 systems, but does not function properly.  (The
library reference says it is only available on Unix).    

The commands module provides 3 functions:

       getoutput(cmd)              returns output
generated by executing 'cmd' in a shell
       getstatusoutput(cmd)     returns (status,output)
generated by executing 'cmd' in a shell
       getstatus(file)                returns output of
"ls -ld file"

Below is a simple patch which enables 'getoutput' and
'getstatusoutut' to work properly on Win32. Since
'getstatusfile' only makes sense on Unix, the patch
prevents it from being defined on Win32.  The current
behavior is maintained on all other platforms.





----------------------------------------------------------------------

>Comment By: Gregory Warnes (warnes)
Date: 2003-04-24 12:07

Message:
Logged In: YES 
user_id=9316

2nd attempt to upload fixed patch.

----------------------------------------------------------------------

Comment By: Martin v. L÷wis (loewis)
Date: 2003-04-24 11:02

Message:
Logged In: YES 
user_id=21627

Apparently, you did not check the checkbox. Can you please 
retry?

----------------------------------------------------------------------

Comment By: Gregory Warnes (warnes)
Date: 2003-04-10 00:50

Message:
Logged In: YES 
user_id=9316

Uploaded a revised patch file, as requested.

----------------------------------------------------------------------

Comment By: Martin v. L÷wis (loewis)
Date: 2003-04-08 02:02

Message:
Logged In: YES 
user_id=21627

Can you please resubmit this as a unified diff? Also, please
make sure you don't incude white-space only changes: Your
patch says that it changes all lines, just because you add a
CR at the end of every line (so either convert the modified
file to Unix conventions, or the original file to Windows
conventions).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=713428&group_id=5470