general: need a better way to select config files at compile time

[Selene ToyKeeper]

The code currently supports a bunch of different drivers and a bunch of different UI configurations, with one config per file. This is nice, but the way it pulls in those files requires having a few extra lines per configuration.

Basically, there are a bunch of "ifdef FOO, include foo.h" clauses. And I want to get rid of those. It's getting messy, as the number of supported build targets increases.

[Thomas Arthofer (arthofer-thomas)]

Found a way, working on it.

/*
C-preprocessor hack to include the cfg-file specified by the define "DRIVER"

Sources:
  https://www.microchip.com/forums/FindPost/845119
  https://stackoverflow.com/questions/9096201/concatenate-string-in-c-include-filename
  https://stackoverflow.com/questions/6742501/whats-the-exact-step-of-macro-expanding/6742619#6742619

*/

#ifndef DRIVER
#error You must define DRIVER to import the correct cfg_<driver>.h file
#endif

#define str(s) #s
#define xstr(s) str(s)
#define INCFILE2(a, n) a ## n
#define INCFILE(a,b) INCFILE2(a,b)

//#pragma message xstr(INCFILE(cfg_, DRIVER).h)
#include xstr(INCFILE(cfg_, DRIVER).h)

[Thomas Arthofer (arthofer-thomas)]

Looks like it's working for anduril, but since I had to rename a few header files, I should also check the other projects.

If you want to take a look, here is a preview: https://zeroflow.at/flashlight/flashlight-firmware-ref-include.zip

[Selene ToyKeeper (toykeeper)]

Thanks! This mechanism seems to work pretty well. I applied it throughout my part of the repository and uploaded a new revision to the fsm branch with the changes. It should be in fsm branch revision 397. Before/after shows no md5sum changes, so I don't think it broke anything.

I used a somewhat simpler version of the include mechanism though, and didn't rename any files.