diff options
author | José Fonseca <jfonseca@vmware.com> | 2014-04-16 13:13:48 +0100 |
---|---|---|
committer | José Fonseca <jfonseca@vmware.com> | 2014-04-16 13:18:06 +0100 |
commit | e3c58cdfd97d390cb4c1a02852ab0417bd68c861 (patch) | |
tree | 8aafd714553f10d794eff8e9c872f3f0e8d0ba99 /scons/gallium.py | |
parent | 11459436d9314681087463f2c006c58b6fcff396 (diff) |
Revert "scons: Enable building through Clang Static Analyzer."
This reverts commit a45a50a4828e1357e9555474bc127c5585b3a420.
Unfortunately gcc dumps argv[0] as the first word of --version, so it is
unreliable for detecting gcc.
In particular `cc --version` and `i686-w64-mingw32-gcc --version` give
wrong results.
A better solution needs to be found -- most likely using C-preprocessing
like autotools does. Revert for now.
Diffstat (limited to 'scons/gallium.py')
-rwxr-xr-x | scons/gallium.py | 31 |
1 files changed, 8 insertions, 23 deletions
diff --git a/scons/gallium.py b/scons/gallium.py index e873c658ad..bd71e51df7 100755 --- a/scons/gallium.py +++ b/scons/gallium.py @@ -104,19 +104,6 @@ def num_jobs(): return 1 -def get_cc_version(env): - # Get the first line of `$CC --version` - pipe = SCons.Action._subproc(env, [env['CC'], '--version'], - stdin = 'devnull', - stderr = 'devnull', - stdout = subprocess.PIPE) - if pipe.wait() != 0: - return '' - - line = pipe.stdout.readline() - return line - - def generate(env): """Common environment generation code""" @@ -132,8 +119,12 @@ def generate(env): if os.environ.has_key('CC'): env['CC'] = os.environ['CC'] # Update CCVERSION to match - line = get_cc_version(env) - if line: + pipe = SCons.Action._subproc(env, [env['CC'], '--version'], + stdin = 'devnull', + stderr = 'devnull', + stdout = subprocess.PIPE) + if pipe.wait() == 0: + line = pipe.stdout.readline() match = re.search(r'[0-9]+(\.[0-9]+)+', line) if match: env['CCVERSION'] = match.group(0) @@ -146,16 +137,10 @@ def generate(env): if os.environ.has_key('LDFLAGS'): env['LINKFLAGS'] += SCons.Util.CLVar(os.environ['LDFLAGS']) - # Detect gcc/clang not by executable name, but through `--version` option, - # to avoid drawing wrong conclusions when using tools that overrice CC/CXX - # like scan-build. - cc_version = get_cc_version(env) - cc_version_words = cc_version.split() - - env['gcc'] = 'gcc' in cc_version_words + env['gcc'] = 'gcc' in os.path.basename(env['CC']).split('-') env['msvc'] = env['CC'] == 'cl' env['suncc'] = env['platform'] == 'sunos' and os.path.basename(env['CC']) == 'cc' - env['clang'] = 'clang' in cc_version_words + env['clang'] = env['CC'] == 'clang' env['icc'] = 'icc' == os.path.basename(env['CC']) if env['msvc'] and env['toolchain'] == 'default' and env['machine'] == 'x86_64': |