From 3f1ce73e3ad6f55784c7ee48a0e8ab0e3db86d1a Mon Sep 17 00:00:00 2001 From: Ryan VanderMeulen Date: Wed, 22 Jan 2014 17:09:34 -0500 Subject: [PATCH] Bug 956398 - Fix bustage due to check-sync-dirs still running on Aurora/Beta. r=glandium, a=bustage --- js/src/config/config.mk | 11 ++++++++++- js/src/config/rules.mk | 6 +++--- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/js/src/config/config.mk b/js/src/config/config.mk index 2214caa427fc..f831204e4597 100644 --- a/js/src/config/config.mk +++ b/js/src/config/config.mk @@ -846,7 +846,7 @@ EXPAND_MKSHLIB = $(EXPAND_LIBS_EXEC) $(EXPAND_MKSHLIB_ARGS) -- $(MKSHLIB) ifneq (,$(MOZ_LIBSTDCXX_TARGET_VERSION)$(MOZ_LIBSTDCXX_HOST_VERSION)) ifneq ($(OS_ARCH),Darwin) -CHECK_STDCXX = objdump -p $(1) | grep -e 'GLIBCXX_3\.4\.\(9\|[1-9][0-9]\)' > /dev/null && echo "TEST-UNEXPECTED-FAIL | | We don't want these libstdc++ symbols to be used:" && objdump -T $(1) | grep -e 'GLIBCXX_3\.4\.\(9\|[1-9][0-9]\)' && exit 1 || exit 0 +CHECK_STDCXX = @$(TOOLCHAIN_PREFIX)objdump -p $(1) | grep -e 'GLIBCXX_3\.4\.\(9\|[1-9][0-9]\)' > /dev/null && echo 'TEST-UNEXPECTED-FAIL | check_stdcxx | We do not want these libstdc++ symbols to be used:' && $(TOOLCHAIN_PREFIX)objdump -T $(1) | grep -e 'GLIBCXX_3\.4\.\(9\|[1-9][0-9]\)' && false || true endif ifdef MOZ_LIBSTDCXX_TARGET_VERSION @@ -857,6 +857,15 @@ HOST_EXTRA_LIBS += $(call EXPAND_LIBNAME_PATH,host_stdc++compat,$(DEPTH)/build/u endif endif +ifeq (,$(filter $(OS_TARGET),WINNT Darwin OS2)) +CHECK_TEXTREL = @$(TOOLCHAIN_PREFIX)readelf -d $(1) | grep TEXTREL > /dev/null && echo 'TEST-UNEXPECTED-FAIL | check_textrel | We do not want text relocations in libraries and programs' || true +endif + +define CHECK_BINARY +$(call CHECK_STDCXX,$(1)) +$(call CHECK_TEXTREL,$(1)) +endef + # autoconf.mk sets OBJ_SUFFIX to an error to avoid use before including # this file OBJ_SUFFIX := $(_OBJ_SUFFIX) diff --git a/js/src/config/rules.mk b/js/src/config/rules.mk index 9e9d5e370a58..df0dbbeaec5f 100644 --- a/js/src/config/rules.mk +++ b/js/src/config/rules.mk @@ -804,7 +804,7 @@ ifdef MOZ_PROFILE_GENERATE endif else # !WINNT || GNU_CC $(EXPAND_CCC) -o $@ $(CXXFLAGS) $(PROGOBJS) $(RESFILE) $(WIN32_EXE_LDFLAGS) $(LDFLAGS) $(WRAP_LDFLAGS) $(LIBS_DIR) $(LIBS) $(MOZ_GLUE_PROGRAM_LDFLAGS) $(OS_LIBS) $(EXTRA_LIBS) $(BIN_FLAGS) $(EXE_DEF_FILE) $(STLPORT_LIBS) - @$(call CHECK_STDCXX,$@) + $(call CHECK_BINARY,$@) endif # WINNT && !GNU_CC ifdef ENABLE_STRIP @@ -860,7 +860,7 @@ ifdef MSMANIFEST_TOOL endif # MSVC with manifest tool else $(EXPAND_CCC) $(CXXFLAGS) -o $@ $< $(WIN32_EXE_LDFLAGS) $(LDFLAGS) $(WRAP_LDFLAGS) $(LIBS_DIR) $(LIBS) $(MOZ_GLUE_PROGRAM_LDFLAGS) $(OS_LIBS) $(EXTRA_LIBS) $(BIN_FLAGS) $(STLPORT_LIBS) - @$(call CHECK_STDCXX,$@) + $(call CHECK_BINARY,$@) endif # WINNT && !GNU_CC ifdef ENABLE_STRIP @@ -965,7 +965,7 @@ endif else # ! DTRACE_LIB_DEPENDENT $(EXPAND_MKSHLIB) $(SHLIB_LDSTARTFILE) $(OBJS) $(SUB_SHLOBJS) $(RESFILE) $(LDFLAGS) $(WRAP_LDFLAGS) $(SHARED_LIBRARY_LIBS) $(EXTRA_DSO_LDOPTS) $(MOZ_GLUE_LDFLAGS) $(OS_LIBS) $(EXTRA_LIBS) $(DEF_FILE) $(SHLIB_LDENDFILE) $(if $(LIB_IS_C_ONLY),,$(STLPORT_LIBS)) endif # DTRACE_LIB_DEPENDENT - @$(call CHECK_STDCXX,$@) + $(call CHECK_BINARY,$@) ifeq (_WINNT,$(GNU_CC)_$(OS_ARCH)) ifdef MSMANIFEST_TOOL -- 2.11.4.GIT