On Fri, Aug 16, 2024 at 4:57 PM Greg Troxel <g...@lexort.com> wrote: > > Builds on NetBSD 10 amd64, and with it installed, passes self tests. > > I believe tests failing before installation is a longstanding issue, and > I believe you might think it's a pkgsrc environment issue. So If you > think that you can have 3.12.2 or whatever installed, and build and test > 3.13.0 pre-installation, that's fine. If that doesn't work, then I > think it's a bug. > > I rebuilt librttopo, libspatialite, gdal-lib, postgis (itself at > 3.5.0alpha2 on my system, but at 3.4.2 in pkgsrc), and then qgis > (3.34.9). I started qgis and it seemed ok displayingusing layers from > gpkg and postgis. Not claiming this is the right test, but I needed to > rebuild anyway. > > > Here is an annotated list of the changes in installed files. There are > some things not explained in NEWS. I don't mean to be super pedantic > about NEWS; it's just that without a NEWS clue I can't sort out "this > should have been listed in NEWS" vs "this shouldn't have been installed". > > > Index: PLIST > =================================================================== > diff -u -p -U 1 -r1.27 PLIST > --- PLIST 5 Jun 2024 22:33:56 -0000 1.27 > +++ PLIST 16 Aug 2024 23:15:42 -0000 > @@ -10,2 +10,3 @@ include/geos/algorithm/CentralEndpointIn > include/geos/algorithm/Centroid.h > +include/geos/algorithm/CircularArcs.h > include/geos/algorithm/ConvexHull.h > > Seems part of "curved geometry types", but not listed.
I mean, it's part of curve support, and that's in News, where are you thinking it could be listed? > @@ -33,2 +34,4 @@ include/geos/algorithm/RobustDeterminant > include/geos/algorithm/SimplePointInRing.h > +include/geos/algorithm/construct/IndexedDistanceToPoint.h > +include/geos/algorithm/construct/IndexedPointInPolygonsLocator.h > include/geos/algorithm/construct/LargestEmptyCircle.h > > Can't find this in the "new" section of NEWS. Comes from: Add LargestEmptyCircle handling of polygon obstacles (#939) > @@ -81,2 +89,3 @@ include/geos/geom/GeometryFactory.h > include/geos/geom/GeometryFilter.h > +include/geos/geom/GeometryTypeName.h > include/geos/geom/HeuristicOverlay.h > > Can't find this in the "new" section of NEWS. From WKBReader: Support curved geometry types (#1106) > @@ -120,2 +134,3 @@ include/geos/geom/util/GeometryExtracter > include/geos/geom/util/GeometryFixer.h > +include/geos/geom/util/GeometryLister.h > include/geos/geom/util/GeometryMapper.h > > Can't find this in the "new" section of NEWS. >From RelateNG > @@ -126,2 +141,3 @@ include/geos/geom/util/PointExtracter.h > include/geos/geom/util/PolygonExtracter.h > +include/geos/geom/util/PolygonalExtracter.h > include/geos/geom/util/ShortCircuitedGeometryVisitor.h From Add LargestEmptyCircle handling of polygon obstacles (#939)