Skip to content

Commit

Permalink
Update to require C++17.
Browse files Browse the repository at this point in the history
This then also bumps the minimum GCC version to 7.
  • Loading branch information
Pentarctagon committed Jan 16, 2021
1 parent ef7d69b commit 90946ca
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/ci-main.yml
Expand Up @@ -67,7 +67,7 @@ jobs:
export TOOL=scons
export CC=clang
export CXX=clang++
export CXX_STD=14
export CXX_STD=17
export CFG=debug
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand Down Expand Up @@ -97,7 +97,7 @@ jobs:
export TOOL=cmake
export CC=clang
export CXX=clang++
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand All @@ -119,7 +119,7 @@ jobs:
export TOOL=cmake
export CC=gcc
export CXX=g++
export CXX_STD=14
export CXX_STD=17
export CFG=debug
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand Down Expand Up @@ -149,7 +149,7 @@ jobs:
export TOOL=scons
export CC=gcc
export CXX=g++
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand All @@ -171,7 +171,7 @@ jobs:
export TOOL=scons
export CC=gcc-9
export CXX=g++-9
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand All @@ -193,7 +193,7 @@ jobs:
export TOOL=scons
export CC=gcc
export CXX=g++
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand All @@ -215,7 +215,7 @@ jobs:
export TOOL=scons
export CC=gcc
export CXX=g++
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand All @@ -237,7 +237,7 @@ jobs:
export TOOL=scons
export CC=gcc
export CXX=g++
export CXX_STD=14
export CXX_STD=17
export CFG=release
export LTO=false
export CACHE_DIR=/home/wesnoth-CI/build
Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Expand Up @@ -67,7 +67,7 @@ endif(ENABLE_MYSQL)

# set what std version to use
if(NOT CXX_STD)
set(CXX_STD "14")
set(CXX_STD "17")
endif()
set(CMAKE_CXX_STANDARD ${CXX_STD})
# make sure to force using it
Expand Down
4 changes: 2 additions & 2 deletions SConstruct
Expand Up @@ -109,7 +109,7 @@ opts.AddVariables(
BoolVariable('ccache', "Use ccache", False),
('ctool', 'Set c compiler command if not using standard compiler.'),
('cxxtool', 'Set c++ compiler command if not using standard compiler.'),
EnumVariable('cxx_std', 'Target c++ std version', '14', ['14', '17', '20']),
EnumVariable('cxx_std', 'Target c++ std version', '17', ['17', '20']),
('sanitize', 'Enable clang and GCC sanitizer functionality. A comma separated list of sanitize suboptions must be passed as value.', ''),
BoolVariable("fast", "Make scons faster at cost of less precise dependency tracking.", False),
BoolVariable("autorevision", 'Use autorevision tool to fetch current git revision that will be embedded in version string', True),
Expand Down Expand Up @@ -371,7 +371,7 @@ if env["prereqs"]:
env["PKG_CONFIG_FLAGS"] = "--dont-define-prefix"

have_server_prereqs = (\
conf.CheckCPlusPlus(gcc_version = "5.4") & \
conf.CheckCPlusPlus(gcc_version = "7") & \
conf.CheckBoost("iostreams", require_version = boost_version) & \
conf.CheckBoostIostreamsGZip() & \
conf.CheckBoostIostreamsBZip2() & \
Expand Down

0 comments on commit 90946ca

Please sign in to comment.