Opened 13 years ago

Closed 13 years ago

#2970 closed defect (fixed)

Additional dependency to added during linking in gdal/apps

Reported by: nsavard Owned by: Even Rouault
Priority: normal Milestone: 1.6.2
Component: GDAL_Raster Version: 1.6.1
Severity: normal Keywords: apps, GNUmakefile,, dependency
Cc: Daniel Morissette, warmerdam


There is an additional dependency added during the link process in the gdal/apps directory. The $ variable is expanded to gdalinfo.$(OBJ_EXT) $(DEP_LIBS) instead of gdalinfo.$(OBJ_EXT) only. The $(DEP_LIBS) adds the $(GDAL_ROOT)/libgdal.$(SO_EXT) dependency.

I guess it was a typo and the $ variable should be $<. I added a patch for this.

Attachments (1)

ticket2970.patch (4.4 KB ) - added by nsavard 13 years ago.
patch to GNUmakefile in gdal/apps

Download all attachments as: .zip

Change History (4)

by nsavard, 13 years ago

Attachment: ticket2970.patch added

patch to GNUmakefile in gdal/apps

comment:1 by warmerdam, 13 years ago

Cc: warmerdam added
Owner: changed from warmerdam to chaitanya


Please review the makefile docs and test the change. Norman's analysis seems plausible to me.

If it is appropriate to apply, it should likely also go into 1.6.

comment:2 by Even Rouault, 13 years ago

Owner: changed from chaitanya to Even Rouault

comment:3 by Even Rouault, 13 years ago

Milestone: 1.6.2
Resolution: fixed
Status: newclosed
Version: unspecified1.6.1

Applied in trunk in r17151 and in branches/1.6 in r17152.

This was due to the changes done for #1878, so it affected 1.6.1 and 1.7.0dev

Note: See TracTickets for help on using tickets.