Skip to content
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.

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

Bump minimum SCons version to 4.0 #85968

Closed
wants to merge 1 commit into from

Conversation

Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Dec 9, 2023

Now matches the minimum version used by godot-cpp. In addition, adjusted a handful of sections that needed workarounds for non-4.0 versions; namely: compile_db is now just natively available & methods.dump is replaced with a built-in alternative. GitHub Actions already have a minimum version of 4.4 specified, so no changes are needed there (unless we wanted to deliberately downgrade).

• Matches minimum version used by godot-cpp
@Repiteo Repiteo requested a review from a team as a code owner December 9, 2023 18:23
@Calinou Calinou added this to the 4.3 milestone Dec 9, 2023
@Calinou
Copy link
Member

Calinou commented Dec 9, 2023

Note that Ubuntu 20.04 only has SCons 3.1.2 packaged, but Ubuntu 22.04 has SCons 4.0.1: https://packages.ubuntu.com/search?keywords=scons

That said, we recommend the use of pip-packaged SCons as opposed to distribution-packaged SCons. This makes it easy to get the latest SCons version regardless of your Linux distribution.

@akien-mga akien-mga modified the milestones: 4.3, 4.x Jun 11, 2024
@akien-mga
Copy link
Member

Partially superseded by #92043. We can still consider moving to 4.0+ in the near future though.

@Repiteo
Copy link
Contributor Author

Repiteo commented Nov 12, 2024

Superseded by #99134

@Repiteo Repiteo closed this Nov 12, 2024
@Repiteo Repiteo deleted the scons-4.0 branch November 12, 2024 17:18
@Repiteo Repiteo removed this from the 4.x milestone Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants