<div dir="ltr">+1 from me as well - simpler code is a great outcome and should be a goal these days. I'd also support SWIG 4 although I wonder how widespread it is - especially on enterprise Linux distros.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jun 4, 2023 at 1:35 PM Seth G <<a href="mailto:sethg@geographika.co.uk">sethg@geographika.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Even,<br>
<br>
+1 from me - as you write removing those multiple PROJ code paths will simplify the code base greatly in some places. <br>
Does GEOS have similar issues? There don't seem to be any version checks in MapServer apart from some comments in the Python MapScript bindings "Requires GEOS support (>=3.0)". <br>
<br>
I'd also like to propose we only support SWIG 4 and higher, and begin removing Python2 specific code (I can look at these two items).<br>
<br>
Seth<br>
<br>
--<br>
web:<a href="https://geographika.net" rel="noreferrer" target="_blank">https://geographika.net</a> & <a href="https://mapserverstudio.net" rel="noreferrer" target="_blank">https://mapserverstudio.net</a><br>
twitter: @geographika<br>
<br>
On Sat, Jun 3, 2023, at 5:15 PM, Even Rouault wrote:<br>
> Hi,<br>
><br>
> I've prepared 2 pull requests <br>
> <a href="https://github.com/MapServer/MapServer/pull/6900" rel="noreferrer" target="_blank">https://github.com/MapServer/MapServer/pull/6900</a> and <br>
> <a href="https://github.com/MapServer/MapServer/pull/6901" rel="noreferrer" target="_blank">https://github.com/MapServer/MapServer/pull/6901</a> to respectively drop <br>
> support for PROJ < 6 and GDAL < 3 for MapServer 8.2<br>
><br>
> Any opposition to that?<br>
><br>
> Those older versions haven't been supported anymore and for a long time <br>
> by their respective projects, the proposed minimal versions have been <br>
> released 4 years ago and our CI doesn't test the dropped versions <br>
> anymore, so I believe it is safe to proceed.<br>
><br>
> The PROJ < 6 drop removes quite a bunch of code paths so it is a nice <br>
> cleanup. The GDAL < 3 has modest impacts, but anyway having PROJ >= 6 <br>
> and GDAL < 3 doesn't make sense as this is a rather unsupported combination.<br>
><br>
> Even<br>
><br>
> -- <br>
> <a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
> My software is free, but my time generally not.<br>
><br>
> _______________________________________________<br>
> MapServer-dev mailing list<br>
> <a href="mailto:MapServer-dev@lists.osgeo.org" target="_blank">MapServer-dev@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
_______________________________________________<br>
MapServer-dev mailing list<br>
<a href="mailto:MapServer-dev@lists.osgeo.org" target="_blank">MapServer-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
</blockquote></div>