Currently SBTarget::LaunchSimple creates a new LaunchInfo which means it ignores any target properties that have been set. Instead, it should start from the target's LaunchInfo and populated the specified fields.
|3918 ↗||(On Diff #283100)|
Unfortunately doing this here doesn't work. Some of the code that's using GetProcessLaunchInfo() depends on the assumption that the m_arguments list in ProcessInfo does not contain argv, while other code, most notably the code to actually launch a binary, depends on it being the first argument. So whatever I do here, I break some code.
@JDevlieghere This change is is failing LLDB Windows buildbot since build 18051. Can you please fix that?
Thanks for the heads up. I had just pushed a fix:
commit 927afdffbb1deebd83b86d024b67687d758521d0 Author: Jonas Devlieghere <firstname.lastname@example.org> Date: Wed Aug 5 15:37:50 2020 -0700 [lldb] Skip test_launch_simple on Windows Because stdio manipulation unsupported on Windows.