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
pari: Make pthread optional, remove unstable #39377
Conversation
@@ -15,10 +17,10 @@ stdenv.mkDerivation 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.
You can probably make libpthreadstubs optional as well
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.
You're right
I don't use Pari/GP directly… intuitively maybe no, given that the Git revision is chosen based on Sage needs, not on some policy that only refers to Pari/GP development itself. |
Yeah I agree. Hopefully they'll switch back to stable releases anyways. |
I hope I don't forget to come back and merge… |
Looks good! If you're not already, make sure you are checking Hydra jobs. This has much more extensive testing than done in PRs and occasionally we will pick up an unexpected failure: |
@matthewbauer I am to a certain extend (mostly only when hydra notifies me about stuff I maintain). But as long as we have hundreds of failing jobs, I don't think its a very useful indicator of which update broke something. |
Yeah definitely. I just know some new contributors are not aware of Hydra so I wanted to mention it. I mainly check the "newly failing jobs" tab. |
Motivation for this change
pthread
optional because sage expects threading to be disabled and threading is disabled by default upstreamThings done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)