-
Notifications
You must be signed in to change notification settings - Fork 132
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 commit hashes in Package Set #200
Comments
Hi @shmish111! Yeah a current limitation is that you cannot reference to commits in the package set, but only branches or tags. This is documented in the section about adding dependencies:
It mentions that commit hashes support is happening at some point, and in fact it's coming with #188. In the meanwhile you can fork the upstream and make a branch pointing at the commit you want. E.g. assuming you pulled your fork locally: $ git checkout 1fcf4f85f8f6c0bb83f4ef9b6de2adad9af82ad3
$ git checkout -b my-temp-branch1
$ git push --set-upstream origin my-temp-branch1 And then in the ace =
mkPackage
[ "effect"
, "web-html"
, "web-uievents"
, "arrays"
, "foreign"
, "nullable"
, "prelude"
]
"https://github.com/shmish111/purescript-ace.git"
"my-temp-branch1" |
Update: sorry I missed the fact that you cannot point at branches. If you fork the upstream then you can also make tags. Does psc-package2nix support tags? If this doesn't work I don't have any more workarounds to suggest, so I'd say you could raise this issue upstream to pp2n |
@shmish111 update: looking at the source for pp2n it looks like it should be able to work with branch names? E.g. see Are you using the latest of pp2n? |
thanks @f-f I haven't tried tags yet as I've finished work for the day, it's my last hope, otherwise I don't know what I can do. |
phew, tags work. So it's not the end of the world but currently if you want to use spago and nix you have to tag everything. This initially makes sense as releases are tagged and that's how bower works normally however one of the main reasons you would want to add something to packages.dhall is if you want to use something that hasn't been released. In this case you really want to point to a sha I think. Looking forward to #188 and thanks for your help. I think I should leave this open as it won't be resolved until #188 is merged. |
You're welcome! 🙂 And yes, I'll pivot this issue towards tracking support for commit hashes, I'll rename accordingly and leave open |
@shmish111 update: I just cut a new release that includes support for commit hashes, could you try it out? |
gives me
If I use
"master"
then it works, however I am using this all within a nix project and pp2n creates derivations that cannot use branch names. Not sure what I'm supposed to do 👎The text was updated successfully, but these errors were encountered: