X Tutup
Skip to content

SCons: Ensure CPPDEFINES is properly utilized#108613

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Repiteo:scons/fix-defines
Sep 16, 2025
Merged

SCons: Ensure CPPDEFINES is properly utilized#108613
Repiteo merged 1 commit intogodotengine:masterfrom
Repiteo:scons/fix-defines

Conversation

@Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Jul 14, 2025

A handful of defines in SCons are declared in non-portable manner. This addresses that issue by guaranteeing CPPDEFINES is used instead of CCFLAGS with compiler-specific prefixes/suffixes.

@Repiteo Repiteo added this to the 4.x milestone Jul 14, 2025
@Repiteo Repiteo requested a review from a team as a code owner July 14, 2025 18:10
Copy link
Member

@Ivorforce Ivorforce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds useful to me.

@Ivorforce Ivorforce modified the milestones: 4.x, 4.6 Jul 15, 2025
@Repiteo Repiteo merged commit 4dec6af into godotengine:master Sep 16, 2025
39 of 40 checks passed
@Repiteo Repiteo deleted the scons/fix-defines branch September 16, 2025 16:57
ProfessorOctopus pushed a commit to ProfessorOctopus/godot that referenced this pull request Sep 17, 2025
SCons: Ensure `CPPDEFINES` is properly utilized
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.

2 participants

X Tutup