-
-
Notifications
You must be signed in to change notification settings - Fork 4
Update Python version requirements to >= 3.12, include Python 3.13 and 3.14 in CI and wheel building, fix CI
#35
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
d4a082f to
c99b4d9
Compare
|
@dimpase wheel building on macos still fails due to an issue with boost (coming from upstream sage, which is used to install a couple of dependencies): It is however installed using homebrew. Any idea how to fix this? |
>= 3.12, include Python 3.13 and 3.14 in CI and wheel building>= 3.12, include Python 3.13 and 3.14 in CI and wheel building, fix CI
…and 3.14 in CI and wheel building
c99b4d9 to
40b3f9f
Compare
|
What " coming from upstream sage" does mean? EDIT - oops, I see what you mean. |
|
let me try this locally |
|
Same issue also on Linux Alpine (boost not found, but installed). |
|
Is the boost check really necessary in the configure script of sage? If boost is not installed, then meson will just disable the one module that actually relies on it. So I would say it would be fine to just print a warning message, and not exit forcefully with an error. |
|
I tried building/testing ( |
|
Why do you even need upstream Sage here? |
|
This package does not depend upon Sage, there is absolutely no need to involve Sage in anything related to building its wheel etc. It's just like |
|
On November 15, 2025 11:47:22 AM EST, Tobias Diez ***@***.***> wrote:
tobiasdiez left a comment (sagemath/pplpy#35)
Is the boost check really necessary in the configure script of sage?
If boost is not installed, then meson will just disable the one module that actually relies on it. So I would say it would be fine to just print a warning message, and not exit forcefully with an error.
There are more spkgs in sage-distro which need boost, so it's better be there.
|
Thanks for testing. I have no idea what then really triggers this issue. Strangely, it also works on macos-15-intel.
Sage-the-distro is used to build gmp, mpfr and mpc, which are needed for building the package/wheel. I would say this PR is then good to go; will try to remove the dependency on sage-the-distro in #34 (using meson's subprojects). |
we don't really test Sage on Alpine. I gather it might be fixed by the following Sage patch (I'll do a PR to add it) --- a/build/pkgs/_prereq/distros/alpine.txt
+++ b/build/pkgs/_prereq/distros/alpine.txt
@@ -23,3 +23,4 @@ bzip2
pkgconf
zlib-dev
boost
+boost-dev |
|
On macOS, the problems must be due to the need for would suffice - but I don't know how to incorporate this. would be better, as it would take care of all the package paths/configs. |
|
I'd try re-factoring the |
|
OK, please use notes I made here in #34 |
Following https://scientific-python.org/specs/spec-0000/.
Fixes the test CI and wheel building (which are currently broken see eg #34). Also remove a few old/outdated files.