-
Notifications
You must be signed in to change notification settings - Fork 14
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
Different dependency providers #13
Comments
@kgann Can you do all of that in a separate PR? Or even multiple PRs, depending on how big it gets? |
@heyLu Yes, no problem. |
I'm starting to think we will run into issues with cyclic and/or duplicate dependencies. Consider 2 dependencies required for a project (perhaps through sub-dependencies): [heyLy/hiccup "0.1.0"]
["git://git.github.org/heyLu/hiccup.git" :type "git"] What do we do in this case? If we download both and place them in different paths and try to put both on the What about requiring the names be present? [heyLy/hiccup "0.1.0"]
[heyLy/hiccup "git://git.github.org/heyLu/hiccup.git" :type "git"]
[heyLu/hiccup "../my/local/hiccup" :type "file"] This would allow us to detect cyclic dependencies as well as duplicates and abort/ignore. This is not 100% as someone could just change the name and have the duplicate dependency fetched. Thoughts? |
FYI I ported clojure "dependency" lib to pixie, this could help making this happen: https://github.com/mpenet/dependency |
As mentioned previously (here and here, we want to be able to fetch dependencies from places other than GitHub.
I think we should support the following:
[user/repo "version"]
for GitHub and BitBucket (GitHub being the default)["git://git.papill0n.org/awesome.git" :type "git"]
and["http://hg.papill0n.org/also-awesome" :type "hg"]
to support git, hg and possibly others directly, on any server["http://files.papill0n.org/awesome-1.0.0.tar.gz" :type "tar"]
["../awesome" :type "file"]
for local dependencies, resolved relative to the project root directory@kgann has suggested using multi-methods for this, seems like a good idea to me!
If you want to work on this, please say so here!
The text was updated successfully, but these errors were encountered: