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

Support multiple/various application targets #64

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

l3ender
Copy link

@l3ender l3ender commented Jun 12, 2019

This PR resolves #63.

In the case of something like a Today Extension, an Xcode project can have multiple targets, each of which will have a Debug/Release build stage. These changes support this scenario by getting/setting the properties only for the appropriate build stage.

One also item that needs to be supported with some targets is that LD_RUNPATH_SEARCH_PATHS can potentially already be populated, and additional effort in needed in order to preserve the existing values if updating.

@akofman Please take a look and let me know if you have questions/thoughts. Thanks!

@l3ender
Copy link
Author

l3ender commented Jun 15, 2019

I merged in a very closely related change, where some targets don't need a bridging header. More detail at l3ender#1.

Let me know if you have any questions/concerns/feedback. Thanks!

@l3ender l3ender mentioned this pull request Jun 17, 2019
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.

Unexpected behavior when project has multiple targets (e.g. today extension)
1 participant