[geos-devel] Fedora 38 GCC 13 3.11.2 build failure

[Paul Ramsey]

Put up a PR with the changes needed to support Gcc13 in older branches, I’ll merge it in. I just don’t have easy access to gcc13 to do the build/test cycle with.

P

> On Apr 25, 2023, at 5:48 AM, Roger Bivand <Roger.Bivand at nhh.no> wrote:
> 
> Having just  upgraded to F38, which comes with GCC 13, I deleted my unpacked tarball, uncompressed 3.11.2, and tried to build. It stopped at 
> [ 51%] Building CXX object CMakeFiles/geos.dir/src/shape/fractal/HilbertEncoder.cpp.o
> In file included from /home/rsb/topics/geos/geos-3.11.2/src/shape/fractal/HilbertEncoder.cpp:15:
> /home/rsb/topics/geos/geos-3.11.2/include/geos/shape/fractal/HilbertEncoder.h:40:28: error: expected ‘)’ before ‘p_level’
>   40 |     HilbertEncoder(uint32_t p_level, geom::Envelope& extent);
>      |                   ~        ^~~~~~~~
>      |                            )
> /home/rsb/topics/geos/geos-3.11.2/include/geos/shape/fractal/HilbertEncoder.h:41:5: error: ‘uint32_t’ does not name a type
>   41 |     uint32_t encode(const geom::Envelope* env);
>      |     ^~~~~~~~
> /home/rsb/topics/geos/geos-3.11.2/include/geos/shape/fractal/HilbertEncoder.h:21:1: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
>   20 | #include <vector>
>  +++ |+#include <cstdint>
>   21 | 
> 
> The HEAD on github builds OK, and ctest runs cleanly; it has the #include <cstdint> declaration. Would this indicate that Fedora 38 users should build from github rather than 3.11.2, since I doubt that GCC 13 fixes are a general requirement?
> 
> Sorry for the distraction,
> 
> Roger
> 
> 
> --
> Roger Bivand
> Emeritus Professor
> Norwegian School of Economics
> Postboks 3490 Ytre Sandviken, 5045 Bergen, Norway
> Roger.Bivand at nhh.no
> _______________________________________________
> geos-devel mailing list
> geos-devel at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/geos-devel