[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Ronald Oussoren
Ronald Oussoren added the comment: Thanks for the patch. I've committed it to 2.7 as well. -- resolution: -> fixed stage: -> committed/rejected status: open -> closed type: -> behavior versions: +Python 2.7 ___ Python tracker

[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Roundup Robot
Roundup Robot added the comment: New changeset c394f2d42ff8 by Ronald Oussoren in branch '3.1': Issue #11569: use absolute path to the sysctl command in multiprocessing to http://hg.python.org/cpython/rev/c394f2d42ff8 New changeset b8f280d0cdbf by Ronald Oussoren in branch '2.7': Issue #11569:

[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Jesse Noller
Jesse Noller added the comment: Ronald - if you're OK w/ patch 2, please commit! -- ___ Python tracker ___ ___ Python-bugs-list maili

[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Ronald Oussoren
Ronald Oussoren added the comment: The patch looks fine. I'd prefer the second patch as this always uses an absolute path (I've ran into numerous problems in the past with scripts that assumed that the commands they looked for where on $PATH and were the system version instead of some local

[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Steffen Daode Nurpmeso
Steffen Daode Nurpmeso added the comment: Maybe i should have used 'hg anno' first ... next time i will. This new patch always uses an absolute path. I have not looked into os.popen() yet, but i do put trust in that it makes a difference :) The reason was nonetheless: 13:29 ~/usr/opt $ python3

[issue11569] multiprocessing/darwin: sysctl(8) mislocation

2011-03-16 Thread Steffen Daode Nurpmeso
New submission from Steffen Daode Nurpmeso : Hello Mac OS X gurus, multiprocessing/__init__py. searches for sysctl(8) via os.popen(), which will fail since the moist fruits provide sysctl(8) in /usr/sbin! The applied patch multiproc_sysctl.patch uses an absolute path. (This may be better anyway,