Skip to content

build(debug): Add CMake option to generate full debug information for VC6 builds#1370

Merged
xezon merged 1 commit intoTheSuperHackers:mainfrom
Mauller:cmake-vc6-full-debug
Jul 27, 2025
Merged

build(debug): Add CMake option to generate full debug information for VC6 builds#1370
xezon merged 1 commit intoTheSuperHackers:mainfrom
Mauller:cmake-vc6-full-debug

Conversation

@Mauller
Copy link

@Mauller Mauller commented Jul 27, 2025

This PR adds a cmake option to make VC6 generate builds with full debug information.

This does slow down the build process so it disabled by default.

This sets the compiler options /Zi and sets the build pools to 1 as generating the full debug info is not supported with VC6 when multi-threaded builds are used.

@Mauller Mauller self-assigned this Jul 27, 2025
@Mauller Mauller added Build Anything related to building, compiling Debug Is mostly debug functionality labels Jul 27, 2025
Copy link

Choose a reason for hiding this comment

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

I think this needs to be moved up near the other set_property(GLOBAL PROPERTY JOB_POOLS ?

Copy link
Author

Choose a reason for hiding this comment

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

Should be good now, i initially went with setting it seperate to keep things a bit cleaner looking

@Mauller Mauller force-pushed the cmake-vc6-full-debug branch from 6513bdf to ab98344 Compare July 27, 2025 15:31
Copy link

@xezon xezon left a comment

Choose a reason for hiding this comment

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

Works well

@xezon xezon changed the title build(debug): Implement option to make VC6 builds generate full debug information build(debug): Add CMake option RTS_BUILD_OPTION_VC6_FULL_DEBUG to generate full debug information for VC6 builds Jul 27, 2025
@xezon xezon changed the title build(debug): Add CMake option RTS_BUILD_OPTION_VC6_FULL_DEBUG to generate full debug information for VC6 builds build(debug): Add CMake option to generate full debug information for VC6 builds Jul 27, 2025
@xezon xezon merged commit fef5251 into TheSuperHackers:main Jul 27, 2025
16 checks passed
@xezon xezon deleted the cmake-vc6-full-debug branch July 27, 2025 16:03
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Anything related to building, compiling Debug Is mostly debug functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants