-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
The CMakeLists.txt
file specifies "Release"
as the default build type in the following lines:
if(NOT CMAKE_BUILD_TYPE)
set(CMAKE_BUILD_TYPE
"Release"
CACHE
STRING
"Build mode ('DebugSan' or 'Debug' or 'Release', default is 'Debug')"
FORCE)
endif()
However, the comment indicates that the default should be 'Debug'
. This discrepancy can lead to confusion and potentially unintended builds in the release mode when the user expects a debug build.
Steps to Reproduce:
- Clone the repository.
- Create a build directory and run
cmake ..
. - Observe the default build type set in the generated
CMakeCache.txt
file.
Expected Behavior:
The default build type should be 'Debug'
as indicated by the comment.
Actual Behavior:
The default build type is set to 'Release'
.
Proposed Solution:
Update the CMakeLists.txt
to set the default build type to 'Debug'
to align with the comment.
if(NOT CMAKE_BUILD_TYPE)
set(CMAKE_BUILD_TYPE
"Debug"
CACHE
STRING
"Build mode ('DebugSan' or 'Debug' or 'Release', default is 'Debug')"
FORCE)
endif()
Additional Context:
Ensuring the default build type is correctly set to 'Debug'
will help developers catch issues early during development and align with the documented behavior.
Metadata
Metadata
Assignees
Labels
No labels