Skip to content
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

Require Python 3.9 #154

Merged
merged 3 commits into from
Mar 17, 2024
Merged

Require Python 3.9 #154

merged 3 commits into from
Mar 17, 2024

Conversation

saraedum
Copy link
Member

@saraedum saraedum commented Feb 1, 2024

build with Python 3.8 seems to fail with

In file included from /home/conda/feedstock_root/build_artifacts/cypari2_1705557903378/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/include/python3.8/pytime.h:6,
                   from /home/conda/feedstock_root/build_artifacts/cypari2_1705557903378/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/include/python3.8/Python.h:85,
                   from cypari2/convert.c:50:
  cypari2/pycore_long.h: In function '_PyLong_SetSignAndDigitCount':
  cypari2/pycore_long.h:92:13: error: 'o' undeclared (first use in this function); did you mean 'op'?
     92 |     Py_SIZE(o) = size;
        |

also our CI only tests with Python 3.9+ so that seem to be required by now.

In any case, active support for Python 3.8 ended May 2021 and security support is going to end in October 2024.

@tornaria
Copy link
Contributor

tornaria commented Feb 1, 2024

This was fixed in #146.

README.rst Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
@saraedum saraedum closed this Feb 1, 2024
@mkoeppe mkoeppe reopened this Mar 16, 2024
@mkoeppe
Copy link

mkoeppe commented Mar 16, 2024

Indeed 3.8 is still broken. Let's remove it?

build with Python 3.8 seems to fail with

```
In file included from /home/conda/feedstock_root/build_artifacts/cypari2_1705557903378/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/include/python3.8/pytime.h:6,
                   from /home/conda/feedstock_root/build_artifacts/cypari2_1705557903378/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/include/python3.8/Python.h:85,
                   from cypari2/convert.c:50:
  cypari2/pycore_long.h: In function '_PyLong_SetSignAndDigitCount':
  cypari2/pycore_long.h:92:13: error: 'o' undeclared (first use in this function); did you mean 'op'?
     92 |     Py_SIZE(o) = size;
        |
```

also our CI only tests with Python 3.9+ so that seem to be required by
now.

In any case, active support for Python 3.8 ended May 2021 and security
support is going to end in October 2024.
@mkoeppe mkoeppe merged commit b1aed0b into sagemath:master Mar 17, 2024
109 of 200 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants