From 2f2e72bae991138cedd0e3d06a115022736cd568 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Thu, 2 Feb 2023 10:15:42 -0800 Subject: Fix compiler flags in Linux presubmit CXX_FLAGS should be CXXFLAGS and the quoting was wrong. As a result, "-Werror -Wdeprecated" was not being applied. https://cmake.org/cmake/help/latest/envvar/CXXFLAGS.html PiperOrigin-RevId: 506656655 Change-Id: Ic5e861be3b9c32257eb9aabb845c931f3cba7122 --- ci/linux-presubmit.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ci/linux-presubmit.sh b/ci/linux-presubmit.sh index 4eb5bbe..9e15d9a 100644 --- a/ci/linux-presubmit.sh +++ b/ci/linux-presubmit.sh @@ -51,7 +51,7 @@ for cc in /usr/local/bin/gcc /opt/llvm/clang/bin/clang; do --workdir="/build" \ --rm \ --env="CC=${cc}" \ - --env="CXX_FLAGS=\"-Werror -Wdeprecated\"" \ + --env=CXXFLAGS="-Werror -Wdeprecated" \ ${LINUX_LATEST_CONTAINER} \ /bin/bash -c " cmake /src \ -- cgit v0.12