Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Headers include paths are written to "User Header Search Paths" instead of "Header Search Paths" in Xcode #1517

Closed
cos-public opened this issue Sep 11, 2020 · 3 comments · Fixed by #1540
Labels

Comments

@cos-public
Copy link
Contributor

What seems to be the problem?
includedirs property in premake project file writes include paths to USER_HEADER_SEARCH_PATHS property in project.pbxproj file instead of HEADER_SEARCH_PATHS. Which forces to use " " instead of < > in cpp files even for 3rd party libraries.

What did you expect to happen?
includedirs should create entries in HEADER_SEARCH_PATHS in .pbxproj file by default, in my opinion.

What have you tried so far?
Manually moving entries in Xcode IDE

How can we reproduce this?
Use includedirs and verify that directories are written to USER_HEADER_SEARCH_PATHS property

What version of Premake are you using?
premake5 (Premake Build Script Generator) 5.0.0-alpha15
Xcode Version 11.7 (11E801a)

@cos-public cos-public added the bug label Sep 11, 2020
@samsinsane
Copy link
Member

Have you tried specifying your 3rd party libraries using sysincludedirs instead?

@cos-public
Copy link
Contributor Author

Yes, thank you. Using sysincludedirs does work, and puts include paths into "Header Search Paths" in Xcode. But there is another entry in Xcode project - "System Header Include Paths" and it seems logical to me for sysincludedirs to put header paths there.

@noresources
Copy link
Contributor

Using sysincludedirs does work, and puts include paths into "Header Search Paths" in Xcode. But there is another entry in Xcode project - "System Header Include Paths" and it seems logical to me for sysincludedirs to put header paths there.

You are right, SYSTEM_HEADER_SEARCH_PATHS is a new setting of Xcode 9+. Premake still use the old HEADER_SEARCH_PATHS setting. See also #1456

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants