Upgrade cysignals to version 1.7.1

[jdemeyer]

Unfortunately, cysignals-1.7.0 is broken on ppc64le GNU/Linux.

Tarball: https://files.pythonhosted.org/packages/81/64/0959438193254c2c6ad127fb5f9aad4140bc4288d752d03b1976ef8366e0/cysignals-1.7.1.tar.gz

Component: packages: standard

Author: Jeroen Demeyer

Branch/Commit: 4ed886b

Reviewer: Vincent Klein

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

[jdemeyer]

Branch: u/jdemeyer/upgrade_cysignals_to_version_1_7_1

[jdemeyer]

New commits:

4ed886b

Upgrade cysignals to version 1.7.1

[jdemeyer]

Commit: 4ed886b

[vbraun]

comment:6

Thats not a supported platform -> next beta cycle

[jdemeyer]

comment:7

Replying to @vbraun:

Thats not a supported platform

What's your definition of "supported platform" and why does this system not satisfy?

Previous versions of Sage worked perfectly with 100% doctests passing and I actually use this machine for serious stuff (i.e. it's not a museum machine that I only use to annoy release managers with bug reports).

PS: I know that I've been messing up with cysignals breakage in this release cycle, but there is always some strange OS quirk that comes up.

[vbraun]

comment:8

Imho supported platform requires having a buildbot

The 2 users of Sage on that arch can just follow the develop branch.

We've seen how easy it is to break something by mucking with signal handlers in this release cycle, so I'm against last minute changes if they can be avoided

[jdemeyer]

comment:9

Replying to @vbraun:

Imho supported platform requires having a buildbot

That just means that we should put a buildbot on that machine. For a while now, I have a patchbot running there.

The 2 users of Sage on that arch can just follow the develop branch.

Can't argue with that I guess.

We've seen how easy it is to break something by mucking with signal handlers in this release cycle, so I'm against last minute changes if they can be avoided

If it helps, there are no actual changes to signal handling, only to threading :-)

[vinklein]

Reviewer: Vincent Klein

[vinklein]

comment:11

All tests run for me. Let me know if you see more specific test to do.

[jdemeyer]

comment:13

@vbraun: can we at least make this a 8.3 blocker then, in the sense that it should be merged with priority in the 8.3 branch?

[vbraun]

Changed branch from u/jdemeyer/upgrade_cysignals_to_version_1_7_1 to 4ed886b