[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: FYI I modified subprocess to use os.posix_spawnp() when it's available: New changeset 07858894689047c77f9c12ddc061d30681368d19 by Victor Stinner in branch 'master': bpo-35537: subprocess can now use os.posix_spawnp (GH-11579)

[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: Well done Joannah Nanjekye :-) Thanks to Joannah for this nice enhancemenet and to all reviewers. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: New changeset 92b8322e7ea04b239cb1cb87b78d952f13ddfebb by Victor Stinner (Joannah Nanjekye) in branch 'master': bpo-35674: Add os.posix_spawnp() (GH-11554) https://github.com/python/cpython/commit/92b8322e7ea04b239cb1cb87b78d952f13ddfebb --

[issue35674] Expose os.posix_spawnp()

Change by Joannah Nanjekye : -- pull_requests: +11187, 11188, 11189 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35674] Expose os.posix_spawnp()

Change by Joannah Nanjekye : -- pull_requests: +11187 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35674] Expose os.posix_spawnp()

Change by Joannah Nanjekye : -- pull_requests: +11187, 11188 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35674] Expose os.posix_spawnp()

Change by Roundup Robot : -- keywords: +patch, patch pull_requests: +11158, 11159 stage: -> patch review ___ Python tracker ___

[issue35674] Expose os.posix_spawnp()

Change by Roundup Robot : -- keywords: +patch, patch, patch pull_requests: +11158, 11159, 11161 stage: -> patch review ___ Python tracker ___

[issue35674] Expose os.posix_spawnp()

Change by Roundup Robot : -- keywords: +patch, patch, patch, patch pull_requests: +11158, 11159, 11160, 11161 stage: -> patch review ___ Python tracker ___

[issue35674] Expose os.posix_spawnp()

Change by Roundup Robot : -- keywords: +patch pull_requests: +11158 stage: -> patch review ___ Python tracker ___ ___

[issue35674] Expose os.posix_spawnp()

Change by Alexey Izbyshev : -- nosy: +izbyshev ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35674] Expose os.posix_spawnp()

Joannah Nanjekye added the comment: Now that there is consesus, I am working on a PR for this. -- ___ Python tracker ___ ___

[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: Pablo: > I also think to expose posix_spawnp() as os.posix_spawnp() seems the more > consistent thing to do as the os/posix module tries to mirror glibc as much > as possible, which helps with discoverability and cross-reference with man > pages. Ok. Let's

[issue35674] Expose os.posix_spawnp()

Pablo Galindo Salgado added the comment: I also think to expose posix_spawnp() as os.posix_spawnp() seems the more consistent thing to do as the os/posix module tries to mirror glibc as much as possible, which helps with discoverability and cross-reference with man pages. --

[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: I'm ok to expose posix_spawnp() as os.posix_spawnp(). Even if we expose posix_spawnp() as os.posix_spawnp(), we can still reconsider to add posix_spawnp() feature into os.posix_spawn() as an optional keyword parameter later :-) Honestly, I have no strong

[issue35674] Expose os.posix_spawnp()

Joannah Nanjekye added the comment: I would go with exposing posix_spawnp() as a separate function to rule out any arguments. I am working on this in this regard unless anyone has a different view. -- nosy: +nanjekyejoannah ___ Python tracker

[issue35674] Expose os.posix_spawnp()

Serhiy Storchaka added the comment: To add some context, the follow_symlinks parameter was added to os.stat() after adding support of *at() functions (like openat() and fstatat()). Since both C functions stat() and lstat() are superseded by fstatat(), the latter was exposed at Python level

[issue35674] Expose os.posix_spawnp()

STINNER Victor added the comment: Historically, Python used to mimick the libc: there are os.stat() and os.lstat() for example. But later, os.stat(*, follow_symlinks=False) keyword-only parameter has been added: if true, lstat() is used internally. --

[issue35674] Expose os.posix_spawnp()

New submission from STINNER Victor : bpo-20104 exposed os.posix_spawn(), but not os.posix_spawnp(). os.posix_spawnp() would be useful to support executable with no directory. See bpo-35537 "use os.posix_spawn in subprocess". I'm not sure what is the best API: * Add os.posix_spawnp()?