setupinfo.py flags() does not run on Windows

Bug #181508 reported by Koen
2
Affects Status Importance Assigned to Milestone
lxml
Fix Released
Low
Unassigned

Bug Description

on line 139 in setupinfo.py (function flags()), close_fds is set to True:
        p = subprocess.Popen(cmd, shell=True, close_fds=True,
                             stdout=subprocess.PIPE, stderr=subprocess.PIPE)
However, on Windows, close_fds is not available, so I've set it to False to make it work for me. Probably should be changed to a condition check.
        p = subprocess.Popen(cmd, shell=True, close_fds=False,
                             stdout=subprocess.PIPE, stderr=subprocess.PIPE)

Revision history for this message
Koen (ksande) wrote :

This is using Python 2.5.1 on Windows XP (32-bit).

Revision history for this message
scoder (scoder) wrote :

Hmm, I didn't add this myself - no idea why it should be needed. Guess I'll just remove it...

Changed in lxml:
importance: Undecided → Low
status: New → Fix Committed
scoder (scoder)
Changed in lxml:
milestone: none → 2.0
scoder (scoder)
Changed in lxml:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.