-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Valgrind documentation/testsuite improvements #10216
Conversation
db1382f
to
f8976b6
Compare
I just disabled another |
At least one |
OK yes, it looks like |
What is the error? |
|
Like some of the |
Yeah, the UnixDomainCM example test does launch a new julia session - maybe the underlying issue is the same as the spawn tests. Does `/home/garrison/julia/usr/bin/../share/doc/julia/examples/clustermanager/simple/test_simple.jl`` exist on your system? https://github.com/JuliaLang/julia/blob/master/Makefile#L40 copies the required files for the test to work. |
|
f8976b6
to
f0da4b5
Compare
I disabled (when running under valgrind) the one remaining I still occasionally get failure of some other test when I run the full suite under valgrind. The most recent one to fail was:
but I've only seen this failure once, and it was while running the full suite. Since it's intermittent, perhaps fixing the existing claimed memory errors in the tests (#10217 in particular) will fix this. I did just manage to get the full test suite to pass using the current state of this pull request, which has only a few tests disabled under valgrind. |
c254e02
to
4579966
Compare
Valgrind does not support different rounding modes; see https://bugs.kde.org/show_bug.cgi?id=136779
4579966
to
6959aae
Compare
Includes information on various flags to pass to valgrind, and how to run the test suite under valgrind
This test consistently fails under valgrind for reasons I do not yet understand.
OK, I rebased everything due to a conflict on |
This was not correctly done in the rebase of 2732199.
This looks really great to me. @tkelman, if the AppVeyor failure is unrelated, shall we merge? |
Valgrind documentation/testsuite improvements
yep |
This pull request (meant to supersede #10015) greatly improves the documentation for running
julia
undervalgrind
, and also disables a few tests that will fail for obvious reasons undervalgrind
.