devkitPro / gba-examples

92 stars 22 forks source link

Add example for using DKP with CMake and VSCode #3

Closed HorstBaerbel closed 5 years ago

HorstBaerbel commented 5 years ago

TODO:

HorstBaerbel commented 5 years ago

Before it gets lost in time, I'll just leave this here. Note that I pushed those to , but it has a MIT license so I included that for the subfolder. I hope this is ok with DKPs license...

WinterMute commented 5 years ago

Sorry, this doesn't really fit in with the way we do things.

All our examples have a standard layout that uses plain Makefiles for several reasons. We don't provide project files for any particular IDE to avoid requiring that IDE in order to build a project. While we do have cmake toolchain files, we don't use them for examples and single platform projects. Requiring end users to have tools that aren't commonly installed is antithetical to our design philosophy.

Merging this PR would also violate the DRY principle since it contains cmake toolchain files (which, fwiw, contain really odd bodges for no readily apparent reason) See https://github.com/devkitPro/pacman-packages/tree/master/pkgbuild-scripts

This is also out of date, and has been for quite some time given that devkitARM is now using gcc 8.2.0,

HorstBaerbel commented 5 years ago

There's no need to use VSCode. You can use CMake from the command line. It's even documented in the README. It's also not out of date. The hardcoded 7.1.0 stuff is just for intellisense. I would upgrade it if r50 is working for me. There might be a way to use environment variables for this. It's rough around the edges, that's why the TODOs are there... I've been using this setup for month now, without issues other that programming errors, which are not CMakes fault...

WinterMute commented 5 years ago

Cmake isn't commonly installed on end user's machines and requires an extra step before building a project. We don't want vscode files or alternative cmake toolchain files, even if they didn't contain peculiar bodgery.

Whether or not you've been using it for months without issues is irrelevant. We've been down this rabbit hole several times and we're not going down it again.