Opened 15 years ago

Closed 15 years ago

#15 closed defect (fixed)

New build system does not install epsg_csv, misses CSV_DATA_DIR macro.

Reported by: warmerdam Owned by: warmerdam
Priority: normal Milestone:
Component: libgeotiff Version:
Keywords: Cc: hobu

Description

After the upgrades to automake/etc the libgeotiff build no longer includes the CSV_DATA_DIR macro used in cpl_csv.c to find the installed csv files.

The install also no longer installs the .csv files in $(datadir)/epsg_csv as expected. It apparently does not install them at all.

Change History (3)

comment:1 by warmerdam, 15 years ago

I believe the following change is enough to fix the CSV_DATA_DIR macro problem.

Index: Makefile.am
===================================================================
--- Makefile.am (revision 1595)
+++ Makefile.am (working copy)
@@ -12,7 +12,8 @@
 TIFF_CFLAGS = @TIFF_INC@ -DHAVE_TIFF=1
 endif
 
-AM_CFLAGS = -I./libxtiff $(PROJ_CFLAGS) $(TIFF_CFLAGS)
+AM_CFLAGS = -I./libxtiff $(PROJ_CFLAGS) $(TIFF_CFLAGS) \
+       -DCSV_DATA_DIR=\"$(datadir)/epsg_csv\"
 
 include_HEADERS =   geotiff.h \
                     geotiffio.h \

comment:2 by hobu, 15 years ago

r1597 is an attempt to fix this, including the patch above with configuration to actually install the csv files.

Please confirm.

comment:3 by warmerdam, 15 years ago

Resolution: fixed
Status: newclosed

This changes appears to fix things up, thanks.

Note: See TracTickets for help on using tickets.