From 564b0aea80db73ed242a6b81f48d7747e48404aa Mon Sep 17 00:00:00 2001 From: "Rob Swindell (ChromeOS)" <rob@synchro.net> Date: Wed, 25 Jan 2023 20:03:00 -0800 Subject: [PATCH] Patching the configure.in seems the better approach The previous method (patching Makefile.in) didn't disable warnings when building 'jsshell' (e.g. js.cpp). --- 3rdp/build/js-disable-warnings.patch | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/3rdp/build/js-disable-warnings.patch b/3rdp/build/js-disable-warnings.patch index 3f4a6295e6..7332972699 100644 --- a/3rdp/build/js-disable-warnings.patch +++ b/3rdp/build/js-disable-warnings.patch @@ -1,11 +1,10 @@ ---- js-1.8.5/js/src/Makefile.in 2023-01-25 18:43:01.101375334 -0800 -+++ js-1.8.5/js/src/Makefile.in 2023-01-25 18:37:27.917346828 -0800 -@@ -652,6 +652,8 @@ - NSPR_STATIC_PATH = $(DIST)/lib - endif +--- js-1.8.5/js/src/configure.in.orig 2023-01-25 20:00:45.568773908 -0800 ++++ js-1.8.5/js/src/configure.in 2023-01-25 19:59:37.694768104 -0800 +@@ -1700,6 +1700,7 @@ + if test "$ac_has_werror_return_type" = "yes"; then + _WARNINGS_CXXFLAGS="${_WARNINGS_CXXFLAGS} -Werror=return-type" + fi ++ _WARNINGS_CXXFLAGS="-w" -+CXXFLAGS += -w -+ - ifdef MOZ_VTUNE - CXXFLAGS += -IC:/Program\ Files/Intel/VTune/Analyzer/Include - EXTRA_DSO_LDOPTS += C:/Program\ Files/Intel/VTune/Analyzer/Lib/VtuneApi.lib + else + _DEFINES_CXXFLAGS='-DMOZILLA_CLIENT -D_JS_CONFDEFS_H_ $(ACDEFINES)' -- GitLab