CI: Add Windows MSYS2, MacOS and QA workflow #24
Conversation
Use `apt-get` instead of `apt`. Avoids a warning that `apt` is not suitable for automation. Use single installation call to avoid ignored exit code. Add separate steps for smoke test and test preparation. Remove permissions settings which were needed for the remove test report. Explicit setting of shell is not necessary anymore, the tests run fine on Ubuntu without it. Maybe the addition of the executable flags or fixing the `shellcheck` issues helped. These were done in previous commits.
Also move the documentation generation to QA.
Add minimal example configuration.
|
FYI: Windows needs about 7, MacOS about 6 minutes. In comparison, Linux is about 1 and a half. |
|
All looks great. I've been scratching my head on how to do the mac build (I don't own one and was looking on ebay for something cheap...). One overall comment is that I've been considering switching the default generator to ninja everywhere (rather than make on Linux for example). Any thoughts? Would including it here make sense given all the spots where you need to explicitly run |
|
All the workflows use Ninja already. It's what I always use in combination with CMake (has its benefits, although maybe not relevant for SRecord). Inside the cmake files you should not set a default generator, it's not really possible and the decision is up to the user. |
Fair call. It's my preference too but no reason to try to impose that on others (even if it were possible). |
Additionally to the Linux builds, there's now also one on MacOS, and 3 MSYS2 based on Windows.
Also a minimal clang-tidy configuration and check was added.