Skip to content

Add scons options for compilation verbosity #104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
clemisch wants to merge 2 commits into
base: main
Choose a base branch
from
+35 −4
Open

Add scons options for compilation verbosity #104

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fd94220
Add warnings options to scons
clemisch Mar 30, 2026
dd67b9e
Only print compiler errors, no warnings
clemisch Jun 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 35 additions & 4 deletions SConstruct
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,10 @@ def ftpyflag(flags):
pattern = re.compile(r'^(-g|-Wstrict-prototypes|-O\d|-fPIC)$')
return [f for f in flags if not (isinstance(f, str) and pattern.match(f))]


def filter_warning_flags(flags):
return [f for f in flags if not (isinstance(f, str) and f.startswith('-W'))]

# copy system environment variables related to compilation
DefaultEnvironment(ENV=subdictionary(os.environ, '''
PATH PYTHONPATH GIT_DIR HOMEPATH HOMEDRIVE
Expand All @@ -68,6 +72,13 @@ vars.Add(EnumVariable(
'build',
'compiler settings',
'fast', allowed_values=('debug', 'fast')))
vars.Add(EnumVariable(
'warnings',
'warning flags policy',
'none', allowed_values=('all', 'none', 'default')))
vars.Add(BoolVariable(
'verbose',
'print build commands', False))
vars.Add(EnumVariable(
'tool',
'C++ compiler toolkit to be used',
Expand All @@ -76,6 +87,8 @@ vars.Add(BoolVariable(
'profile',
'build with profiling information', False))
vars.Update(env)
SetOption('silent', not env['verbose'])
SetOption('no_progress', not env['verbose'])

# Use C++ compiler specified by the 'tool' option.
if env['tool'] == 'intelc':
Expand Down Expand Up @@ -149,7 +162,8 @@ else:
# use sysconfig on Windows
pythonconfig = None
xpython = pjoin(env['prefix'], 'python.exe')
print(f"Using python-config: {pythonconfig} from {xpython}")
if env['verbose']:
print(f"Using python-config: {pythonconfig} from {xpython}")


common_cppdefs = ['REAL=double', 'BOOST_ERROR_CODE_HEADER_ONLY']
Expand All @@ -158,6 +172,8 @@ env.AppendUnique(CPPDEFINES=common_cppdefs)
if env['PLATFORM'] == 'win32':
env.AppendUnique(CPPDEFINES=['BOOST_ALL_NO_LIB'])
env.AppendUnique(CCFLAGS=['/EHsc', '/MD'])
if env['warnings'] == 'none':
env.PrependUnique(CCFLAGS=['/w'])

if env['build'] == 'debug':
env.Append(CCFLAGS=['/Zi', '/Od', '/FS'])
Expand All @@ -172,13 +188,28 @@ else:
# not using sysconfig here because of parsing issues
env.ParseConfig(f"{pythonconfig} --cflags")
env.Replace(CCFLAGS=ftpyflag(env['CCFLAGS']))

env.PrependUnique(CCFLAGS=['-Wextra'])
if env['warnings'] != 'all':
env.Replace(CCFLAGS=filter_warning_flags(env['CCFLAGS']))

if env['warnings'] == 'all':
env.PrependUnique(CCFLAGS=['-Wextra'])
elif env['warnings'] == 'none':
env.PrependUnique(CCFLAGS=['-w'])
env.PrependUnique(LINKFLAGS=['-w'])
# GCC emits an unconditional warning when LTO objects are linked
# without an explicit parallelization mode.
if ('-flto' in env['CCFLAGS'] and
any(isinstance(f, str) and
f.startswith('-flto-partition=')
for f in env['CCFLAGS'])):
env.PrependUnique(LINKFLAGS=['-flto=auto'])
env.PrependUnique(CXXFLAGS=['-std=c++11'])

if env['tool'] == 'intelc':
# options for Intel C++ compiler on hpc dev-intel07
env.AppendUnique(CCFLAGS=['-w1', '-fp-model', 'precise'])
if env['warnings'] != 'none':
env.AppendUnique(CCFLAGS=['-w1'])
env.AppendUnique(CCFLAGS=['-fp-model', 'precise'])
env.PrependUnique(LIBS=['imf'])
fast_opts = ['-fast', '-no-ipo']
else:
Expand Down
Loading