oss-fuzz/infra/base-images/base-clang
jonathanmetzman 7fb04e4fda
Fix clang updating script to actually track Chrome's (#3806)
Fix clang updating script to actually update clang based on Chrome's

With this change, clang (llvm-project) will be updated from:
e84b7a5fe230e42b8e6fe451369874a773bf1867
to Chrome's current version:
99ac9ce7016d701b43b8f0c308dc3463da57d983.

This commit fixes two bad bugs:
1. Using the output of a command instead of the return code.
2. Using ! to determine whether the command succeeded ! in bash
will do the same thing for 0 and 1.

These bugs were hidden by OUR_LLVM_REVISION which we traditionally
set to the last commit we rolled back to. If this were a comment
instead of stored as a variable that is not supposed to be used
unless FORCE_OUR_REVISION is set, this bug probably could have
been caught earlier. Instead, the code always used the value
provided by this variable.

Fixes https://github.com/google/oss-fuzz/issues/3805
2020-05-13 17:07:18 -07:00
..
Dockerfile [infra] Install LLVM binaries in base-runner using base-clang artifacts (#3067). (#3072) 2019-11-26 10:40:58 -08:00
checkout_build_install_llvm.sh Fix clang updating script to actually track Chrome's (#3806) 2020-05-13 17:07:18 -07:00