Opened 11 years ago

Closed 10 years ago

#2970 closed defect (fixed)

Additional dependency to libgdal.so 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, libgdal.so, dependency
Cc: Daniel Morissette, warmerdam

Description

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 11 years ago.
patch to GNUmakefile in gdal/apps

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by nsavard

Attachment: ticket2970.patch added

patch to GNUmakefile in gdal/apps

comment:1 Changed 11 years ago by warmerdam

Cc: warmerdam added
Owner: changed from warmerdam to chaitanya

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 Changed 10 years ago by Even Rouault

Owner: changed from chaitanya to Even Rouault

comment:3 Changed 10 years ago by Even Rouault

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.