-
Notifications
You must be signed in to change notification settings - Fork 317
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
[Config] POC: enable assertions even in release mode #4496
base: master
Are you sure you want to change the base?
Conversation
[ci-build][with-all-tests][force-full-build] |
1 similar comment
[ci-build][with-all-tests][force-full-build] |
c444e8f
to
93b73f3
Compare
[ci-build][with-all-tests] |
93b73f3
to
a258b44
Compare
[ci-build][with-all-tests] |
a258b44
to
99efa31
Compare
eb6ffbc
to
81cee92
Compare
[ci-build][with-all-tests][force-full-build] |
4cdf7fe
to
07a63e2
Compare
07a63e2
to
aa1e30a
Compare
[ci-build][with-all-tests][force-full-build] |
967fc3a
to
b7367e9
Compare
[ci-build][with-all-tests][force-full-build] |
1 similar comment
[ci-build][with-all-tests][force-full-build] |
[ci-depends-on] detected during build #21. To unlock the merge button, you must
|
b7367e9
to
4340f63
Compare
[ci-depends-on] detected during build #22. To unlock the merge button, you must
|
4340f63
to
8879008
Compare
8879008
to
342cf7a
Compare
342cf7a
to
2838366
Compare
2838366
to
df9919b
Compare
[ci-build][with-all-tests][force-full-build] |
df9919b
to
98b82fc
Compare
[ci-build][with-all-tests][force-full-build] |
98b82fc
to
06913d9
Compare
06913d9
to
5e6b7a4
Compare
5e6b7a4
to
8a32ce6
Compare
[ci-build][with-all-tests][force-full-build] |
The CI only compiles in debug for one configuration and only when a commit is done on the master.
AND a lot of failures just happen because of timeouts. So the readings are uncomfortable, to say the least
So This PR simply enable assertions in release mode (overriding the flags) and should show where the assertions happen,
By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).
Reviewers will merge this pull-request only if