Upgrade gmpy2 to add python 3.10 support

[mkoeppe]

gmpy2 fails to build with Python 3.10: error: ‘_PyHASH_NAN’ undeclared - https://bugzilla.redhat.com/show_bug.cgi?id=1959010 ...

2.1.0rc1 has a fix.

Upstream: Fixed upstream, but not in a stable release.

CC: @videlec @dimpase @orlitzky @kiwifb

Component: packages: standard

Author: Matthias Koeppe

Branch: 41f7f4e

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/31855

[mkoeppe]

Branch: u/mkoeppe/patch_upgrade_gmpy2_to_add_python_3_10_support

[mkoeppe]

Commit: 8cd9ece

[mkoeppe]

Author: Matthias Koeppe

[mkoeppe]

New commits:

8cd9ece

build/pkgs/gmpy2/patches: Add patches from Fedora @python/python3.10

[videlec]

comment:3

Why not use upstream patch aleaxit/gmpy#297?

[mkoeppe]

comment:4

No idea, why doesn't upstream make a proper release at some point?

[videlec]

comment:5

Replying to @mkoeppe:

No idea, why doesn't upstream make a proper release at some point?

No idea, worth asking. Though the patch was merged a week ago.

[dimpase]

Upstream: Reported upstream. No feedback yet.

[dimpase]

comment:7

Replying to @videlec:

Replying to @mkoeppe:

No idea, why doesn't upstream make a proper release at some point?

No idea, worth asking. Though the patch was merged a week ago.

I've opened
aleaxit/gmpy#303

[dimpase]

comment:8

How about using the upstream patch https://patch-diff.githubusercontent.com/raw/aleaxit/gmpy/pull/297.diff
instead?

[mkoeppe]

comment:10

Looks like upstream is preparing a proper release now - aleaxit/gmpy@9742452

[sagetrac-git]

Changed commit from 8cd9ece to 41f7f4e

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

41f7f4e

build/pkgs/gmpy2: Update to 2.1.0rc1

[mkoeppe]

Changed upstream from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.

[mkoeppe]

Reviewer: https://github.com/mkoeppe/sage/actions/runs/1242854091, https://github.com/mkoeppe/sage/actions/runs/1242854090

[orlitzky]

comment:16

OK by me when the GH actions finish. Everyone is already using beta5, so it's not like rc1 is a step down in purported release quality.

[mkoeppe]

comment:17

Let's get this in please.

[enriqueartal]

comment:18

I am in Fedora 35 now and I would like to test it. I can use python3.10 (following the hack of the packager of sagemath in Fedora) but I do not know how to deal with cysignals. Any clue?

[mkoeppe]

comment:19

See #32576 for cysignals

[enriqueartal]

comment:20

I saw it but I am not how to deal with it. Apparently in Fedora they were able to compile cysignals and to use it as a system package.

[mkoeppe]

comment:21

You can help on #32576 by reviewing the proposed changes to cysignals

[enriqueartal]

comment:22

This is the status. I used the configure file of sagemath.spec in fedora which allows to use python3.10.0 in the system. Using the link in #32576 I used the revised cysignals, and I also used gmpy2.1.0rc1, both are compiled. The only unsuccessful package is pyzmq; the fedora system package python3-zmq exists and I was able to recompile it locally. The error in Sage is:

gcc build/temp.linux-x86_64-3.9/tmp/timer_createq5lapz40.o -o build/temp.linux-x86_64-3.9/a.out
error: [Errno 2] No such file or directory: 'a.out'

It happens with 22.2.1 and 22.0.3
I do not know how to use the system package for this.

[mkoeppe]

comment:23

I have opened #32828 for the issues with pyzmq. I think they are unrelated to python 3.10

[mkoeppe]

Changed reviewer from https://github.com/mkoeppe/sage/actions/runs/1242854091, https://github.com/mkoeppe/sage/actions/runs/1242854090 to https://github.com/mkoeppe/sage/actions/runs/1429845185, https://github.com/mkoeppe/sage/actions/runs/1429845182

[mkoeppe]

Changed reviewer from https://github.com/mkoeppe/sage/actions/runs/1429845185, https://github.com/mkoeppe/sage/actions/runs/1429845182 to https://github.com/mkoeppe/sage/actions/runs/1430376300

[mkoeppe]

comment:26

Tests look good, let's please get this update in.

[mkoeppe]

Changed reviewer from https://github.com/mkoeppe/sage/actions/runs/1430376300 to https://github.com/mkoeppe/sage/actions/runs/1430376304, https://github.com/mkoeppe/sage/actions/runs/1430376300

[dimpase]

Changed reviewer from https://github.com/mkoeppe/sage/actions/runs/1430376304, https://github.com/mkoeppe/sage/actions/runs/1430376300 to Dima Pasechnik

[dimpase]

comment:29

lgtm

[mkoeppe]

comment:30

Thanks!

[vbraun]

Changed branch from u/mkoeppe/patch_upgrade_gmpy2_to_add_python_3_10_support to 41f7f4e

[dimpase]

Changed commit from 41f7f4e to none

[dimpase]

comment:32

gmpy2 stable version 2.1 has been released.