13
0

wscript: drop unnecessary str() on configure variables

CC is already set to a string. (And if it ever should be None, we want
to handle that explicitly.)

(And #autowaf.display_msg handle Booleans just fine.)
This commit is contained in:
Mads Kiilerich 2022-10-15 14:13:11 +02:00 committed by Paul Davis
parent 0e3e51d958
commit 615326be9b
3 changed files with 4 additions and 4 deletions

View File

@ -488,7 +488,7 @@ def build(bld):
# usability of the 64 bit windows assembler depends on the compiler target, # usability of the 64 bit windows assembler depends on the compiler target,
# not the build host, which in turn can only be inferred from the name # not the build host, which in turn can only be inferred from the name
# of the compiler. # of the compiler.
if re.search ('x86_64-w64', str(bld.env['CC'])): if re.search ('x86_64-w64', bld.env['CC']):
obj.source += [ 'sse_functions_xmm.cc' ] obj.source += [ 'sse_functions_xmm.cc' ]
obj.source += [ 'sse_functions_64bit_win.s', 'sse_avx_functions_64bit_win.s' ] obj.source += [ 'sse_functions_64bit_win.s', 'sse_avx_functions_64bit_win.s' ]
avx_sources = [ 'sse_functions_avx.cc' ] avx_sources = [ 'sse_functions_avx.cc' ]

View File

@ -35,7 +35,7 @@ def configure(conf):
if Options.options.internal_shared_libs: if Options.options.internal_shared_libs:
conf.define('INTERNAL_SHARED_LIBS', 1) conf.define('INTERNAL_SHARED_LIBS', 1)
#autowaf.display_msg(conf, "Unit tests", str(conf.env['BUILD_TESTS'])) #autowaf.display_msg(conf, "Unit tests", conf.env['BUILD_TESTS'])
#print #print
def build(bld): def build(bld):

View File

@ -520,7 +520,7 @@ int main() { return 0; }''',
if conf.env['build_target'] == 'armhf' or conf.env['build_target'] == 'aarch64': if conf.env['build_target'] == 'armhf' or conf.env['build_target'] == 'aarch64':
conf.define('ARM_NEON_SUPPORT', 1) conf.define('ARM_NEON_SUPPORT', 1)
elif conf.env['build_target'] == 'mingw': elif conf.env['build_target'] == 'mingw':
if re.search ('x86_64-w64', str(conf.env['CC'])) is not None: if re.search ('x86_64-w64', conf.env['CC']) is not None:
conf.define ('FPU_AVX_FMA_SUPPORT', 1) conf.define ('FPU_AVX_FMA_SUPPORT', 1)
conf.define ('FPU_AVX512F_SUPPORT', 1) conf.define ('FPU_AVX512F_SUPPORT', 1)
elif conf.env['build_target'] == 'i386' or conf.env['build_target'] == 'i686' or conf.env['build_target'] == 'x86_64': elif conf.env['build_target'] == 'i386' or conf.env['build_target'] == 'i686' or conf.env['build_target'] == 'x86_64':
@ -636,7 +636,7 @@ int main() { return 0; }''',
# usability of the 64 bit windows assembler depends on the compiler target, # usability of the 64 bit windows assembler depends on the compiler target,
# not the build host, which in turn can only be inferred from the name # not the build host, which in turn can only be inferred from the name
# of the compiler. # of the compiler.
if re.search ('x86_64-w64', str(conf.env['CC'])) is not None: if re.search ('x86_64-w64', conf.env['CC']) is not None:
compiler_flags.append ("-DBUILD_SSE_OPTIMIZATIONS") compiler_flags.append ("-DBUILD_SSE_OPTIMIZATIONS")
if not build_host_supports_sse: if not build_host_supports_sse:
print("\nWarning: you are building Ardour with SSE support even though your system does not support these instructions. (This may not be an error, especially if you are a package maintainer)") print("\nWarning: you are building Ardour with SSE support even though your system does not support these instructions. (This may not be an error, especially if you are a package maintainer)")