Re: [Lldb-commits] [PATCH] D22457: Unify process launching code on linux

2016-07-21 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL276288: Unify process launching code on linux (authored by labath). Changed prior to commit: https://reviews.llvm.org/D22457?vs=64296&id=64894#toc Repository: rL LLVM https://reviews.llvm.org/D22457

Re: [Lldb-commits] [PATCH] D22457: Unify process launching code on linux

2016-07-21 Thread Pavel Labath via lldb-commits
labath marked 2 inline comments as done. labath added a comment. https://reviews.llvm.org/D22457 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D22457: Unify process launching code on linux

2016-07-21 Thread Tamas Berghammer via lldb-commits
tberghammer accepted this revision. tberghammer added a comment. This revision is now accepted and ready to land. Looks good with 2 minor comments Comment at: source/Host/linux/ProcessLauncherLinux.cpp:28-29 @@ +27,4 @@ + +// If there is no PATH variable specified inside the env