OSGeo / grass

GRASS GIS - free and open-source geospatial processing engine
https://grass.osgeo.org
Other
854 stars 311 forks source link

r.in.pdal: replace HAVE_PDAL_NOFILENAMEWRITER configure definition #4783

Closed marisn closed 2 days ago

marisn commented 3 days ago

with a simple version check in the code to speed up configure process

As discussed in the https://github.com/OSGeo/grass/pull/4750

echoix commented 3 days ago

I'd prefer someone to confirm the approach, but I won't block and help getting it through if no one jumps in

marisn commented 2 days ago

I realized the check will fail with PDAL 3.0, but as a major version increment will force us to review the code anyway, that's a future developer problem :D