[Lldb-commits] [PATCH] D105732: [lldb] Update logic to close inherited file descriptors.

[Jim Ingham via lldb-commits]

Sure, thanks for the context!  

Actually, the posix_spawn usage is only in the darwin Host code, probably because for posix_spawn to work for debugging
you need a couple of non-standard flags that were added on Darwin for us (the CoreOS folks really wanted us to use posix_spawn).
So you probably can't use it on Linux w/o some kernel support.

Jim


> On Aug 12, 2021, at 3:56 PM, David Blaikie via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> dblaikie added a comment.
> 
> In D105732#2942716 <https://reviews.llvm.org/D105732#2942716>, @jingham wrote:
> 
>> Do modern Linux's not have posix_spawn?  If it exists that's a better interface, and lets the system handle a lot of the complicated machinations you have to do by hand if you roll it yourself out of fork and exec.
> 
> https://man7.org/linux/man-pages/man3/posix_spawn.3.html - looks like it's there. I was reporting on the current implementation/adding some detail on the current state of things.
> 
> 
> CHANGES SINCE LAST ACTION
>  https://reviews.llvm.org/D105732/new/
> 
> https://reviews.llvm.org/D105732
>