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: | Owned by: | warmerdam | |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | default | Version: | unspecified |
Severity: | minor | Keywords: | |
Cc: |
Description (last modified by )
configure.in does not detect the absence of libgif, although it detects the presence of libungif. Patch will be supplied soon.
Attachments (1)
Change History (7)
by , 19 years ago
Attachment: | gdal-diff.txt added |
---|
Patch to fix LDFLAGS-issues, the ungif-thing and bug 670
comment:2 by , 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 , 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 , 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 , 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 , 13 years ago
Description: | modified (diff) |
---|---|
Resolution: | → wontfix |
Status: | assigned → closed |
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.