pkgsubprocess.py:
108-118: I think this might be simpler as:
if not env:
    env = [ fancy list stuff ]

self.pid = posix_spawnp(blah)

pspawn.c:
Comments please? At least for what the general purpose of each function is?
I didn't see other problems here, but that doesn't mean much.

Brock


[EMAIL PROTECTED] wrote:
> Folks,
> We've run into a lot of trouble in recent builds when we attempt to fork
> and are low on memory.  This change doesn't do anything to address the
> actual memory usage; however, it makes it more likely that we'll
> successfully fork a new process when memory is tight.
>
> This change adds an Python interface to posix_spawn(3C) and creates a
> subclass of the subprocess.Popen class to use it.
>
> Webrev can be found at the location below.
>
>       http://cr.opensolaris.org/~johansen/webrev-4612/
>
> Thanks,
>
> -j
> _______________________________________________
> pkg-discuss mailing list
> [email protected]
> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
>   

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to