Opened 9 years ago

Closed 9 years ago

#5946 closed enhancement (duplicate)

Support Kakadu 7.5 in GDAL 1.11

Reported by: Martin Mikita Owned by: warmerdam
Priority: normal Milestone:
Component: ConfigBuild Version: 1.11.2
Severity: normal Keywords:
Cc: Robert Coup

Description

I created patch that applies commit 27787 from trunk for the latest GDAL 1.11.2 and fixes ticket #5745 compilation on Windows for x64.

Could it be reviewed and included into 1.11 branch and be released in the next 1.11 release?

Attachments (1)

gdal-1.11.2-kakadu.patch (5.9 KB ) - added by Martin Mikita 9 years ago.
Patch for Kakadu 7.5 into GDAL 1.11.2. Fixes Windows x64 compilation

Download all attachments as: .zip

Change History (8)

by Martin Mikita, 9 years ago

Attachment: gdal-1.11.2-kakadu.patch added

Patch for Kakadu 7.5 into GDAL 1.11.2. Fixes Windows x64 compilation

comment:1 by Even Rouault, 9 years ago

Cc: Robert Coup added

comment:2 by Robert Coup, 9 years ago

Looks good - only hesitation I had initially about backporting it was that it'd break existing compiles previously using older (working) versions of Kakadu. Thoughts @rouault?

in reply to:  2 comment:3 by Even Rouault, 9 years ago

Replying to rcoup:

Looks good - only hesitation I had initially about backporting it was that it'd break existing compiles previously using older (working) versions of Kakadu. Thoughts @rouault?

Yes, that was my fear too. JP2KAK builds looks so fragile...

comment:4 by Robert Coup, 9 years ago

Kakadu is licensed on a subscription basis (well, mine certainly is), so AFAIK there's no particular reason for running old versions (eg. v6) since customers will always have access to the latest version. @MikiBOB, is that your understanding as well?

comment:5 by Martin Mikita, 9 years ago

Yes. The only reason why somebody will use old version is, when Kakadu's creator changes new version code and GDAL cannot compile with it. We want to use the latest Kakadu, but now we have to apply this patch so we can compile GDAL with it.

comment:6 by Robert Coup, 9 years ago

@rouault personally I'm okay with it - will be obvious when you compile, doesn't affect any users by default, is easy to resolve, and can be documented in the release notes simply ("If you're using Kakadu < v7.5 you now need to define these macros").

comment:7 by Even Rouault, 9 years ago

Resolution: duplicate
Status: newclosed

Adressed by #6052

Note: See TracTickets for help on using tickets.