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
byrd: init at git-20200312 #110831
byrd: init at git-20200312 #110831
Conversation
Result of 1 package built:
That said, the resulting binary is mostly useless without the corresponding |
Revised. |
install -m 0755 -t $out/bin byrd | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
@@ -0,0 +1,31 @@ | |||
{ stdenv, fetchFromGitHub, vte, libconfig, glib, pkg-config }: |
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.
{ stdenv, fetchFromGitHub, vte, libconfig, glib, pkg-config }: | |
{ lib, stdenv, fetchFromGitHub, vte, libconfig, glib, pkg-config }: |
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 is new. I've always used stdenv.lib; when was this change introduced?
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.
Yeah, it is new. The change was 18 days ago c06b2b3#diff-b0739f44f25865ddf604159bdc99cc82f3c192f46b92215da2594d3e5747f0b0.
stdenv.lib is eventually removed and doing changes which introduce new occurrences of it are not allowed anymore.
I marked this as stale due to inactivity. → More info |
this was a sample PR to demonstrate the process of adding something to nixpkgs for someone. i do not think this is worth merging |
Motivation for this change
byrd is a simple terminal emulator.
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)