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
ipbt: init at 20190601.d1519e0 #82177
Conversation
pkgs/tools/misc/ipbt/default.nix
Outdated
@@ -0,0 +1,22 @@ | |||
{ stdenv, fetchurl, perl, ncurses }: | |||
|
|||
let version = "20190601.d1519e0"; in stdenv.mkDerivation { |
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.
let version = "20190601.d1519e0"; in stdenv.mkDerivation { | |
stdenv.mkDerivation rec { |
pkgs/tools/misc/ipbt/default.nix
Outdated
{ stdenv, fetchurl, perl, ncurses }: | ||
|
||
let version = "20190601.d1519e0"; in stdenv.mkDerivation { | ||
inherit version; |
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.
inherit version; | |
version = "20190601.d1519e0"; |
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.
The commit messages don't actually match CONTRIBUTING.md
. Please revisit it, especially the section on commit messages.
pkgs/tools/misc/ipbt/default.nix
Outdated
|
||
meta = with stdenv.lib; { | ||
description = "A high-tech ttyrec player for Unix"; | ||
homepage = https://www.chiark.greenend.org.uk/~sgtatham/ipbt/; |
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.
Please quote the homepage (NixOS/rfcs#45)
pkgs/tools/misc/ipbt/default.nix
Outdated
meta = with stdenv.lib; { | ||
description = "A high-tech ttyrec player for Unix"; | ||
homepage = https://www.chiark.greenend.org.uk/~sgtatham/ipbt/; | ||
license = licenses.mit; |
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.
Please don't align on =
.
@timokau sorry about that -- i've made those changes and squashed everything back into one commit |
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.
Looks perfect now, thank you!
Motivation for this change
Adds a derivation for the
ipbt
program.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)