GeezerVison

Build size and comparison to main:

Run in InfiniEmu

Looks like a nice improvement overall :) Thanks for sending it in

I'm not sure about storing the toolchain inside the tree. Maybe environment variables like with the VSCode instructions would be more flexible? What are your thoughts, I do agree that having to configure env vars can be a bit annoying

Thanks! 😁 Hmm... I see your point that using raw environment variables is very flexible, I just found it a bit annoying to use though - which is why I added the cmake presets file... These changes will technically not prevent users from manually setting the environment variables and calling cmake without the --preset option, but then that means we need to support multiple workflows. Sadly, there's no way to "use the environment variable if defined, otherwise, use the ${fileDir}/sdk-toolchain value(s)"... At least I haven't been able to figure it out without needing to compromise on flexibility...

To me, ideally, the buildsteps should just be:

• Clone the project (with submodules)
• cmake --preset Release
• cmake --build build/Release

Where the CMakeLists.txt would download the SDK and toolchain for you at configure-time. I would love to try to get that done but it would require touching the CI and it would be a fairly big change to the now normal work flow, so I'm not sure if it would be too disruptive. Perhaps adding the presets just adds confusion and we should just close this PR?

Closing as this became stale.