You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Can we default the build type to RelWithDebInfo for SDL and the satellite libraries? Can we also add a note to the cmake output if tests are not enabled?
I'd like users to be in a good position if they naively just run cmake, so they get good performance and have the ability to debug and know that there are test programs available. That should help the newbie experience a lot, as seen in this video: https://www.youtube.com/watch?v=PuE98lipGU8
The text was updated successfully, but these errors were encountered:
RelWithDebInfo for SDL3 is the default since 48b6b93.
You mean something like this?
-- Build Shared Library: ON (SDL_SHARED)
-- Build Static Library: ON (SDL_STATIC)
-- Build Test Library: ON (SDL_TEST_LIBRARY)
-- Build Tests: ON (SDL_TESTS, needs SDL_TEST_LIBRARY)
-- Build Examples: ON (SDL_EXAMPLES)
-- Build Static Library with Position Independent Code: OFF
RelWithDebInfo for SDL3 is the default since 48b6b93.
Great! Can you roll this out to the satellite libraries as well?
You mean something like this?
-- Build Shared Library: ON (SDL_SHARED)
-- Build Static Library: ON (SDL_STATIC)
-- Build Test Library: ON (SDL_TEST_LIBRARY)
-- Build Tests: ON (SDL_TESTS, needs SDL_TEST_LIBRARY)
-- Build Examples: ON (SDL_EXAMPLES)
-- Build Static Library with Position Independent Code: OFF
No, I mean something a little more verbose, e.g.
You can build test and sample programs by running cmake with -DSDL_TESTS=ON
Can we default the build type to RelWithDebInfo for SDL and the satellite libraries? Can we also add a note to the cmake output if tests are not enabled?
I'd like users to be in a good position if they naively just run cmake, so they get good performance and have the ability to debug and know that there are test programs available. That should help the newbie experience a lot, as seen in this video: https://www.youtube.com/watch?v=PuE98lipGU8
The text was updated successfully, but these errors were encountered: