-
-
Notifications
You must be signed in to change notification settings - Fork 31
Build wheel, update CI and improve build system #188
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
Conversation
|
by the way, I have a testing-related uv (?) question. Suppose I want to test something in a uv venv using a sagemath wheel built on the same machine (with PS. Never mind, TIL about |
|
are we going with Spec0 for sage 10.8? |
|
OK. When we get closer to the release, we'll need to update docs about tests etc. |
|
Thanks for the quick review!
I would say yes, and have opened sagemath/sage#40936 to drop Python 3.11.
While it would be nice to add more developer docs, I don't have the time for this at the moment. (Also the windows support is not likely to happen very soon.) Thus, from my side at least, now would be a good moment for a release. |
|
CI tests fail. |
| input: 'cypari2.py.in', | ||
| output: 'cypari2.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.
Should .py be .pc 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.
the name of this file is wrong. pc, not py.
@tobiasdiez @collares
should be fixed
A few interconnected changes:
uvin CI.install-pariscript instead of relying on sage-the-distro to setup the build environementpari_datadirof the pari lib) and then inject it into the autogenNotes: