GeoTools
  1. GeoTools
  2. GEOT-3135

JP2KReader should not depend on Kakadu to get the UUIDBox

    Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.7-M0
    • Fix Version/s: 2.7.6
    • Component/s: jp2k plugin
    • Labels:
      None

      Description

      The JP2KReader appears to have been enhanced over the past year to work somewhat without the Kakadu native libraries. However, when the code tries to get the GeoTIFF metadata, from the JP2, it won't even try if its not dealing with a Kakadu based image reader. That requirement isn't necessary since Sun specified with the metadata tree looks like for JP2: http://download.java.net/media/jai-imageio/javadoc/1.1/com/sun/media/imageio/plugins/jpeg2000/package-summary.html I am attaching a patch here that fixes this. It works correctly in my environment. I don't have a JP2 with GeoTiff headers I can share for testing and so there is no test.

      My employer has relinquished copyright claims and I submitted the required form to Tyler Mitchell at OSGeo.

      I would really appreciate this getting into v2.7.

      1. jp2k_without_kakadu.patch
        9 kB
        David Smiley
      2. jp2k_without_kakadu.patch
        8 kB
        David Smiley

        Activity

        Hide
        David Smiley added a comment -
        I am attaching an updated patch so that it will apply cleanly on trunk.

        Note that JP2KReader.java should be run through an auto-indenter since there are plenty of of lines using tabs and even tabs plus spaces inconsistently. I didn't do the auto-indenting because I want it to be clear what my patch is doing.

        I'm looking forward to this getting committed so I need not use patched geotools.
        Show
        David Smiley added a comment - I am attaching an updated patch so that it will apply cleanly on trunk. Note that JP2KReader.java should be run through an auto-indenter since there are plenty of of lines using tabs and even tabs plus spaces inconsistently. I didn't do the auto-indenting because I want it to be clear what my patch is doing. I'm looking forward to this getting committed so I need not use patched geotools.

          People

          • Assignee:
            Daniele Romagnoli
            Reporter:
            David Smiley
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: