<p dir="ltr">Yeah don't change anything like this in a LTR. It's not a good idea for trust.</p>
<br><div class="gmail_quote"><div dir="ltr">On Tue, 25 Apr 2017, 5:24 PM Matthias Kuhn <<a href="mailto:matthias@opengis.ch">matthias@opengis.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Please don't. Embedded code copies are a PITA. QGIS has too many of those already (although they're generally not required).<br>
<br>
I agree in general. I think this case here is a bit different, given<br>
that this is already partially present (in a particularly cumbersome and<br>
incomplete way) and is going to be phased out soon.<br>
<br>
>>> 5) require GDAL < 2.2 for building QGIS 2.18<br>
> What about<br>
><br>
> 6) #ifdef the offending plugin out in 2.18 for GDAL >= 2.2?<br>
<br>
If that's going to be done I would update that to error out in CMake<br>
with an incompatiblity error and force the packager to make a decision<br>
(actively disable dxf2shp or pin GDAL to the older version).<br>
<br>
Which decision would you propose for the packages provided upstream<br>
(like OSGeo4W)?<br>
<br>
><br>
> Given that the plugin is removed in QGIS 3.0 anyway, I think early<br>
> retirement is a perfectly valid option. It's also the best option for<br>
> GDAL, and avoids any wasted effort temporarily fixing code which only<br>
> has a short time left to live anyway...<br>
<br>
I was thinking the same, but breaking peoples workflows in patch<br>
releases is a good way to undermine the trust in LTR.<br>
<br>
Matthias<br>
<br>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>