[Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

[elextr]

I suggest using this, @kugel- has already made a PR for Scintilla 5 and who knows what that needs.

Of course its not _just_ the compiler, but also the standard library (which is where string_view lives) that needs to be modern enough, but the compiler is possibly a proxy for that ... maybe.  

I just investigated this Mint (ie Ubuntu LTS based) system that has been allowed to handle compilers itself, it has gcc 8-10 available in the repos with 9.3 installed by default, and libstdc++ 8-10 available but 10.3 installed by default, interesting, I guess its ABI compatible with 9.3 so it can still be used with a 9.3 compiler but benefit from the bugfixes/additions.  It also has clang 8-12 available with clang 10 installed by default and libc++ 8-12 available, but none installed.  So clang is using libstdc++ here.  All are Ubuntu packages, so I would expect Ubuntu LTS systems to be the same.

So I suggest we use this as "reasonable efforts, feel free to contribute better tests if you think they are needed".

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2862#issuecomment-900710576
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210817/8ba0de98/attachment.htm>