csmith-project / creduce

C-Reduce, a C and C++ program reducer
Other
1.23k stars 125 forks source link

Don't override CMAKE_CXX_FLAGS_RELEASE #249

Open diizzyy opened 1 year ago

diizzyy commented 1 year ago

This is bad practice and causes more work for packagers in various repos. Just add a note in INSTALL.md if it still worth using it and/or a toggle.

Reference: https://github.com/csmith-project/creduce/blob/master/CMakeLists.txt#L123

eeide commented 1 year ago

Thank you for opening this issue! I agree that the CMake stuff needs to be revisited and updated.