-
Notifications
You must be signed in to change notification settings - Fork 823
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
System dependencies #11
Comments
Is this needed? |
Would be useful for something like Google Maps where you have to specify a whole list of system SDK's. Have a look at Google Maps Install Manually |
This should be technically possible from your build settings @sbarow. What about adding an extra parameter in the target specification where you specify those frameworks, and generates the BUILD_SETTINGS necessary for doing the linking automatically? |
Hey guys, Is this still not possible? How do you exactly achieve adding SDK dependencies? |
@PeymanKh - You can achieve this by passing configuration settings for your target i.e.
Use |
@rahul-malik Thanks, that's great. |
Thanks @rahul-malik. |
I think they are fairly uncommon but there are cases where you can specify additional sdk paths in xcconfig |
In general, it’d be nice to have support for this out of the box. Maybe |
I think this is an interesting use case for a general design principle here. Should the spec make it easier to express things that are configurable in xcconfig? |
Hey all! I’d like to resurrect this topic, as I prefer to explicitly declare all dependencies but also want a single place for it that other engineers can edit. Without them having to know xcconfig syntax and linker flags 😃 I think adding another type of key to the regular Something like: targets:
Foo:
dependencies:
- system: UserNotification
optional: true # defaults to false That should result in:
What do you think @yonaskolb @rahul-malik Update: The support for optional (uses |
Hi @rastersize. Sorry for the late reply. I like that solution, though I would opt for the name |
I've opened an initial PR here #430 I decided to go with I'm still not sure why this is required, because if you're using something like the |
@yonaskolb |
I've finalized on targets:
MyApp
dependencies:
- sdk: Contacts.framework
- sdk: libc++.tbd
- sdk: some/random/path/in/sdk/root/file.file |
This would allow to linking of system SDK frameworks
The text was updated successfully, but these errors were encountered: