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
rgxg: Init at 0.1.2 #99302
rgxg: Init at 0.1.2 #99302
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.
Also, your PR should have eventually two commits:
maintainers: add <me>
<package-name>: init at <version>
doInstallCheck = true; | ||
installCheckTarget = "check"; |
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 wrong with regular doCheck
and checkTarget
?
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.
nothing,
i don't understand everything yet, this was copied from an other package, and i was just happy that it worked
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.
I see you are new :) Welcome aboard. I don't know where you borrowed that from, but it's considered more common to use doCheck = true
. check
is the default check (make) target for doCheck
, so you should spare yourself 1 line of from the expression. For more info, refer to https://nixos.org/nixpkgs/manual/#chap-stdenv .
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.
I don't know how to edit my commits here, so i created a new pr with 2 commits:
#99422
Thanks for the weclome
Co-authored-by: Doron Behar <doron.behar@gmail.com>
Co-authored-by: Doron Behar <doron.behar@gmail.com>
Thank you for your time and feedback, |
i created a new one: #99422 |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)