Skip to content

Enable all unit test programs for Cygwin#103

Merged
brarcher merged 3 commits into
masterfrom
cygwin
Dec 23, 2016
Merged

Enable all unit test programs for Cygwin#103
brarcher merged 3 commits into
masterfrom
cygwin

Conversation

@brarcher

@brarcher brarcher commented Dec 23, 2016

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

Jenkins: test this please

1 similar comment
@brarcher

Copy link
Copy Markdown
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