[geos-devel] build failure of 3.10.3 on smartos (= illumos)

[Paul Ramsey]

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 <gdt at 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 at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/geos-devel