[Lldb-commits] [PATCH] D34236: Delete ProcessLauncherPosix

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

Unfortunately, posix_spawn does not satisfy all our needs for launching processes on non-darwin platforms (according to its design rationale, that was never the intention). The most notable one is the "launch a process for debugging". Darwin seems to have added extensions which make it possible, so it makes sense to keep using it. However, it also makes the usage of that function non-portable. The only reason FreeBSD was able to get away with it was because it rolled it's own version of "launch for debug" code (source/Plugins/Process/FreeBSD/ProcessMonitor.cpp). After switching to a more generic launching primitive, that code could disappear.

So, overall, I sense no opposition to this from the darwin or netbsd side. I'll wait with it a while though, to see if the freebsd side wakes up.


https://reviews.llvm.org/D34236