[Lldb-commits] [PATCH] D67012: [lit] Only set DYLD_LIBRARY_PATH for shared builds

2019-08-30 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370549: [lit] Only set DYLD_LIBRARY_PATH for shared builds (authored by JDevlieghere, committed by ). Changed prior to commit: https://reviews.llvm.org/D67012?vs=218133&id=218186#toc Repository: rL L

[Lldb-commits] [PATCH] D67012: [lit] Only set DYLD_LIBRARY_PATH for shared builds

2019-08-30 Thread Frederic Riss via Phabricator via lldb-commits
friss accepted this revision. friss added a comment. This revision is now accepted and ready to land. This looks good to me with one small provision: Comment at: lldb/lit/CMakeLists.txt:72 LLVM_ENABLE_ZLIB + LLVM_SHARED_LIBS LLDB_IS_64_BITS) The other bo

[Lldb-commits] [PATCH] D67012: [lit] Only set DYLD_LIBRARY_PATH for shared builds

2019-08-30 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere updated this revision to Diff 218133. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67012/new/ https://reviews.llvm.org/D67012 Files: lldb/lit/CMakeLists.txt lldb/lit/Suite/lit.cfg lldb/lit/Suite/lit.site.cfg.in llvm/cmake/modules/LLVMConfig.cmake.in Index: llvm/cm

[Lldb-commits] [PATCH] D67012: [lit] Only set DYLD_LIBRARY_PATH for shared builds

2019-08-30 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: aprantl, mgorny. Herald added subscribers: llvm-commits, abidh. Herald added projects: LLDB, LLVM. JDevlieghere updated this revision to Diff 218133. In r370135 I committed a temporary workaround for the sanitized bot to not set (D