-
-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
python38Packages.logica: init at 1.3.13 #121469
Conversation
Result of 2 packages built successfully:
1 suggestion:
Result of 2 packages built successfully:
1 suggestion:
|
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 think this is better suited for all-packages.nix
and buildPythonApplication
.
@@ -29325,6 +29325,8 @@ in | |||
|
|||
leo3-bin = callPackage ../applications/science/logic/leo3/binary.nix {}; | |||
|
|||
logica = python3.pkgs.callPackage ../applications/science/logic/logica {}; |
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.
logica = python3.pkgs.callPackage ../applications/science/logic/logica {}; | |
logica = callPackage ../applications/science/logic/logica { }; |
@@ -0,0 +1,33 @@ | |||
{ lib, buildPythonApplication, python, fetchPypi, isPy27 }: |
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.
{ lib, buildPythonApplication, python, fetchPypi, isPy27 }: | |
{ lib, python3 }: |
# PyPI tarball doesn't ship all the tests files, and upstream repository does | ||
# not have a setup.py |
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.
They must somehow create that setup.py
. So we should be able to use fetchFromGitHub
and do create it in preBuild
.
Motivation for this change
Add https://logica.dev/
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)