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
travis: sanitize=address,undefined,signed-integer-overflow #3190
Conversation
I don't understand this error, nor do I understand why it build nghttp2 again here since it is supposed to be cached... :-/ |
Mixing clang and GCC also doesn't look right, does it?
|
Exactly, that's the side-effect of nghttp2 getting built (again) with the setup that is meant only for the curl build. I tried to setup so that the nghttp2 build would remain cached and thus not having to get rebuilt so that the CFLAGS + clang settings wouldn't be used in the nghttp2 build... |
Ah, you want to reuse the build from previous jobs? Aren't the builds run in parallel on different machines? I thought any caching was only best-effort, but maybe I don't understand how Travis works well enough yet. |
right, well obviously it didn't work so I need to come up with a way that makes the sanitizer stuff only affect the curl build and not the other ones... |
|
…rflow ... using clang
c2804a1
to
7df8563
Compare
Excellent suggestion. Rebased and squashed to make it easier to review. |
... using clang Closes curl#3190
... using clang