<div dir="ltr"><div>Given that the original motivation is unknown and that we would not do the same now, i.e., we would treat GDAL as any other library, it seems we can go ahead.</div><div><br></div><div>The only reason I can think of is GRASS driver for GDAL which introduces a cyclic dependency, but if anything is needed, it is perhaps solved in GDAL.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 7 Apr 2022 at 11:10, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com" target="_blank">wenzeslaus@gmail.com</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"><div dir="ltr"><div>Hi devs,<br></div><div><br></div><div></div><div>We are about to remove dynamic loading of GDAL from the raster library code in #2290. GDAL will then be loaded as any other library by the system.<br></div><div><br></div><div><a href="https://github.com/OSGeo/grass/pull/2290" rel="noreferrer" target="_blank">https://github.com/OSGeo/grass/pull/2290</a></div><div><br></div><div>Currently, the libraries are loaded with <span><span>dlopen and <span><span>LoadLibrary and</span></span></span></span> <span><span>GDAL_DYNAMIC</span> is set to enable that code. This was introduced in </span>2008 by Glynn in r33559.<br></div><div><br></div><div><a href="https://github.com/OSGeo/grass/commit/265039761908433c58b07b9b47fcb16f5126e88c" target="_blank">https://github.com/OSGeo/grass/commit/265039761908433c58b07b9b47fcb16f5126e88c</a></div><div></div><div><a href="https://trac.osgeo.org/grass/changeset/33559" target="_blank">https://trac.osgeo.org/grass/changeset/33559</a></div><div><br></div><div>To be sure removing it completely is a good step, I wanted to check the original motivation. Any ideas? The commit message doesn't mention the motivation and I'm not getting anything from Internet searches.</div><div><br></div><div>Let me know what you think.</div><div><br></div><div>Best,<br></div><div>Vaclav<br></div></div>
</blockquote></div>