Opened 19 years ago

Closed 13 years ago

#652 closed defect (wontfix)

configure.in does not detect the absence of libgif when libungif is present

Reported by: dirk@… Owned by: warmerdam
Priority: high Milestone:
Component: default Version: unspecified
Severity: minor Keywords:
Cc:

Description (last modified by Even Rouault)

configure.in does not detect the absence of libgif, although it detects the
presence of libungif.

Patch will be supplied soon.

Attachments (1)

gdal-diff.txt (14.8 KB ) - added by dirk@… 19 years ago.
Patch to fix LDFLAGS-issues, the ungif-thing and bug 670

Download all attachments as: .zip

Change History (7)

comment:1 by warmerdam, 19 years ago

Dirk,

A patch would be appreciated. 


by dirk@…, 19 years ago

Attachment: gdal-diff.txt added

Patch to fix LDFLAGS-issues, the ungif-thing and bug 670

comment:2 by warmerdam, 19 years ago

Also, why the inclusion of:
+
+/*
+#include <bits/c++config.h>
+#include <complex>
+#include <cmath>

I also hate reversing the order of all the LIBS directives.  What I had
might not have been ideal, but it is very hard for me to predict what problems
reversing everything might cause. 

comment:3 by warmerdam, 19 years ago

Doh, I lost part of my last comment.  I started out saying that the danger
of supplying such an all encompassing patch is that it scares the pants off
me, and I am unlikely to apply it.  The first issue I raised was why
you have put in this tiff library change, but I now gather it is an effort
to work around the very long commandline, is that right?  

I am very hesitant to add this special case for the libtiff code.  It really
seems like the solution is for you to up your commandline length, or to
build some libraries (like libtiff) externally. 


comment:4 by dirk@…, 19 years ago

About the long-command-line-thing:
You're right. I should have broken up in two patches, I know. Sorry.
So one part is for the long command line and the other one is for libungif.

About the reversing: I did change the order to achieve that an externally given
LDFLAGS in honored first. And because I've seen it not too often this way. I can
let the order as it is as long as a user can supply the LDFLAGS externally.

There is a bug in the patch anyways. GNUmakefile should start with:
      7 GDAL_OBJ        =       $(GDAL_ROOT)/frmts/o/*.o \
      8                         $(GDAL_ROOT)/gcore/*.o \
      9                         $(GDAL_ROOT)/port/*.o \
     10                         $(GDAL_ROOT)/alg/*.o
     11 
     12 ifeq ($(TIFF_SETTING),internal)
     13 GDAL_OBJ += $(GDAL_ROOT)/frmts/o/gdal-tiff.a
     14 endif
Otherwise builds without internal libtiff are broken. Sorry.

comment:5 by dirk@…, 19 years ago

The reason for the inclusion of commented out lines is probably because working
too late in the night. Sorry for that as well.

Regarding the command line length: It looks like a limitation of the IRIX linker.

comment:6 by Even Rouault, 13 years ago

Description: modified (diff)
Resolution: wontfix
Status: assignedclosed

The libungif project has no interest now that the LZW patent has expired. As far as I know, all Linux distro now ship libgif. Closing

Note: See TracTickets for help on using tickets.