This is an archive of the discontinued LLVM Phabricator instance.

Fix building tests without system headers on Darwin
ClosedPublic

Authored by beanz on Oct 11 2016, 2:08 PM.

Details

Summary

Default installations of OS X do not have system headers installed at /usr/include. This patch allows the LLDB test executables to properly compile when built on a system without headers at /usr/include by specifying a default value for the apple-sdk flag as "macosx".

Diff Detail

Repository
rL LLVM

Event Timeline

beanz updated this revision to Diff 74293.Oct 11 2016, 2:08 PM
beanz retitled this revision from to Fix building tests without system headers on Darwin.
beanz updated this object.
beanz added reviewers: tfiala, zturner.
beanz added a subscriber: lldb-commits.
zturner accepted this revision.Oct 11 2016, 3:55 PM
zturner edited edge metadata.

If you wanted to get really fancy, you could actually locate the system include directory and set the default to that. But that's up to you, just an idea.

This revision is now accepted and ready to land.Oct 11 2016, 3:55 PM
This revision was automatically updated to reflect the committed changes.
tfiala edited edge metadata.Oct 13 2016, 9:22 AM

(Retro) yep, this looked fine.