Skip to content
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

[CI] ADD log for crashes in testing summary #195

Merged

Conversation

guparan
Copy link
Contributor

@guparan guparan commented Mar 7, 2017

This should avoid us to scroll through the entire build output searching for a crashed unit test (often due to a segfault).
It is just re-printing the output that was already printed. Some dump is still needed to debug the crash (see PR #191).

This PR fixes issue #149.


This PR:

  • builds with SUCCESS for all platforms on the CI.
  • does not generate new warnings nor unit test failures.
  • does not break existing scenes.
  • does not break API compatibility.
  • has been reviewed
  • is more than 1 week old.

Reviewers will merge only if all these checks are true.

@guparan guparan added location: test pr: fast merge Minor change that can be merged without waiting for the 7 review days and removed location: test labels Mar 8, 2017
@matthieu-nesme matthieu-nesme merged commit 44d8e9f into sofa-framework:master Mar 8, 2017
@guparan guparan deleted the unit_tests_crash_output branch March 24, 2017 17:00
@guparan guparan added this to the v17.06 milestone Jun 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: fast merge Minor change that can be merged without waiting for the 7 review days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants