<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 29, 2021 at 11:13 PM Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 29, 2021 at 5:20 PM Moritz Lennert <<a href="mailto:mlennert@club.worldonline.be" target="_blank">mlennert@club.worldonline.be</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
Am 29. Januar 2021 20:54:06 GMT+00:00 schrieb Markus Metz <<a href="mailto:markus.metz.giswork@gmail.com" target="_blank">markus.metz.giswork@gmail.com</a>>:<br>
><br>
>For new PROJ<br>
>versions, C99 is a requirement.<br>
<br>
<br>
If proj requires it, doesn't it automatically become a requirement for GRASS as well ?<br></blockquote><div><br></div><div>That's a good point. We have the same situation with GDAL. It requires C++11 [1]. Although this may not set the requirement absolutely, it sets it practically.</div><div><br></div><div>Additionally, increasing the minimum for C let's say to C17,  may allow increasing the minimum for C++ because practically there is no compiler which would support one and not the other, but that depends on the particular combination.<br></div><div><br></div><div>In CI, I have set the GCC compilation tests only for C99 and above and C++11 and above [2] (i.e., not requiring anything below that). However, note that the C is GNU C, not ISO (which we currently don't conform to). The "pedantic" keeping of the standard fails for both C and C++ (and thus it is not currently checked in the CI).<br></div></div></div></blockquote><div><br></div><div>Hm, actually, the original commit message [3] discusses this including some additional details such as C17 not being available in Ubuntu 18.04.<br></div><div><br></div><div>[3] <a href="https://github.com/OSGeo/grass/commit/706e7e1aa0927c34cfbcf83388e10192a5bc5043">https://github.com/OSGeo/grass/commit/706e7e1aa0927c34cfbcf83388e10192a5bc5043</a> </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div><div></div><div><br></div><div>[1] <a href="https://gdal.org/development/rfc/rfc68_cplusplus11.html" target="_blank">https://gdal.org/development/rfc/rfc68_cplusplus11.html</a></div><div>[2] <a href="https://github.com/OSGeo/grass/blob/master/.github/workflows/gcc.yml#L11" target="_blank">https://github.com/OSGeo/grass/blob/master/.github/workflows/gcc.yml#L11</a></div></div></div>
</blockquote></div></div>