Don't import cysignals.alarm on Windows - Githubissues

sagemath / sage

Main repository of SageMath

https://www.sagemath.org

Other

1.46k stars 482 forks source link

Don't import cysignals.alarm on Windows #39010

Open tobiasdiez opened 2 days ago

tobiasdiez commented 2 days ago

cysignals.alarm is not available on Windows, so we don't try to import it. Probably more work needs to be done to actually fix the methods that make use of these methods on Windows; but that's left for further PRs.

Also remove a cysignals import from an all_xyz.py file that was duplicated in the main all.py file.

:memo: Checklist

[ ] The title is concise and informative.
[ ] The description explains in detail what this PR is about.
[ ] I have linked a relevant issue or discussion.
[ ] I have created tests covering the changes.
[ ] I have updated the documentation and checked the documentation preview.

:hourglass: Dependencies

github-actions[bot] commented 2 days ago

Documentation preview for this PR (built with commit ae6b1eebfc713a0ba81150a08f2c38d4b86b964a; changes) is ready! :tada: This preview will update shortly after each push to this PR.