Well, it's super rare, platformwise so I don't really categorize it as "our problem", but it's nice to be clearer. What's the simplest way to even find out if a change fixes this? I'm tempted to make everything std::sqrt() on some kind of principle, but I have no idea if that will actually change anything. Probably just changing the 5 to 5.0 would do the trick.
P > On Jun 21, 2022, at 3:44 AM, Greg Troxel <g...@lexort.com> wrote: > > > pkgsrc is getting ready for quarterly branch, and I noticed geos fails > to build on smartos. > > > https://us-central.manta.mnx.io/pkgsrc/public/reports/upstream-trunk/20220620.2256/geos-3.10.3/build.log > > The issue is sqrt(5) leading to ambiguous overload with long double, > double, float. > > Appparently that is supposed to be ok, and maybe cmath is supposed to be > included, maybe using std::sqrt, maybe neither. And maybe c++11 is > supposed to handle int as an arg to sqrt. > > Opinions as to whether this is a geos issue vs a smartos toolchain issue > vs ? > > > > > _______________________________________________ > geos-devel mailing list > geos-devel@lists.osgeo.org > https://lists.osgeo.org/mailman/listinfo/geos-devel _______________________________________________ geos-devel mailing list geos-devel@lists.osgeo.org https://lists.osgeo.org/mailman/listinfo/geos-devel
