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
pythonPackages.lit: init at 0.5 #23643
Conversation
pkgs/top-level/python-packages.nix
Outdated
@@ -8242,6 +8242,25 @@ in { | |||
}; | |||
}; | |||
|
|||
lit = buildPythonApplication rec { |
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.
applications do not belong here
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.
Oh, my mistake. Will fix shortly.
pkgs/top-level/python-packages.nix
Outdated
disabled = isPy3k; | ||
|
||
src = pkgs.fetchurl { | ||
url = "https://files.pythonhosted.org/packages/5b/a0/dbed2c8dfb220eb9a5a893257223cd0ff791c0fbc34ce2f1a957fa4b6c6f/lit-0.5.0.tar.gz"; |
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 fetchPypi
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'm not sure how to make fetchPypi work here--all the variants AFAICT give a 404 :/. I'm pushing what I have to demonstrate. Not sure why this package would be different than others?
I've pushed c766960. |
@FRidh great, thank you! It's curious that it no longer 404's! .. after some investigation it appears this is because the version needed to be "0.5.0" not simply "0.5". D'oh. Thanks for taking care of this! |
Motivation for this change
'lit' is handy for running test-suites of various LLVM projects.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)