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
ac - init at 0.1.1 #31342
ac - init at 0.1.1 #31342
Conversation
src = fetchFromGitHub { | ||
owner = "vmchale"; | ||
repo = "fastcat"; | ||
rev = "5031c47631ea5a31d4eb8a584c8226a698453125"; |
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.
Can you also tag this release on github, so we can use the version tag here instead of the commit.
meta = with stdenv.lib; { | ||
homepage = https://github.com/vmchale/fastcat; | ||
license = licenses.bsd3; | ||
maintainers = []; |
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.
Do you want to maintain this package?
@@ -370,6 +370,8 @@ with pkgs; | |||
|
|||
acbuild = callPackage ../applications/misc/acbuild { }; | |||
|
|||
ac = callPackage ../tools/system/ac { }; |
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.
Two- and three-letter names like ac
at top-level should be reserved for old well-known Unix utilities. I think it's better to call it fastcat
or ats-cat
to avoid confusion. Also, it should be under ../tools/misc
, like coreutils
.
Feel free to open a new pull request if you're still interested in getting this into Nixpkgs. |
Motivation for this change
Add
ac
as a package.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)