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
almonds: init at 2015-12-27 #34169
almonds: init at 2015-12-27 #34169
Conversation
80e5226
to
e17a045
Compare
The package doesn't have a license (https://github.com/Tenchi2xh/Almonds), so i didn't set it, not sure what's the usual way to handle this |
pkgs/top-level/all-packages.nix
Outdated
@@ -18981,6 +18981,8 @@ with pkgs; | |||
|
|||
### SCIENCE/MATH | |||
|
|||
almonds = callPackage ../applications/science/math/almonds { }; |
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.
Better use pythonPackages.callPackage
. This way you won't need pythonPackages
as an argument.
{ pythonPackages, fetchFromGitHub, ncurses }: | ||
|
||
pythonPackages.buildPythonPackage { | ||
name = "almonds-2015-12-27"; |
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.
What's wrong with the last stable release?
Also please split this up into pname
and version
and leave out name
.
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.
Actually, I think the latter would only make sense when using buildPythonPackage
. Sorry :)
description = "Terminal Mandelbrot fractal viewer"; | ||
homepage = https://github.com/Tenchi2xh/Almonds; | ||
maintainers = with maintainers; [ infinisil ]; | ||
platforms = platforms.linux; |
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.
Any reason for this?
It should build on Darwin as well. Since buildPythonApplication
automatically sets the platform, you can just leave out platforms
.
@@ -0,0 +1,28 @@ | |||
{ pythonPackages, fetchFromGitHub, ncurses }: | |||
|
|||
pythonPackages.buildPythonPackage { |
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.
Use buildPythonApplication
since it's not a python module.
If there's no license, I'd say we don't have any freedom with regards to using the code. So it is unfree. |
e17a045
to
e2b5c9b
Compare
Hmm, I'm not so sure how I feel about the unfree license, since this hides it from some listings (such as https://nixos.org/nixos/packages.html) and it being unfree was most likely not the intention of the author, used it for now anyways though. I fixed all the stuff you mentioned @dotlambda |
}; | ||
|
||
buildInputs = [ | ||
pytest |
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 move this into nativeBuildInputs
and add checkPhase = "py.test";
.
meta = with stdenv.lib; { | ||
description = "Terminal Mandelbrot fractal viewer"; | ||
homepage = https://github.com/Tenchi2xh/Almonds; | ||
license = licenses.unfree; |
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 comment about why it is unfree would be nice.
e2b5c9b
to
8e8a259
Compare
Done that |
@GrahamcOfBorg build almonds |
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.
Failure for system: x86_64-linux
Package ‘almonds-1.25b’ in /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/applications/science/math/almonds/default.nix:23 has an unfree license (‘unfree’), refusing to evaluate.
a) For `nixos-rebuild` you can set
{ nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.
b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
{ allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.
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.
Failure for system: aarch64-linux
Package ‘almonds-1.25b’ in /var/lib/gc-of-borg/nix-test-rs-3/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-3/pkgs/applications/science/math/almonds/default.nix:23 has an unfree license (‘unfree’), refusing to evaluate.
a) For `nixos-rebuild` you can set
{ nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.
b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
{ allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.
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.
Failure for system: x86_64-darwin
Package ‘almonds-1.25b’ in /tmp/nix-ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/applications/science/math/almonds/default.nix:23 has an unfree license (‘unfree’), refusing to evaluate.
a) For `nixos-rebuild` you can set
{ nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.
b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
{ allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.
Oh, i should've changed the commit message |
Motivation for this change
Almonds is a pretty cool terminal mandelbrot set explorer!
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)