mirror of
https://github.com/nodejs/node.git
synced 2024-11-24 20:29:23 +01:00
build: fix version checks in configure.py
Fixes: https://github.com/nodejs/node/issues/29927 Refs: https://github.com/nodejs/node/pull/29931 PR-URL: https://github.com/nodejs/node/pull/29965 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
This commit is contained in:
parent
c8df5cf74a
commit
d769ebc2b7
@ -14,6 +14,7 @@ import shutil
|
||||
import bz2
|
||||
|
||||
from distutils.spawn import find_executable as which
|
||||
from distutils.version import StrictVersion
|
||||
|
||||
# If not run from node/, cd to node/.
|
||||
os.chdir(os.path.dirname(__file__) or '.')
|
||||
@ -1231,10 +1232,10 @@ def configure_openssl(o):
|
||||
# supported asm compiler for AVX2. See https://github.com/openssl/openssl/
|
||||
# blob/OpenSSL_1_1_0-stable/crypto/modes/asm/aesni-gcm-x86_64.pl#L52-L69
|
||||
openssl110_asm_supported = \
|
||||
('gas_version' in variables and float(variables['gas_version']) >= 2.23) or \
|
||||
('xcode_version' in variables and float(variables['xcode_version']) >= 5.0) or \
|
||||
('llvm_version' in variables and float(variables['llvm_version']) >= 3.3) or \
|
||||
('nasm_version' in variables and float(variables['nasm_version']) >= 2.10)
|
||||
('gas_version' in variables and StrictVersion(variables['gas_version']) >= StrictVersion('2.23')) or \
|
||||
('xcode_version' in variables and StrictVersion(variables['xcode_version']) >= StrictVersion('5.0')) or \
|
||||
('llvm_version' in variables and StrictVersion(variables['llvm_version']) >= StrictVersion('3.3')) or \
|
||||
('nasm_version' in variables and StrictVersion(variables['nasm_version']) >= StrictVersion('2.10'))
|
||||
|
||||
if is_x86 and not openssl110_asm_supported:
|
||||
error('''Did not find a new enough assembler, install one or build with
|
||||
|
Loading…
Reference in New Issue
Block a user