Ticket #1920 (closed defect: fixed)

Opened 9 months ago

Last modified 8 months ago

GDAL 1.4 and libtiff4 incompatible

Reported by: warmerdam Assigned to: warmerdam
Priority: normal Milestone: 1.4.3
Component: GDAL_Raster Version: 1.4.2
Severity: normal Keywords: gtiff bigtiff
Cc:

Description

It *seems* like gdal from the 1.4 branch will build against libtiff4 (aka the BigTIFF release), but results are subtly wrong. For instance gdalchksum.py results on the file gdalautotest/gcore/data/sstgeo.tif are wrong.

If this can be easily fixed, then that would be good, otherwise we should ensure that gdal 1.4.3 will explicitly fail to build with libtiff4 "explaining" why.

Attachments

gdal_svn_branch1.4_bug1920.patch (4.8 kB) - added by rouault on 10/22/07 04:56:09.

Change History

10/22/07 04:55:14 changed by rouault

This issue is due to variables of type 'uint32' used in place of 'toff_t' in a few places in geotiff.cpp. The attached patch gives the same checksums for 1.4 branch and trunk. (it's more or less r11744 and r12221 applied to 1.4 branch)

10/22/07 04:56:09 changed by rouault

  • attachment gdal_svn_branch1.4_bug1920.patch added.

10/22/07 11:15:00 changed by warmerdam

Even,

Unfortunately this just papers over subtler incompatibility between GDAL 1.4.x and libtiff4. I believe the solution is to cause the GDAL configure and/or GDAL build to error out if given libtiff4.

10/22/07 13:10:51 changed by rouault

I've commited in branch 1.4 in r12500 a test in configure.in that prevents building GDAL with libtiff >= 4.0. (the test is taken from configure.in from trunk) Please regenerate configure.

10/24/07 20:33:54 changed by warmerdam

  • status changed from new to closed.
  • resolution set to fixed.

I have confirmed that the patch seems to operate as I would hope. Closing.

(and committing updated configure)

10/27/07 23:40:12 changed by warmerdam

Added compile time check in frmts/gtiff/geotiff.cpp (r12564) to confirm we aren't building with libtiff4. This is in case someone uses libtiff4 on windows with GDAL (not caught by configure).