Ticket #2970 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Additional dependency to libgdal.so added during linking in gdal/apps

Reported by: nsavard Owned by: rouault
Priority: normal Milestone: 1.6.2
Component: GDAL_Raster Version: 1.6.1
Severity: normal Keywords: apps,GNUmakefile,libgdal.so,dependency
Cc: dmorissette, 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

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

Change History

Changed 3 years ago by nsavard

patch to GNUmakefile in gdal/apps

Changed 3 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.

Changed 3 years ago by rouault

  • owner changed from chaitanya to rouault

Changed 3 years ago by rouault

  • status changed from new to closed
  • version changed from unspecified to 1.6.1
  • resolution set to fixed
  • milestone set to 1.6.2

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.