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
xxh: init at 0.8.8 #147189
xxh: init at 0.8.8 #147189
Conversation
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.
Formatting needs more space in my opinion.
Co-authored-by: legendofmiracles <30902201+legendofmiracles@users.noreply.github.com>
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
Co-authored-by: legendofmiracles <30902201+legendofmiracles@users.noreply.github.com>
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
My shot at helping in review (since I find xxh useful and would love to have it on nixos). I've built and tested this PR with:
It works correctly on my machine. Reviewed points
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
In the past days 0.8.8 has been released, I also wrote a patch to update it here + some smaller other changes |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Motivation for this change
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes