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

Adding .hxx to the list of recognized header file extensions #320

Merged
merged 1 commit into from
Oct 23, 2015

Conversation

jasonvasquez
Copy link
Contributor

I've run into a few cases with some third party open source libraries I'm wrapping in Cocoapods, where they use .hxx as a header extension in C++ projects. I have .podspec prepare_command's to actually rename all the files and update references to them, otherwise, we generate build rules for all the .hxx files. That is obviously not optimal. Xcode recognizes these as headers already, we just need to slot them in the right build phase.

@orta
Copy link
Member

orta commented Oct 23, 2015

seems 👍 to me

@kylef
Copy link
Contributor

kylef commented Oct 23, 2015

Thanks for the pull request @jasonvasquez. Would you be able to add a short piece to the CHANGELOG so you get credit for this change please.

@jasonvasquez
Copy link
Contributor Author

done - thanks!

@kylef
Copy link
Contributor

kylef commented Oct 23, 2015

Awesome. Until the next CocoaPod's release I'd advise you to stick to the prepare_command. Once it's out you can use it, but I would ensure you mark that your Pod would require X version of CocoaPods or newer to ensure that it isn't used on previous CocoaPod's installs.

For example:

    spec.cocoapods_version = '>= 1.0.0'

https://guides.cocoapods.org/syntax/podspec.html#cocoapods_version

kylef added a commit that referenced this pull request Oct 23, 2015
Adding .hxx to the list of recognized header file extensions
@kylef kylef merged commit 0e38ccd into CocoaPods:master Oct 23, 2015
@jasonvasquez
Copy link
Contributor Author

Good idea - thanks!

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

Successfully merging this pull request may close these issues.

3 participants