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

First test simple cysignals installation before running the full sage install #152

Closed kliem closed 2 years ago

kliem commented 2 years ago

As travis is not working currently, it makes sense to have some checks working.

embray commented 2 years ago

Yes, I thought there was already an issue about this. I thought I got the switch to travis-ci.com working in #129, but it seems to have disappeared.

embray commented 2 years ago

It appears we ran out of free OSS credits. They are supposed to be replenished today, but I don't know when...

I have a fear that GitHub is going to pull the same switch as well with Actions--it's only a matter of time, once they have enough lock-in... But in the meantime it can't hurt to have.

kliem commented 2 years ago

Yes, #139 is another pull requests that adds github actions. It don't care very much. Whatever works at the moment should be good. I agree that github will probably charge at some point for the Actions or put a limit on in.

kliem commented 2 years ago

@mkoeppe

I used this pull request to modify your test workflow. What do you think?

Currently, the tests take very long, even if there is an obvious failure.

https://github.com/sagemath/cysignals/actions/runs/1491849522

Currently, on mac the test suite fails. So what is the point on trying many different builds.

So we first try a simple build of cysignals on cygwin, ubuntu and mac-os. If the test suites pass, we start the dependent jobs and test whether it works within sage.

mkoeppe commented 2 years ago

Yes, this is a good idea, LGTM.