Skip to content

Commit

Permalink
Update release notes due to valgrind being unable to launch the test …
Browse files Browse the repository at this point in the history
…suite and scan-build producing too many false positives.
  • Loading branch information
P-p-H-d committed Nov 16, 2018
1 parent a8b5c3f commit 5d5c238
Showing 1 changed file with 4 additions and 15 deletions.
19 changes: 4 additions & 15 deletions doc/DEV.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,28 +2,17 @@ How to make a release?
======================

1) Check that all test pass on all primary systems.
Run 'make check' on all primary systems.
Run 'make check' or 'make checkall' on all primary systems.
No error shall be reported.

2) Check that no memory leak is reported:
Run 'make sanitize' in tests folder on a system.
NOTE: valgrind seems completely lost in all the tests which spawn a lot of threads.

3) Check that no undefined behavior, nor data race, ... is reported by sanitizer:
2) Check that no undefined behavior, nor data race, ... is reported by sanitizer:
Run 'make sanitize' in tests folder on a system.
This needs at least GCC 7

4) Check that no new warning is reported by static analyzer:
Run 'make scan-build' in tests folder on a system.
This needs at least clang 4.
Currently there is a false positive detected in m-shared:
"Argument to free() is offset by 8 bytes from the start of memory allocated by malloc()"
scan-build is not able to properly track that this case cannot happen.

5) Check that coverage is reasonable.
3) Check that coverage is reasonable.
Run 'make coverage' and analyze result.

6) Publish the release
4) Publish the release

In case of problem, open a problem report.

Expand Down

0 comments on commit 5d5c238

Please sign in to comment.