From 47e07c534fe994f2f03b84e1167132a65dada166 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Sun, 28 Jul 2019 21:14:57 +0200 Subject: [PATCH] build: remove support for s390 (but not s390x) Upstream V8 removed support for s390 earlier this year and it's known to no longer build. Remove the support from our build scripts. Fixes: https://github.com/nodejs/node/issues/28866 --- common.gypi | 4 ---- configure.py | 6 ++---- deps/openssl/openssl-cl_asm.gypi | 2 -- deps/openssl/openssl-cl_asm_avx2.gypi | 2 -- deps/openssl/openssl-cl_no_asm.gypi | 2 -- deps/openssl/openssl_asm.gypi | 2 -- deps/openssl/openssl_asm_avx2.gypi | 2 -- deps/openssl/openssl_no_asm.gypi | 2 -- tools/v8_gypfiles/broken/standalone.gypi | 2 +- tools/v8_gypfiles/toolchain.gypi | 14 +++++++------- tools/v8_gypfiles/v8.gyp | 6 +++--- 11 files changed, 13 insertions(+), 31 deletions(-) diff --git a/common.gypi b/common.gypi index 634909c1ad45..e073be1d14c7 100644 --- a/common.gypi +++ b/common.gypi @@ -388,10 +388,6 @@ 'cflags': [ '-m64', '-mminimal-toc' ], 'ldflags': [ '-m64' ], }], - [ 'target_arch=="s390"', { - 'cflags': [ '-m31', '-march=z196' ], - 'ldflags': [ '-m31', '-march=z196' ], - }], [ 'target_arch=="s390x"', { 'cflags': [ '-m64', '-march=z196' ], 'ldflags': [ '-m64', '-march=z196' ], diff --git a/configure.py b/configure.py index a791efebbca1..cc805d3fd165 100755 --- a/configure.py +++ b/configure.py @@ -47,7 +47,7 @@ valid_os = ('win', 'mac', 'solaris', 'freebsd', 'openbsd', 'linux', 'android', 'aix', 'cloudabi') valid_arch = ('arm', 'arm64', 'ia32', 'mips', 'mipsel', 'mips64el', 'ppc', - 'ppc64', 'x32','x64', 'x86', 'x86_64', 's390', 's390x') + 'ppc64', 'x32','x64', 'x86', 'x86_64', 's390x') valid_arm_float_abi = ('soft', 'softfp', 'hard') valid_arm_fpu = ('vfp', 'vfpv3', 'vfpv3-d16', 'neon') valid_mips_arch = ('loongson', 'r1', 'r2', 'r6', 'rx') @@ -867,7 +867,6 @@ def host_arch_cc(): '__PPC64__' : 'ppc64', '__PPC__' : 'ppc64', '__x86_64__' : 'x64', - '__s390__' : 's390', '__s390x__' : 's390x', } @@ -876,8 +875,7 @@ def host_arch_cc(): for i in matchup: if i in k and k[i] != '0': rtn = matchup[i] - if rtn != 's390': - break + break if rtn == 'mipsel' and '_LP64' in k: rtn = 'mips64el' diff --git a/deps/openssl/openssl-cl_asm.gypi b/deps/openssl/openssl-cl_asm.gypi index 2dd71a73b53e..b7138f98008a 100644 --- a/deps/openssl/openssl-cl_asm.gypi +++ b/deps/openssl/openssl-cl_asm.gypi @@ -10,8 +10,6 @@ 'includes': ['config/archs/linux-ppc64le/asm/openssl-cl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/asm/openssl-cl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/asm/openssl-cl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/asm/openssl-cl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/deps/openssl/openssl-cl_asm_avx2.gypi b/deps/openssl/openssl-cl_asm_avx2.gypi index f05c02171a60..c1c398835f3b 100644 --- a/deps/openssl/openssl-cl_asm_avx2.gypi +++ b/deps/openssl/openssl-cl_asm_avx2.gypi @@ -10,8 +10,6 @@ 'includes': ['config/archs/linux-ppc64le/asm_avx2/openssl-cl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/asm_avx2/openssl-cl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/asm_avx2/openssl-cl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/asm_avx2/openssl-cl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/deps/openssl/openssl-cl_no_asm.gypi b/deps/openssl/openssl-cl_no_asm.gypi index f3e1373802cc..2b65fb6aec5b 100644 --- a/deps/openssl/openssl-cl_no_asm.gypi +++ b/deps/openssl/openssl-cl_no_asm.gypi @@ -10,8 +10,6 @@ 'includes': ['config/archs/linux-ppc64le/no-asm/openssl-cl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/no-asm/openssl-cl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/no-asm/openssl-cl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/no-asm/openssl-cl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/deps/openssl/openssl_asm.gypi b/deps/openssl/openssl_asm.gypi index c839b66f5083..4975da56e0d8 100644 --- a/deps/openssl/openssl_asm.gypi +++ b/deps/openssl/openssl_asm.gypi @@ -10,8 +10,6 @@ 'includes': ['config/archs/linux-ppc64le/asm/openssl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/asm/openssl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/asm/openssl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/asm/openssl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/deps/openssl/openssl_asm_avx2.gypi b/deps/openssl/openssl_asm_avx2.gypi index c5a636a3301a..4d7580c386e5 100644 --- a/deps/openssl/openssl_asm_avx2.gypi +++ b/deps/openssl/openssl_asm_avx2.gypi @@ -10,8 +10,6 @@ 'includes': ['config/archs/linux-ppc64le/asm_avx2/openssl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/asm_avx2/openssl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/asm_avx2/openssl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/asm_avx2/openssl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/deps/openssl/openssl_no_asm.gypi b/deps/openssl/openssl_no_asm.gypi index e492c43ffbe8..588aa543f469 100644 --- a/deps/openssl/openssl_no_asm.gypi +++ b/deps/openssl/openssl_no_asm.gypi @@ -11,8 +11,6 @@ 'includes': ['config/archs/linux-ppc64le/no-asm/openssl.gypi'], }, 'target_arch=="ppc64" and OS=="linux"', { 'includes': ['config/archs/linux-ppc64/no-asm/openssl.gypi'], - }, 'target_arch=="s390" and OS=="linux"', { - 'includes': ['config/archs/linux32-s390x/no-asm/openssl.gypi'], }, 'target_arch=="s390x" and OS=="linux"', { 'includes': ['config/archs/linux64-s390x/no-asm/openssl.gypi'], }, 'target_arch=="arm" and OS=="linux"', { diff --git a/tools/v8_gypfiles/broken/standalone.gypi b/tools/v8_gypfiles/broken/standalone.gypi index 2a41237a608c..f01b5975fc58 100644 --- a/tools/v8_gypfiles/broken/standalone.gypi +++ b/tools/v8_gypfiles/broken/standalone.gypi @@ -165,7 +165,7 @@ }, { 'gomadir': '