Skip to content

Commit

Permalink
SCons: Load optional ninja tool before setting its options
Browse files Browse the repository at this point in the history
SCons 4.8.0 made this stricter, as we were advised in godotengine#94805.

Fixes godotengine#94805.
  • Loading branch information
akien-mga committed Jul 31, 2024
1 parent b6dee96 commit d2064fe
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions SConstruct
Original file line number Diff line number Diff line change
Expand Up @@ -1025,14 +1025,13 @@ if env["ninja"]:
Exit(255)

SetOption("experimental", "ninja")
env.Tool("ninja")

# By setting this we allow the user to run ninja by themselves with all
# the flags they need, as apparently automatically running from scons
# is way slower.
SetOption("disable_execute_ninja", True)

env.Tool("ninja")

# Threads
if env["threads"]:
env.Append(CPPDEFINES=["THREADS_ENABLED"])
Expand Down

0 comments on commit d2064fe

Please sign in to comment.