Skip to content

Commit 06d2ec4

Browse files
committed
Merge pull request godotengine#99134 from Repiteo/scons/bump-scons-python
SCons: Bump minimum version of SCons to 4.0 & Python to 3.8
2 parents 606f0eb + 66fe2c8 commit 06d2ec4

File tree

5 files changed

+8
-19
lines changed

5 files changed

+8
-19
lines changed

.github/actions/godot-deps/action.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,5 @@ runs:
2727
shell: bash
2828
run: |
2929
python -c "import sys; print(sys.version)"
30-
python -m pip install wheel
3130
python -m pip install scons==${{ inputs.scons-version }}
3231
scons --version

.github/workflows/linux_builds.yml

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ concurrency:
1717

1818
jobs:
1919
build-linux:
20-
# If unspecified, stay one LTS before latest to increase portability of Linux artifacts.
21-
runs-on: ${{ matrix.os || 'ubuntu-22.04' }}
20+
# Stay one LTS before latest to increase portability of Linux artifacts.
21+
runs-on: ubuntu-22.04
2222
name: ${{ matrix.name }}
2323
strategy:
2424
fail-fast: false
@@ -61,8 +61,6 @@ jobs:
6161
artifact: false
6262
# Test our oldest supported SCons/Python versions on one arbitrary editor build.
6363
legacy-scons: true
64-
# Python 3.6 unavailable on 22.04.
65-
os: ubuntu-20.04
6664

6765
- name: Editor with ThreadSanitizer (target=editor, tests=yes, dev_build=yes, use_tsan=yes, use_llvm=yes, linker=lld)
6866
cache-name: linux-editor-thread-sanitizer
@@ -132,8 +130,8 @@ jobs:
132130
uses: ./.github/actions/godot-deps
133131
with:
134132
# Sync with Ensure*Version in SConstruct.
135-
python-version: 3.6
136-
scons-version: 3.1.2
133+
python-version: 3.8
134+
scons-version: 4.0
137135

138136
- name: Setup GCC problem matcher
139137
uses: ammaraskar/gcc-problem-matcher@master

SConstruct

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
#!/usr/bin/env python
22
from misc.utility.scons_hints import *
33

4-
EnsureSConsVersion(3, 1, 2)
5-
EnsurePythonVersion(3, 6)
4+
EnsureSConsVersion(4, 0)
5+
EnsurePythonVersion(3, 8)
66

77
# System
88
import atexit
@@ -1060,13 +1060,6 @@ if env["vsproj"]:
10601060
env.vs_srcs = []
10611061

10621062
if env["compiledb"]:
1063-
if env.scons_version < (4, 0, 0):
1064-
# Generating the compilation DB (`compile_commands.json`) requires SCons 4.0.0 or later.
1065-
print_error(
1066-
"The `compiledb=yes` option requires SCons 4.0 or later, but your version is %s." % scons_raw_version
1067-
)
1068-
Exit(255)
1069-
10701063
env.Tool("compilation_db")
10711064
env.Alias("compiledb", env.CompilationDatabase())
10721065

methods.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -409,8 +409,7 @@ def mySubProcess(cmdline, env):
409409
"shell": False,
410410
"env": env,
411411
}
412-
if sys.version_info >= (3, 7, 0):
413-
popen_args["text"] = True
412+
popen_args["text"] = True
414413
proc = subprocess.Popen(cmdline, **popen_args)
415414
_, err = proc.communicate()
416415
rv = proc.wait()

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ python_version = "3.8"
1616
extend-exclude = ["thirdparty"]
1717
extend-include = ["SConstruct", "SCsub"]
1818
line-length = 120
19-
target-version = "py37"
19+
target-version = "py38"
2020

2121
[tool.ruff.lint]
2222
extend-select = [

0 commit comments

Comments
 (0)