On Tue, Aug 30, 2011 at 08:31:04AM +0200, Ales Kozumplik wrote:
> On 08/29/2011 08:50 PM, Brian C. Lane wrote:
> >Is this any safer? Seems like Popen is still going to have the same file
> >open twice. In RHEL6 we seem to be using pipes and tee to manage the
> >output streams from the process.
> Yes, but I wouldn't like to fix this particular bug at the cost of
> merging all of that.
> I think in fact both of the ways are 'safe': it is just the
> operating system doesn't advance the seek position of each of the
> (underlying) descriptors (which are indeed used by subprocess.py by
> calling .fileno() on the file object) in the old version, but it
> does that in the second, just as described by man 2 open, section
> Does that convince you? We might see a bug or two about this in the
> future but it is already broken at the moment.
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
Anaconda-devel-list mailing list