Skip to content

Commit

Permalink
mobile: Force mobile-release-common build to use C++17
Browse files Browse the repository at this point in the history
It was defaulting to C++20 due to
envoyproxy#32585, but for mobile, we are
still on C++17. This change enables local builds to build with C++17
without encountering toolchain errors.

Signed-off-by: Ali Beyad <abeyad@google.com>
  • Loading branch information
abeyad committed Mar 21, 2024
1 parent 58b801b commit 2b312c3
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions mobile/.bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,9 @@ build:mobile-remote-release-clang-android-publish --linkopt=-fuse-ld=lld
build:mobile-remote-release-clang-android-publish-xds --config=mobile-remote-release-clang-android-publish
build:mobile-remote-release-clang-android-publish-xds --define=envoy_mobile_xds=enabled

# Temporary revert to C++17 for mobile NDK builds.
build:mobile-release-common --cxxopt=-std=c++17 --host_cxxopt=-std=c++17

# Compile releases optimizing for size (eg -Os, etc).
build:mobile-release-common --config=sizeopt

Expand Down

0 comments on commit 2b312c3

Please sign in to comment.