search

sagemath / cysignals

cysignals: interrupt and signal handling for Cython. Source repository for https://pypi.org/project/cysignals/
GNU Lesser General Public License v3.0
44 stars 23 forks source link

Try to fix AppVeyor builds #144

Closed embray closed 3 years ago

embray commented 3 years ago

Fixes #143

I'm not sure if this is the problem, but I think AppVeyor no longer supports "os:" (which was never well documented anyways).

Try OS as environment variable explicitly instead.

Also change the PYTHON environment variable to PYTHON_PATH which makes its purpose a bit clearer.

embray commented 3 years ago

This looks to be working now. cysetjmp is disabled on Cygwin via the configure check.

Interestingly, the configure check for cysetjmp does work on my local Windows machine, and the tests pass with it compiled. I wonder what the difference is. It's using the same GCC version in both cases. Some older GCCs had problems on Windows related to SIMD registers, but the GCC being used on AppVeyor shouldn't be affected by that.

embray commented 3 years ago

I think the problem with appveyor only affects the master branch, but I'll see next time I push to the 1.10.x branch.