]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph: Remove unavailable option with clang 734/head
authorChristophe Courtaut <christophe.courtaut@gmail.com>
Tue, 15 Oct 2013 14:23:37 +0000 (16:23 +0200)
committerChristophe Courtaut <christophe.courtaut@gmail.com>
Thu, 17 Oct 2013 16:00:39 +0000 (18:00 +0200)
Some options are unavailable in clang. (-Wstrict-null-sentinel,
-rdynamic)

They were reintroduced by this commit
https://github.com/ceph/ceph/commit/6949d221ada12e6be4b5ebf94b58edd17288fb16

This patch fixes it.

Signed-off-by: Christophe Courtaut <christophe.courtaut@gmail.com>
src/Makefile-env.am

index 6a4e09512a29a99d8e49cb424aad8bf0a5e0ea5c..9b611596c7f2868fa972728416fd3c9bc5542d70 100644 (file)
@@ -52,7 +52,6 @@ AM_COMMON_CPPFLAGS = \
        -DCEPH_LIBDIR=\"${libdir}\"
 
 AM_COMMON_CFLAGS = \
-       -rdynamic \
        -Wall \
        ${WARN_TYPE_LIMITS} \
        ${WARN_IGNORED_QUALIFIERS} \
@@ -61,6 +60,9 @@ AM_COMMON_CFLAGS = \
        -Werror=format-security \
        -fno-strict-aliasing \
        -fsigned-char
+if !CLANG
+       AM_COMMON_FLAGS += -rdynamic
+endif
 
 AM_CFLAGS = $(AM_COMMON_CFLAGS)
 AM_CPPFLAGS = $(AM_COMMON_CPPFLAGS)
@@ -68,8 +70,10 @@ AM_CXXFLAGS = \
        @AM_CXXFLAGS@ \
        $(AM_COMMON_CFLAGS) \
        -Wnon-virtual-dtor \
-       -Wno-invalid-offsetof \
-       -Wstrict-null-sentinel
+       -Wno-invalid-offsetof
+if !CLANG
+       AM_CXXFLAGS += -Wstrict-null-sentinel
+endif
 
 # note: this is position dependant, it affects the -l options that
 # come after it on the command line. when you use ${AM_LDFLAGS} in