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

External Cartfile specifications for dependencies #36

Closed
jspahrsummers opened this issue Oct 25, 2014 · 5 comments
Closed

External Cartfile specifications for dependencies #36

jspahrsummers opened this issue Oct 25, 2014 · 5 comments

Comments

@jspahrsummers
Copy link
Member

It'd be nice to inject a custom Cartfile for use by a dependency of your project—in other words, specify external Cartfiles for dependencies listed in your top-level Cartfile.

I can foresee a few uses for this:

  1. Updating a project's dependencies before the maintainers are able to do so themselves, without needing to fork the project
  2. Relaxing a project's dependency requirements, if they've been specified too strictly
  3. Using Carthage on projects that have no awareness of it, or no desire to use it

That last point might make this a candidate for our initial release, since few—if any—projects will have Cartfiles when this repository is made public.

@Carthage/owners Any thoughts on whether this should be a prerequisite to shipping?

@jspahrsummers
Copy link
Member Author

Tentatively dropping this into the milestone, mostly just so I don't overlook it later. Let me know if you think it should be pulled out.

@jspahrsummers jspahrsummers added this to the Initial Release milestone Oct 25, 2014
@alanjrogers
Copy link
Contributor

@jspahrsummers Great idea, I think this is important for an initial ship so that people can start using it without having to wait for library maintainers to update.

@robrix
Copy link
Contributor

robrix commented Oct 30, 2014

Agreed.

@jspahrsummers jspahrsummers self-assigned this Nov 7, 2014
@jspahrsummers
Copy link
Member Author

I've kinda 180ºd on this. I can't think of a good syntax off of the top of my head, and this introduces some additional complexity around locating those external references.

Since there is a workaround available (forking), I'm interested in punting this in the interest of minimizing scope. Please speak up if you think I'm crazy. 😉

@jspahrsummers jspahrsummers removed their assignment Nov 7, 2014
@jspahrsummers jspahrsummers removed this from the Initial Release milestone Nov 7, 2014
@jspahrsummers
Copy link
Member Author

This hasn't really been a problem in practice—probably partly thanks to Carthage's interoperability with other dependency management solutions (CocoaPods, submodules, subtrees).

After all, this only applies to dependencies that themselves have dependencies, which is a fairly small group.

natanrolnik pushed a commit to natanrolnik/Carthage that referenced this issue Feb 2, 2017
…ions

Reflect Result's parameterized error to documentations
hujunfeng pushed a commit to hujunfeng/Carthage that referenced this issue Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants