Katherine Whitlock
Katherine Whitlock
Yeah I think this might actually be an upstream CTest bug, since I've run into it before
> I think we also need to configure fatfs as re entrant to do this safely, this change means multiple different dirs can be held but fatfs still can't access...
@jamiefaye if you're referring to sxDIR in https://github.com/SynthstromAudible/DelugeFirmware/blob/d4dda6825904b8ccb3450588b5feee592d7c8029/src/deluge/storage/smsysex.cpp#L26 this PR does not touch that
> Thank you so much for working on this! I did a quick test and for some reason found that the binary cmake is creating is much larger than one...
Oh perfect. Yeah, I probably need to think of a better way to have both sane default optimization levels and propagate user-selected ones. I'll poke at that today.
Changes: - Made a separate "daisy-tests" CMake project for test building/running. This allows IDEs such as VSCode and CLion to be able to build (and run) both the unit tests...
@asavartsov looks like I missed a file in the merge conflict resolution, whoops. Thanks for finding that! That should fix the undefined error for the linker. Instead of using `TOOLCHAIN_PREFIX`...
@asavartsov thank you so much for testing this all out! It looks like because the Makefile was building everything as one large static blob it didn't care so much about...
@asavartsov no worries! I'd rather get this all worked out in the PR than have to deal with it in mainline. 1) yeah the current setup will not propagate the...
Yeah considering how heavily it's used in certain places of libDaisy, and even shown in the examples, I think that's a reasonable solution. Ideally we could convert them to a...