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
teip: init at 1.2.1 #94094
teip: init at 1.2.1 #94094
Conversation
@@ -3132,6 +3132,12 @@ | |||
githubId = 7385287; | |||
name = "Lana Black"; | |||
}; | |||
greymd = { |
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.
IIRC, adding yourself as a maintainer is preferred to be a separate commit.
Something like, maintainers: add greymd. Then apply the teip commit.
src = fetchFromGitHub { | ||
owner = "greymd"; | ||
repo = "teip"; | ||
rev = "52b7a8b4a6cbf354a3620936cec7add257e2f483"; |
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.
rev = "52b7a8b4a6cbf354a3620936cec7add257e2f483"; | |
rev = "v${version}"; |
|
||
features = [ "oniguruma" ]; | ||
|
||
preFixup = '' |
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.
preFixup = '' | |
postInstall = '' |
I marked this as stale due to inactivity. → More info |
homepage = "https://github.com/greymd/teip"; | ||
license = with licenses; [ mit ]; | ||
maintainers = with maintainers; [ greymd ]; | ||
platforms = platforms.x86_64; |
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 won't build on other platforms? Also, I think this should be x86_64-linux
or perhaps in your case I'd set platforms.linux
.
description = "Highly efficient \"Masking tape\" for standard input"; | ||
longDescription = '' | ||
Select partial standard input and replace with the result of another command efficiently | ||
''; |
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'd put in description
what you put in longDescription
, "Masking tape" might be a confusing description for some people.
# Disabling tests because criterion build was hanging for me | ||
# on Darwin |
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.
# Disabling tests because criterion build was hanging for me | |
# on Darwin | |
# Disabling tests because criterion build was hanging for me | |
# on Darwin |
@joefiorini please address the review comments. |
preFixup = '' | ||
installManPage "$src/man/teip.1" | ||
|
||
installShellCompletion --zsh $src/completion/zsh/_teip |
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.
bash, fish?
I marked this as stale due to inactivity. → More info |
Motivation for this change
This module is a very useful, general purpose Unix tool that I think others would like to install via nix.
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)