[Lldb-commits] [PATCH] D25886: [Test Suite] Properly respect --framework option
Chris Bieneman via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon May 8 11:39:39 PDT 2017
beanz added inline comments.
Comment at: lldb/trunk/packages/Python/lldbsuite/test/lldbtest.py:1418
- 'LD_EXTRAS': "-L%s -llldb" % lib_dir}
+ 'LD_EXTRAS': "-L%s/../lib -llldb -Wl,-rpath,%s/../lib" % (lib_dir, lib_dir)}
> Why do we need to make this `-L$(lib_dir)/../lib` instead of the original `-L$(lib_dir)`? This breaks cases where `lib_dir` is `lib64` and not `lib`
Basically because this is hack on top of hack on top of hack...
Confusingly `lib_dir` is actually set to the `bin` dir not the correct setting for `lib`/`lib64`. Except if lldb supports the `-P` flag, and your host is FreeBSD or Linux, in which case we construct the lib path from the python path...
So this change was needed to make Darwin or NetBSD work, but broke FreeBSD and Linux in lib64 configurations... great...
We *really* need to clean up all of this because the garbled mess of platform-specific hacks is a really big problem.
More information about the lldb-commits