Skip to content

Enable setting DEBUG_MODE for C through build.py#2572

Merged
fenhl merged 1 commit into
OoTRandomizer:Devfrom
djevangelia:debugmode
May 31, 2026
Merged

Enable setting DEBUG_MODE for C through build.py#2572
fenhl merged 1 commit into
OoTRandomizer:Devfrom
djevangelia:debugmode

Conversation

@djevangelia
Copy link
Copy Markdown

DEBUG_MODE is defined in debug.h and must be manually changed in the file when switching between debug and non-debug build. It's tedious and tampers with the file. It would be helpful to be able to change when building.

Here, by running build.py --debug-c, DEBUG_MODE is set to 1 in the Makefile, so the C is compiled with DEBUG_MODE set. If DEBUG_MODE isn't externally defined, it is still defined as 0 by debug.h.

Testing

I compiled with and without --debug-c with the vanilla build.py and Makefile as far as I could (I need some changes to compile with my compiler).

@GSKirox GSKirox added Component: ASM/C Changes some internals of the ASM/C libraries Status: Needs Review Someone should be looking at it Type: Maintenance Code style, infrastructure, updating dependencies labels May 9, 2026
@fenhl fenhl added Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release and removed Status: Needs Review Someone should be looking at it labels May 9, 2026
@fenhl fenhl added this to the next milestone May 31, 2026
@fenhl fenhl merged commit 760df78 into OoTRandomizer:Dev May 31, 2026
6 checks passed
@fenhl fenhl removed the Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release label May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: ASM/C Changes some internals of the ASM/C libraries Type: Maintenance Code style, infrastructure, updating dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants