Migrate from setuptools to meson-python build backend #182
Conversation
Co-authored-by: tobiasdiez <[email protected]>
Co-authored-by: tobiasdiez <[email protected]>
Co-authored-by: tobiasdiez <[email protected]>
tobiasdiez
force-pushed
the
copilot/fix-3964b91e-9f2a-4559-822d-031eb0fb710f
branch
from
8901850 to
3815f81
Compare
|
There are still failing CI runs, but those were also failing for other PRs. So this should be good to go. |
|
cc a few people that might be interested in this: @dimpase @antonio-rojas @tornaria @orlitzky |
meson.build
Outdated
| ['c', 'cython'], | ||
| version: files('VERSION'), | ||
| license: 'GPL v3', | ||
| default_options: ['c_std=c17', 'cpp_std=c++17', 'python.install_env=auto'], |
meson.build
Outdated
| project('cypari2', | ||
| ['c', 'cython'], | ||
| version: files('VERSION'), | ||
| license: 'GPL v3', |
cypari2/meson.build
Outdated
| sources: pyx, | ||
| subdir: 'cypari2', | ||
| install: true, | ||
| #include_directories: [inc_dirs, 'cypari2'], |
|
Thanks @orlitzky, you were right in all points! Should be fixed now. |
There was a problem hiding this comment.
can you say a bit in the docs about pari as the dependency. It's a bit fragile as there is no version check, and it's unclear what to do if libpari is in a nonstandard location.
It would be interesting to figure out how to build wheels which don't pack libpari in (do the current binary wheels get libpari packed in?)
| @@ -139,10 +135,22 @@ same computations be done via | |||
| The complete documentation of cypari2 is available at http://cypari2.readthedocs.io and | |||
There was a problem hiding this comment.
http -> https, please (also below)
Do you know what versions are acceptable? I have no idea ;-)
Will do. Pari really needs a pkg-config file ;-) |
This PR migrates cypari2 from the legacy setuptools build system to the modern meson-python backend. Hopefully, this fixes already some build issues on some platforms. It should also pave the way for a windows version.
Initial version was created by Github's copilot.