Skip to content

Enable all unit test programs for Cygwin #103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 23, 2016
Merged

Enable all unit test programs for Cygwin #103

merged 3 commits into from
Dec 23, 2016

Conversation

brarcher
Copy link
Contributor

@brarcher brarcher commented Dec 23, 2016

Cygwin on Windows 2012 (i.e. AppVeyor) does not pass the tests which raise(SIGFPE). However, the same tests pass on Windows 7 and Cygwin. Until this is resolved, the remaining tests should be run to prevent regressions. This pull request enables the remaining unit test programs while preventing the raise(SIGFPE) tests from running only on Cygwin.

@brarcher brarcher force-pushed the cygwin branch 3 times, most recently from 713ff4d to a28381a Compare December 23, 2016 03:47
The test_fpe and test_mark_point tests currently fail on Cygwin.
Instead of the test process receiving a SIGFPE it instead fails
with a SIGSEGV. The reason is not yet know. However, attempting a
simple program that installs a SIGFPE handler then does a
raise(SIGFPE) does result in the handler being called.

This tests need to be enabled again once the reason for the failures
is resolved.

#97
@brarcher brarcher changed the title Cygwin Enable unit tests for Cygwin Dec 23, 2016
@brarcher brarcher changed the title Enable unit tests for Cygwin Enable more tests for Cygwin Dec 23, 2016
@brarcher brarcher changed the title Enable more tests for Cygwin Enable all unit test programs for Cygwin Dec 23, 2016
@brarcher
Copy link
Contributor Author

Jenkins: test this please

1 similar comment
@brarcher
Copy link
Contributor Author

Jenkins: test this please

@brarcher brarcher merged commit b4ab8b3 into master Dec 23, 2016
@brarcher brarcher deleted the cygwin branch December 23, 2016 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant