This is an automated email from the ASF dual-hosted git repository. chhsiao pushed a commit to branch 1.5.x in repository https://gitbox.apache.org/repos/asf/mesos.git
commit 0de1fed16d01d30d01b057f5a50f616bf5553506 Author: Benno Evers <[email protected]> AuthorDate: Wed Oct 17 22:46:54 2018 +0200 Disabled warnings-as-errors for gRPC build. --- 3rdparty/Makefile.am | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/3rdparty/Makefile.am b/3rdparty/Makefile.am index f4cb731..2b68134 100644 --- a/3rdparty/Makefile.am +++ b/3rdparty/Makefile.am @@ -313,24 +313,19 @@ LIB_GRPC = $(GRPC)/libs/opt/libgrpc++.a \ $(LIB_GRPC): $(GRPC)-build-stamp -# NOTE: The `-Wno-deprecated-declarations` flag is added so that gRPC -# can be built with OpenSSL 1.1.0. See: -# https://github.com/grpc/grpc/pull/11496 -# NOTE: The `-Wno-unused-function` flag is added to address errors when -# compling `src/core/tsi/ssl_transport_security.cc`. See: -# https://github.com/grpc/grpc/pull/11675 +# We use EXTRA_CPPFLAGS to pass `-Wno-error`, because that one needs to be +# appended and not prepended to the constructed flags. $(GRPC)-build-stamp: $(GRPC)-stamp \ $(PROTOBUF)-build-stamp cd $(GRPC) && \ - CPPFLAGS="-Wno-deprecated-declarations \ - -Wno-unused-function \ - $(PROTOBUF_INCLUDE_FLAGS) \ + CPPFLAGS="$(PROTOBUF_INCLUDE_FLAGS) \ $(SSL_INCLUDE_FLAGS) \ $(ZLIB_INCLUDE_FLAGS)" \ LDFLAGS="$(PROTOBUF_LINKER_FLAGS) \ $(SSL_LINKER_FLAGS) \ $(ZLIB_LINKER_FLAGS)" \ $(MAKE) $(AM_MAKEFLAGS) \ + EXTRA_CPPFLAGS=-Wno-error \ HAS_PKG_CONFIG=false \ NO_PROTOC=false \ PROTOC=$(PROTOC)
