You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ch...@apache.org on 2019/01/07 23:11:09 UTC

[mesos] 02/04: Removed the unnecessary `-Wno-array-bounds` flag for gRPC build.

This is an automated email from the ASF dual-hosted git repository.

chhsiao pushed a commit to branch 1.7.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit c4bdd4b949e2554a351ac66fcdeba7b622853d4f
Author: Chun-Hung Hsiao <ch...@mesosphere.io>
AuthorDate: Wed Dec 19 14:25:55 2018 -0800

    Removed the unnecessary `-Wno-array-bounds` flag for gRPC build.
---
 3rdparty/Makefile.am | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/3rdparty/Makefile.am b/3rdparty/Makefile.am
index b0cfbc9..acf425c 100644
--- a/3rdparty/Makefile.am
+++ b/3rdparty/Makefile.am
@@ -450,16 +450,12 @@ $(LIB_GRPC): $(GRPC)-build-stamp
 #
 # We use EXTRA_CPPFLAGS to pass `-Wno-error`, because that one needs to be
 # appended and not prepended to the constructed flags.
-#
-# NOTE: We ignore `array-bounds` errors generated by the GLIBC's `__strcmp_cg`
-# macro used in gRPC's c-ares library when compiled with Clang.
 $(GRPC)-build-stamp: $(GRPC)-stamp			\
                      $(PROTOBUF)-build-stamp
 	cd $(GRPC) &&					\
 	  CPPFLAGS="$(PROTOBUF_INCLUDE_FLAGS)		\
 	            $(SSL_INCLUDE_FLAGS)		\
 	            $(ZLIB_INCLUDE_FLAGS)		\
-	            -Wno-array-bounds			\
 	            $(CPPFLAGS)"			\
 	  CFLAGS="$(CFLAGS)"				\
 	  CXXFLAGS="$(CXXFLAGS)"			\