Skip to content

Modularization of sagelib: Distributions sagemath-{brial,combinat,eclib,flint,gap,giac,glpk,graphs,groups,homfly,lcalc,libbraiding,libecm,linbox,modules,mpmath,ntl,pari,plot,polyhedra,schemes,singular,standard-no-symbolics,symbolics}#35095

Closed
mkoeppe wants to merge 4104 commits intosagemath:developfrom
mkoeppe:t/32432/modularization_of_sagelib__break_out_a_separate_package_sagemath_polyhedra

Conversation

@mkoeppe
Copy link
Copy Markdown
Contributor

@mkoeppe mkoeppe commented Feb 12, 2023

📚 Description

Screenshot 2023-09-09 at 11 57 02 PM

📝 Checklist

  • I have made sure that the title is self-explanatory and the description concisely explains the PR.
  • I have linked an issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

This is part of:

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 14, 2023

Codecov Report

Patch coverage: 63.80% and project coverage change: -1.22 ⚠️

Comparison is base (c00e6c2) 88.62% compared to head (ef82aaa) 87.40%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #35095      +/-   ##
===========================================
- Coverage    88.62%   87.40%   -1.22%     
===========================================
  Files         2148     2180      +32     
  Lines       398855   399144     +289     
===========================================
- Hits        353480   348878    -4602     
- Misses       45375    50266    +4891     
Impacted Files Coverage Δ
src/sage/all__sagemath_categories.py 0.00% <0.00%> (ø)
src/sage/all__sagemath_polyhedra.py 0.00% <0.00%> (ø)
src/sage/arith/all.py 100.00% <ø> (ø)
src/sage/arith/misc.py 90.49% <ø> (-0.50%) ⬇️
src/sage/calculus/test_sympy.py 0.00% <ø> (ø)
src/sage/categories/additive_magmas.py 97.77% <ø> (ø)
src/sage/categories/additive_semigroups.py 100.00% <ø> (ø)
src/sage/categories/affine_weyl_groups.py 100.00% <ø> (ø)
src/sage/categories/algebra_functor.py 80.00% <ø> (-20.00%) ⬇️
src/sage/categories/algebras.py 100.00% <ø> (ø)
... and 232 more

... and 283 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 22, 2023

I ran

./bootstrap && ./sage -sh -c '(cd pkgs/sagemath-polyhedra && SAGE_NUM_THREADS=16 tox -v -v -v)'

after full rebuild on macOS monterey, and got

            [37/54] gcc -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk -I./sage/cpython -I/private/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/pip-build-env-_on11jf5/overlay/lib/python3.11/site-packages/gmpy2 -I./sage/rings -I/private/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/pip-build-env-_on11jf5/overlay/lib/python3.11/site-packages -I/usr/local/opt/python@3.11/Frameworks/Python.framework/Versions/3.11/include/python3.11 -Ibuild/cythonized -I/Users/kwankyu/GitHub/sage-dev/pkgs/sagemath-polyhedra/.tox/.pkg/include -I/usr/local/opt/python@3.11/Frameworks/Python.framework/Versions/3.11/include/python3.11 -c build/cythonized/sage/structure/coerce.c -o build/temp.macosx-12-x86_64-cpython-311/build/cythonized/sage/structure/coerce.o -fno-strict-aliasing -DCYTHON_CLINE_IN_TRACEBACK=1
            build/cythonized/sage/structure/coerce.c:779:10: fatal error: 'gmp.h' file not found
            #include "gmp.h"
                     ^~~~~~~
            1 error generated.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 22, 2023

I've updated the instructions for testing.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

sagemath-polyhedra fails to build for me. It says gmp.h is missing.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Is this on macOS with homebrew? DId you run .homebrew-build-env?

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

Yes. My bash profile does source /Users/kwankyu/GitHub/sage-dev/.homebrew-build-env.

I manually pasted the line and tried again. But I got the same.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

I am trying again after deleting pkgs/sagemath-polyhedra/.tox directory.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

I got the same failure. Part of the log

.pkg: 89422 W install_requires> python -I -m pip install 'cysignals>=1.10.2' 'cython<1.0,>=0.29.21' 'gmpy2>=2.1.0' memory_allocator 'pkgconfig>=1.5.1' 'sage-setup~=10.0b1' 'sagemath-categories~=10.0b1' 'sagemath-environment~=10.0b1' 'setuptools>=49.6.0' 'wheel>=0.36.2' [tox/tox_env/api.py:428]
Collecting cysignals>=1.10.2
  Using cached cysignals-1.11.2-cp311-cp311-macosx_12_0_x86_64.whl
Collecting cython<1.0,>=0.29.21
  Using cached Cython-0.29.33-py2.py3-none-any.whl (987 kB)
Collecting gmpy2>=2.1.0
  Using cached gmpy2-2.1.5-cp311-cp311-macosx_10_9_x86_64.whl (806 kB)
Collecting memory_allocator
  Using cached memory_allocator-0.1.3.tar.gz (19 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting pkgconfig>=1.5.1
  Using cached pkgconfig-1.5.5-py3-none-any.whl (6.7 kB)
Collecting sage-setup~=10.0b1
  Using cached sage_setup-10.0b1-py3-none-any.whl
Collecting sagemath-categories~=10.0b1
  Using cached sagemath-categories-10.0b1.tar.gz (504 kB)
  Installing build dependencies ... error
  error: subprocess-exited-with-error
  
  × pip subprocess to install build dependencies did not run successfully.
  │ exit code: 1
  ╰─> [946 lines of output]

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Have you tried the command make wheels?

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

No. Shall I?

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Yes, that's what I would recommend as the best way of testing

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

I ran make wheels. It ended with

[sagemath_objects-10.0.beta1] * Creating venv isolated environment...
[sagemath_objects-10.0.beta1] * Installing packages in isolated environment... (cysignals >=1.10.2, cython >=0.29.21, <1.0, gmpy2 >=2.1.0, sage-setup ~= 10.0b1, sagemath-environment ~= 10.0b1, setuptools >=49.6.0, wheel >=0.36.2)
[sagemath_objects-10.0.beta1] Looking in links: file:///Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/sagemath_environment-10.0b1-py3-none-any.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/setuptools-65.6.3-py3-none-any.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/Cython-0.29.32-cp311-cp311-macosx_12_0_x86_64.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/gmpy2-2.1.2-cp311-cp311-macosx_12_0_x86_64.whl
[sagemath_objects-10.0.beta1] ERROR: Could not find a version that satisfies the requirement sage-setup~=10.0b1 (from versions: none)
[sagemath_objects-10.0.beta1] ERROR: No matching distribution found for sage-setup~=10.0b1
[sagemath_objects-10.0.beta1] 
[sagemath_objects-10.0.beta1] Traceback (most recent call last):
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/__main__.py", line 375, in main
[sagemath_objects-10.0.beta1]     built = build_call(
[sagemath_objects-10.0.beta1]             ^^^^^^^^^^^
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/__main__.py", line 235, in build_package_via_sdist
[sagemath_objects-10.0.beta1]     sdist = _build(isolation, builder, outdir, 'sdist', config_settings, skip_dependency_check)
[sagemath_objects-10.0.beta1]             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/__main__.py", line 145, in _build
[sagemath_objects-10.0.beta1]     return _build_in_isolated_env(builder, outdir, distribution, config_settings)
[sagemath_objects-10.0.beta1]            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/__main__.py", line 113, in _build_in_isolated_env
[sagemath_objects-10.0.beta1]     env.install(builder.build_system_requires)
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/env.py", line 214, in install
[sagemath_objects-10.0.beta1]     _subprocess(cmd)
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/env.py", line 79, in _subprocess
[sagemath_objects-10.0.beta1]     raise e
[sagemath_objects-10.0.beta1]   File "/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/build/env.py", line 76, in _subprocess
[sagemath_objects-10.0.beta1]     subprocess.run(cmd, check=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
[sagemath_objects-10.0.beta1]   File "/usr/local/Cellar/python@3.11/3.11.2_1/Frameworks/Python.framework/Versions/3.11/lib/python3.11/subprocess.py", line 571, in run
[sagemath_objects-10.0.beta1]     raise CalledProcessError(retcode, process.args,
[sagemath_objects-10.0.beta1] subprocess.CalledProcessError: Command '['/private/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/build-env-y415lely/bin/python', '-Im', 'pip', 'install', '--use-pep517', '--no-warn-script-location', '-r', '/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/build-reqs-t6wqxz1a.txt']' returned non-zero exit status 1.
[sagemath_objects-10.0.beta1] 
[sagemath_objects-10.0.beta1] ERROR Command '['/private/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/build-env-y415lely/bin/python', '-Im', 'pip', 'install', '--use-pep517', '--no-warn-script-location', '-r', '/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/build-reqs-t6wqxz1a.txt']' returned non-zero exit status 1.
[sagemath_objects-10.0.beta1] ********************************************************************************
[sagemath_objects-10.0.beta1] Failure building sdist and wheel
[sagemath_objects-10.0.beta1] ********************************************************************************
make[3]: *** [sagemath_objects-SAGE_VENV-no-deps] Error 1
make[2]: *** [/Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/installed/sagemath_objects-10.0.beta1] Error 2
make[1]: *** [wheels] Error 2

real	0m39.310s
user	0m29.301s
sys	0m8.784s
***************************************************************
Error building Sage.

The following package(s) may have failed to build (not necessarily
during this run of 'make wheels'):

It is safe to delete any log files and build directories, but they
contain information that is helpful for debugging build problems.
WARNING: If you now run 'make' again, the build directory of the
same version of the package will, by default, be deleted. Set the
environment variable SAGE_KEEP_BUILT_SPKGS=yes to prevent this.

make: *** [wheels] Error 1

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Could you try if ./bootstrap fixes this problem?

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

I ran ./bootstrap && make wheels. I got the same error.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Could you check the contents of venv/var/lib/sage/wheels/sage*.whl please?

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

$ ls venv/var/lib/sage/wheels/sage*.whl
venv/var/lib/sage/wheels/sage_sws2rst-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sagemath_environment-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sagenb_export-3.3-py3-none-any.whl
venv/var/lib/sage/wheels/sagetex-3.6.1-py3-none-any.whl

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

The sage_setup wheel is missing, which should have been one of the first wheels that make wheels makes.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

Part of the log:

make --no-print-directory SAGE_EDITABLE=no SAGE_WHEELS=yes sage_sws2rst sage_setup sagemath_environment sagemath_objects sagemath_repl sagemath_categories sagemath_polyhedra sage_conf sagelib sage_docbuild
make[2]: Nothing to be done for `sage_sws2rst'.
make[2]: Nothing to be done for `sage_setup'.
make --no-print-directory python_build-no-deps
sage-logger -p 'sage --pip install -r "/Users/kwankyu/GitHub/sage-dev/build/pkgs/python_build/requirements.txt"' '/Users/kwankyu/GitHub/sage-dev/logs/pkgs/python_build.log'
[python_build] Requirement already satisfied: build in /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages (from -r /Users/kwankyu/GitHub/sage-dev/build/pkgs/python_build/requirements.txt (line 1)) (0.10.0)
[python_build] Requirement already satisfied: packaging>=19.0 in /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages (from build->-r /Users/kwankyu/GitHub/sage-dev/build/pkgs/python_build/requirements.txt (line 1)) (21.3)
[python_build] Requirement already satisfied: pyproject_hooks in /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages (from build->-r /Users/kwankyu/GitHub/sage-dev/build/pkgs/python_build/requirements.txt (line 1)) (1.0.0)
[python_build] Requirement already satisfied: pyparsing!=3.0.5,>=2.0.2 in /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages (from packaging>=19.0->build->-r /Users/kwankyu/GitHub/sage-dev/build/pkgs/python_build/requirements.txt (line 1)) (3.0.9)
make --no-print-directory sagemath_objects-SAGE_VENV-no-deps
[sagemath_objects-10.0.beta1] * Creating venv isolated environment...
[sagemath_objects-10.0.beta1] * Installing packages in isolated environment... (cysignals >=1.10.2, cython >=0.29.21, <1.0, gmpy2 >=2.1.0, sage-setup ~= 10.0b1, sagemath-environment ~= 10.0b1, setuptools >=49.6.0, wheel >=0.36.2)
[sagemath_objects-10.0.beta1] Looking in links: file:///Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/wheel-0.38.4-py3-none-any.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/sagemath_environment-10.0b1-py3-none-any.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/gmpy2-2.1.2-cp311-cp311-macosx_12_0_x86_64.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/cysignals-1.11.2-cp311-cp311-macosx_12_0_x86_64.whl
[sagemath_objects-10.0.beta1] Processing /Users/kwankyu/GitHub/sage-dev/local/var/lib/sage/venv-python3.11/var/lib/sage/wheels/setuptools-65.6.3-py3-none-any.whl
[sagemath_objects-10.0.beta1] ERROR: Could not find a version that satisfies the requirement sage-setup~=10.0b1 (from versions: none)
[sagemath_objects-10.0.beta1] ERROR: No matching distribution found for sage-setup~=10.0b1
[sagemath_objects-10.0.beta1] 

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

It says Nothing to be done for sage_setup!

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Try with this change please

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

Yes. make wheel built sage_setup etc.

What changed? $(SAGE_VENV) is different from venv? venv is a symbolic link to local/var/lib/sage/venv-python3.11.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

When this Makefile is run, the current directory is build/make, and the venv points nowhere.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

Ah. Then that was a bug. I wonder why that was not found before...

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

I broke this in #35104, which is in "needs review". Fixed now also there.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

Okay.

It seems all is going well. It is now building sagemath_polyhedra wheel. Thanks.

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

After successful make wheels,

$ ls venv/var/lib/sage/wheels/sage*.whl     
venv/var/lib/sage/wheels/sage_conf-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sage_docbuild-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sage_setup-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sage_sws2rst-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sagemath_categories-10.0b1-cp311-cp311-macosx_12_0_x86_64.whl
venv/var/lib/sage/wheels/sagemath_environment-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sagemath_objects-10.0b1-cp311-cp311-macosx_12_0_x86_64.whl
venv/var/lib/sage/wheels/sagemath_polyhedra-9.6rc3-cp311-cp311-macosx_12_0_x86_64.whl
venv/var/lib/sage/wheels/sagemath_repl-10.0b1-py3-none-any.whl
venv/var/lib/sage/wheels/sagemath_standard-10.0b1-cp311-cp311-macosx_12_0_x86_64.whl
venv/var/lib/sage/wheels/sagenb_export-3.3-py3-none-any.whl
venv/var/lib/sage/wheels/sagetex-3.6.1-py3-none-any.whl

I did (because I had to) rm -r pkgs/sagemath-polyhedra/.tox, and I ran again ./bootstrap && ./sage -sh -c '(cd pkgs/sagemath-polyhedra && SAGE_NUM_THREADS=16 tox -v -v -v)'.

Unfortunately, I got

.pkg: 76432 W install_requires> python -I -m pip install 'cysignals>=1.10.2' 'cython<1.0,>=0.29.21' 'gmpy2>=2.1.0' memory_allocator 'pkgconfig>=1.5.1' 'sage-setup~=10.0b1' 'sagemath-categories~=10.0b1' 'sagemath-environment~=10.0b1' 'setuptools>=49.6.0' 'wheel>=0.36.2' [tox/tox_env/api.py:428]
Collecting cysignals>=1.10.2
  Using cached cysignals-1.11.2-cp311-cp311-macosx_12_0_x86_64.whl
Collecting cython<1.0,>=0.29.21
  Using cached Cython-0.29.33-py2.py3-none-any.whl (987 kB)
Collecting gmpy2>=2.1.0
  Using cached gmpy2-2.1.5-cp311-cp311-macosx_10_9_x86_64.whl (806 kB)
Collecting memory_allocator
  Using cached memory_allocator-0.1.3.tar.gz (19 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting pkgconfig>=1.5.1
  Using cached pkgconfig-1.5.5-py3-none-any.whl (6.7 kB)
Collecting sage-setup~=10.0b1
  Using cached sage_setup-10.0b1-py3-none-any.whl
Collecting sagemath-categories~=10.0b1
  Using cached sagemath-categories-10.0b1.tar.gz (504 kB)
  Installing build dependencies ... error
  error: subprocess-exited-with-error
  
  × pip subprocess to install build dependencies did not run successfully.
  │ exit code: 1
  ╰─> [946 lines of output]
            [ 9/54] gcc -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk -I/private/var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/pip-build-env-q62uj30s/overlay/lib/python3.11/site-packages -I/usr/local/opt/python@3.11/Frameworks/Python.framework/Versions/3.11/include/python3.11 -Ibuild/cythonized -I/Users/kwankyu/GitHub/sage-dev/pkgs/sagemath-polyhedra/.tox/.pkg/include -I/usr/local/opt/python@3.11/Frameworks/Python.framework/Versions/3.11/include/python3.11 -c build/cythonized/sage/misc/randstate.c -o build/temp.macosx-12-x86_64-cpython-311/build/cythonized/sage/misc/randstate.o -fno-strict-aliasing -DCYTHON_CLINE_IN_TRACEBACK=1
            build/cythonized/sage/misc/randstate.c:773:10: fatal error: 'gmp.h' file not found
            #include "gmp.h"
                     ^~~~~~~
            1 error generated.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Try again with this change please

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

Also, could you check which version of tox you use?

@kwankyu
Copy link
Copy Markdown
Collaborator

kwankyu commented Feb 23, 2023

(sage-sh) kwankyu@Hera:sage-dev$ tox --version
4.4.4 from /usr/local/Cellar/tox/4.4.4/libexec/lib/python3.11/site-packages/tox/__init__.py

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented Feb 23, 2023

That version should be fine (early 4.x versions had serious defects.)

Matthias Koeppe added 29 commits June 22, 2024 17:05
…ect] dependencies, [project.optional-dependencies] too
…package names, treat dashes and underscores as equivalent
…b__break_out_a_separate_package_sagemath_polyhedra
…lib__break_out_a_separate_package_sagemath_polyhedra
…zation_of_sagelib__break_out_a_separate_package_sagemath_polyhedra
@orlitzky
Copy link
Copy Markdown
Contributor

orlitzky commented Feb 6, 2026

This is available in https://github.com/passagemath/passagemath now, and the PR against sage is hopelessly outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment