[libc++] Remove install_name and compatibility_version on OS X
authorLouis Dionne <ldionne@apple.com>
Mon, 8 Apr 2019 21:37:42 +0000 (21:37 +0000)
committerLouis Dionne <ldionne@apple.com>
Mon, 8 Apr 2019 21:37:42 +0000 (21:37 +0000)
commitfecbf5918b818f319f97b1f60ac591c159c0f089
tree1d80435251c172b91be0950783c7d41339000347
parent867131a96cadb59f69e8f4339c52448e20ce9b8a
[libc++] Remove install_name and compatibility_version on OS X

CMake already specifies those, and we never actually want those to be
used. In fact, r357811 re-ordered those flags in a way that the
explicitly-provided install_name was overriding the CMake-provided
install_name (instead of the other way around). This caused the dylib
to be considered a system dylib, and hence the explicitly provided rpath
to be ignored. This, in turn, caused some unit tests to start linking
against the system libc++.dylib instead of the freshly-built one.
Specifically, the unit tests that started linking against the system
dylib are those that didn't specify a DYLD_LIBRARY_PATH, such as
last_write_time.sh.cpp.

llvm-svn: 357946
libcxx/lib/CMakeLists.txt