Re: CVS commit: pkgsrc/geography/gdal-lib

[Greg Troxel <gdt%lexort.com@localhost>]

"Makoto Fujiwara" <mef%netbsd.org@localhost> writes:

> (geography/gdal-lib) Trying to fix on NetBSD/9.3, sorry not intensively considered

Thanks for looking at this.  I had noticed it was broken from bulk
reports and not gotten to it yet.   I updated my desktop (where I run
qgis) to 10 quite a while ago.
>  USE_LANGUAGES+=              c c++
> -USE_CC_FEATURES+=    c99
> -USE_CXX_FEATURES+=   c++11
> +USE_CC_FEATURES+=    c17

Upstream's docs still says c99.   I just poked them about it.  I am
building on 9/amd64 as you committed and will try again without c17.

> +USE_CXX_FEATURES+=   c++17

Upstream's docs says c++17 so this is manifestly correct.

However, gcc.mk says we need gcc7 for c++17, although gcc itself says 8
for some templates.

I wonder if really c++17 needs 8, and the change to c17 made it pick 10.

>  USE_TOOLS+=          gmake
>  USE_TOOLS+=          pkg-config
> @@ -72,6 +72,7 @@ CMAKE_CONFIGURE_ARGS+=      -DNetCDF_DIR=${BU
>  #CMAKE_CONFIGURE_ARGS+=      -Dxerces
>  #CMAKE_CONFIGURE_ARGS+=      -Dxerces-inc=${BUILDLINK_PREFIX.xerces-c}/include
>  #CMAKE_CONFIGURE_ARGS+=      -Dxerces-lib=-L${BUILDLINK_PREFIX.xerces-c}/lib\ -lxerces-c
> +CMAKE_CONFIGURE_ARGS+=       -DACCEPT_MISSING_SQLITE3_RTREE:BOOL=ON

That makes sense; NetBSD's sqlite3 is deficient.  It is interesting that
I was able to build without this (and use geopackages).