[gdal-dev] Use Kakadu "apps library" (#4116)

[Even Rouault]

On jeudi 6 juillet 2017 10:37:32 CEST Even Rouault wrote:
> Hi Philippe,
> 
> > A few days ago, I have updated a rather old ticket ( #4116
> > <https://trac.osgeo.org/gdal/ticket/4116>  , 6 years old).
> > I was just wondering if this update had been noticed, since I didn't want
> > to open a new ticket.
> 
> Not sure I noticed it, but I see
> https://trac.osgeo.org/gdal/ticket/6940 and the attached patch. It looks
> like #6940 is the cumulative patch for #4116 + building as a plugin, right
> ?
> 
> It seems your patch might break people using older Kakadu versions ?
> Do you know what is the minimum one for which your patch can work ?
> At least for documentation purposes (in which case, could you update the
> nmake.opt line "# Uncomment and adjust paths if you have Kakadu 6.0 or newer
> with the appropriate version)
> 
> Are people using Kakadu on Windows OK with that change ? If so, I'll
> (blindly, because of the Windows + Kakadu combo) apply the patch of #6940
> in trunk.
> 
> Ah, actually one thing to fix in
> https://trac.osgeo.org/gdal/attachment/ticket/6940/gdal_kakadu_as_a_dynamic_
> plugin.patch is to comment with # the KAK related lines. Otherwise this will
> break compilation for default builds not using Kakadu. Would you mind
> updating it ?

I've committed the updated version of the patch (which keeps a way of building the driver 
with the old way for older Kakadu)

Even

-- 
Spatialys - Geospatial professional services
http://www.spatialys.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170706/e4fd80c3/attachment.html>