Skip to content

Subject: Porting Envoy to C++17 #11570 #11941

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.

Sign up for GitHub

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

Merged
mattklein123 merged 37 commits into from
Jul 15, 2020
+15 −10
Merged

Subject: Porting Envoy to C++17 #11570 #11941

Show file tree
Hide file tree
Changes from 36 commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
d5644c4
Subject: porting Envoy to C++17
Jun 11, 2020
b8462a2
Subject: fixing the CI builds
Jun 12, 2020
b38c8d9
fix of a bug (loss of precision when doing implicit conversion)
Jun 15, 2020
1b35238
added compiler flag to silence incorrect MSVC warning
Jun 15, 2020
93991ef
trying to see if adding external dep in build solves linker issue
Jun 22, 2020
2707547
another attempt to fix msvc linkage problems
Jun 22, 2020
09d2ed6
trying one last time
Jun 23, 2020
07a5d44
fixing the windows ci and get rid of the warnings
Jun 26, 2020
f477318
api builds fix
Jun 26, 2020
807d4c3
trying to fix api ci
Jun 26, 2020
f7726fd
does this fix ci problem
Jun 26, 2020
713e306
get rid of the separate c++14 flag added to clang_tools.bzl
Jun 29, 2020
da4c5cc
getting rid of unnecessary c++ flag
Jun 30, 2020
c9c2799
getting rid of unnecessary c++ flag
Jun 30, 2020
6a7a254
getting ready for submission
Jul 1, 2020
16f981f
add macos flag
Jul 1, 2020
89e1872
add the flag to macos build
Jul 1, 2020
9423af5
fixing macos c++17 flag
Jul 1, 2020
5c0b94f
adding the flags in envoy_internal.bzl back because they are failing …
Jul 1, 2020
711507a
to force the rerun of CI
Jul 2, 2020
1fb2976
revert the change manually
Jul 7, 2020
3d39125
manually revert the patches
Jul 7, 2020
38b3d65
manually revert the patch changes
Jul 7, 2020
4b016bd
Merge branch 'master' of github.com:envoyproxy/envoy into PR2
Jul 8, 2020
4c69a00
silence the incorrect unused return value warning
Jul 8, 2020
49b22cb
getting rid of the duplicate flags
Jul 10, 2020
f699973
rerun the CI
Jul 10, 2020
a79eab1
rerun the CI
Jul 10, 2020
5676989
rerun the CI
Jul 10, 2020
3dcc3c9
Merge branch 'master' of github.com:envoyproxy/envoy into PR2
Jul 10, 2020
a4ff870
fix gcc compiler warning
Jul 10, 2020
b550468
previous run was failing a flaky test
Jul 11, 2020
576f5d7
more flaky tests
Jul 11, 2020
0e22d6e
a workaround on the behavorial difference between gcc and clang
Jul 13, 2020
19cb152
previous CI results in weird abrupted status
Jul 13, 2020
6c255f0
Merge branch 'master' of github.com:envoyproxy/envoy into PR2
Jul 13, 2020
e4aa7be
use the updated upstream to remove the duplicates in cel-cpp patch
Jul 14, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .bazelrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ build --enable_platform_specific_config

# Enable position independent code, this option is not supported on Windows and default on on macOS.
build:linux --copt=-fPIC
build:linux --cxxopt=-std=c++17

# We already have absl in the build, define absl=1 to tell googletest to use absl for backtrace.
build --define absl=1
Expand Down Expand Up @@ -65,6 +66,7 @@ build:clang-asan --config=asan
build:clang-asan --linkopt -fuse-ld=lld

# macOS ASAN/UBSAN
build:macos --cxxopt=-std=c++17
build:macos-asan --config=asan
# Workaround, see https://github.com/bazelbuild/bazel/issues/6932
build:macos-asan --copt -Wno-macro-redefined
Expand Down Expand Up @@ -263,6 +265,7 @@ build:windows --define manual_stamp=manual_stamp
build:windows --copt="-DCARES_STATICLIB"
build:windows --copt="-DNGHTTP2_STATICLIB"
build:windows --copt="-DCURL_STATICLIB"
build:windows --cxxopt="/std:c++17"

# Required to work around build defects on Windows MSVC cl
# Unguarded gcc pragmas in quiche are not recognized by MSVC
Expand Down
Loading