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
chez-matchable: init at 1.0 #73419
chez-matchable: init at 1.0 #73419
Conversation
pname = "chez-matchable"; | ||
version = "1.0"; | ||
|
||
src = fetchgit { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe use fetchFromGitHub
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the difference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetchFromGitHub fetches the tarball from github, which makes downloads faster
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay, done! (assuming it looks ok now?)
|
||
src = fetchgit { | ||
url = "https://github.com/fedeinthemix/chez-matchable.git"; | ||
rev = "73e46432ae70ec72eba6ef116bd84ad9ee38b2f2"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, please use the version as the rev
- the Git tag.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done! Otherwise it's ok?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
builds with nix-review, LGTM
EDIT: wait, the version is not 1.0 anymore, can you change the commit message?
EDIT EDIT: ah nvm, I can change it when squasing
Ach, sorry for missing that. Thanks! |
Motivation for this change
Adding a package definition for the hygenic pattern matching library developed by Alex Shinn.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @