-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
fplll: 5.2.0 -> 5.2.1, pythonPackages.fpylll: 0.3.0dev -> 0.4.0dev #40995
Conversation
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: fplll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: fplll Partial log (click to expand)
|
I needed to add a patch to fix the README encoding for python3 (fplll/fpylll#121). @GrahamcOfBorg build python2Packages.fpylll python3Packages.fpylll |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
"building of '/nix/store/xpg1zk3gwmlapnsvlmvw3p3zizs7c3ky-python2.7-Cython-0.28.1.drv' timed out after 3600 seconds" Failure seems unrelated, I think this is ready for merge. |
@@ -46,6 +46,11 @@ buildPythonPackage rec { | |||
py.test | |||
''; | |||
|
|||
patches = [ | |||
# https://github.com/fplll/fpylll/issues/121 | |||
./readme-encoding.patch |
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.
Upstream patched it
fplll/fpylll@8ce149b
They're fast :)
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 apparently the author is thinking about cutting a new release that includes that fix, so maybe its worth waiting for that (https://trac.sagemath.org/ticket/25404#comment:8).
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've updated to release 0.4.1
which includes the fix. I had to change from fetchPyPi
to fetchFromGitHub
since its not on PyPi yet, is that a problem? I think fetching straight from the source is better anyways if pypi doesn't have any other advantages.
@GrahamcOfBorg build python2Packages.fpylll python3Packages.fpylll |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: fplll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: fplll Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: fplll Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python2Packages.fpylll, python3Packages.fpylll Partial log (click to expand)
|
Failure is still unrelated. |
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 split the commit into two seperate ones and add a pythonPackages.
prefix to the appropriate one.
Its one commit for a reason since the two updates depend on each other. I've added the prefix. |
@GrahamcOfBorg build fplll |
Success on x86_64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Thanks! |
Success on aarch64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: fplll Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: fplll Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: fplll Partial log (click to expand)
|
Motivation for this change
This fixes the fpylll tests on 32 bit machines (see fplll/fpylll#112).
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)