search

tzlaine / parser

A C++ parser combinator library.
Boost Software License 1.0
69 stars 12 forks source link

CI should test clang + libstdc++, newer compilers #163

Closed Jannik2099 closed 3 months ago

Jannik2099 commented 3 months ago

libstdc++ is the STL used on most linux targets, and clang will default to libstdc++ on practically all of them.

At the same time, incompatibilities may be more common particularly around new language features, as libstdc++ is developed against the language support of g++ foremost - see e.g. https://github.com/llvm/llvm-project/issues/54052 for a recent example.

I would propose adding a new CI workflow based on an up to date distro (my goto for this is fedora) to test the latest gcc and clang versions - the stock Ubuntu image offered by Github is and will always be outdated since it's on a 2 year LTS schedule.

Is that roughly to your liking? I would work on a PR if so.

tzlaine commented 3 months ago

I regularly test on Linux + GCC 13 and Linux + GCC 12 locally; this is how all the development occurs. CI handles older GCCs. I definitely would like to have coverage for Clang 15 and Clang 16, on Linux and elsewhere. So yes, that would be very much to my liking. :) I am currently using Github actions and Drone testing provided by the C++ Alliance to Boost repos, and neither one covers recent Clangs as far as I can tell.