This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Do not set CMAKE_BUILD_WITH_INSTALL_NAME_DIR
ClosedPublic

Authored by ldionne on Jul 6 2021, 9:36 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG6e43f3fc141f: [libc++] Do not set CMAKE_BUILD_WITH_INSTALL_NAME_DIR
Summary

I'm not sure what that gains us, and it creates a problem when
trying to run the tests against libc++ with a custom install name
dir (e.g. /usr/lib), since the library that we link against (in
the build tree) will advertise itself as /usr/lib/libc++.dylib,
so we end up linking against the system dylib at runtime.

Diff Detail

Event Timeline

ldionne created this revision.Jul 6 2021, 9:36 AM
ldionne requested review of this revision.Jul 6 2021, 9:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2021, 9:36 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Jul 6 2021, 11:46 AM
This revision is now accepted and ready to land.Jul 6 2021, 11:46 AM
This revision was automatically updated to reflect the committed changes.