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
git-lfs: 1.3.1 -> 2.0.2 #25004
git-lfs: 1.3.1 -> 2.0.2 #25004
Conversation
''; | ||
|
||
maintainers = [ lib.maintainers.twey ]; |
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.
this should be in a meta section:
meta = {
maintainers = [ lib.maintainers.twey ];
};
It would be also nice to have a homepage attribute as well as license and description their.
go generate ./commands | ||
popd | ||
''; | ||
|
||
postInstall = '' | ||
rm -v $bin/bin/{man,script} | ||
rm -v $bin/bin/{man,script,genmakefile} |
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 would actually move $bin/bin/man
to share/
.
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.
Are you sure? This is an executable that (as far as I can tell) is responsible for generating the manpages.
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.
Ok. I though this was a directory.
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.
A directory would require -r
.
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)