Removed unnecessary gRPC build flags in Mesos.

The `-Wno-deprecated-declarations` and `-Wno-unused-function` flags are
no longer required to build the bundled gRPC 1.10.0.

Review: https://reviews.apache.org/r/67017


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/89adbf27
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/89adbf27
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/89adbf27

Branch: refs/heads/master
Commit: 89adbf276465d9e611b00adeeaffdf8062d721d6
Parents: 6cf942a
Author: Chun-Hung Hsiao <[email protected]>
Authored: Tue May 8 15:26:12 2018 -0700
Committer: Chun-Hung Hsiao <[email protected]>
Committed: Wed May 9 12:05:13 2018 -0700

----------------------------------------------------------------------
 3rdparty/Makefile.am | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/89adbf27/3rdparty/Makefile.am
----------------------------------------------------------------------
diff --git a/3rdparty/Makefile.am b/3rdparty/Makefile.am
index 8d9fa85..6740940 100644
--- a/3rdparty/Makefile.am
+++ b/3rdparty/Makefile.am
@@ -329,18 +329,10 @@ 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
 $(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)             \

Reply via email to