diff options
author | Matheus Marchini <matheusdot@gmail.com> | 2017-11-03 22:57:18 -0200 |
---|---|---|
committer | Refael Ackermann <refack@gmail.com> | 2017-11-27 17:16:37 -0500 |
commit | 4a7487b171d4e22a8e8879bd424f4f605d44abaf (patch) | |
tree | e9ebb133c8cc1ef645dda034b04a69e4d4022416 /configure | |
parent | cc7fe6032163d6980cef949b3eb854993c228a8f (diff) | |
download | android-node-v8-4a7487b171d4e22a8e8879bd424f4f605d44abaf.tar.gz android-node-v8-4a7487b171d4e22a8e8879bd424f4f605d44abaf.tar.bz2 android-node-v8-4a7487b171d4e22a8e8879bd424f4f605d44abaf.zip |
build: fix bsd build with gcc
BSD build with GCC was broken because it was checking for the
llvm_version variable on common.gypi, even though llvm wasn't
installed (or needed).
PR-URL: https://github.com/nodejs/node/pull/16737
Fixes: https://github.com/nodejs/node/issues/16257
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Diffstat (limited to 'configure')
-rwxr-xr-x | configure | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -692,8 +692,7 @@ def check_compiler(o): # to a version that is not completely ancient. warn('C compiler too old, need gcc 4.2 or clang 3.2 (CC=%s)' % CC) - if is_clang: - o['variables']['llvm_version'] = get_llvm_version(CC) + o['variables']['llvm_version'] = get_llvm_version(CC) if is_clang else 0 # Need xcode_version or gas_version when openssl asm files are compiled. if options.without_ssl or options.openssl_no_asm or options.shared_openssl: |