Hi all,

It is great to know the Ingres driver pulled from SVN has been compiled 
successfully.

However it is not flexible enough or in other word, a little improvement should 
be applied:

1. One switch for turn on/off the Ingres definition. Currently by uncomment 
several definitions in nmake.opt is a little complex. Also the header files and 
library files directories must be specified by hand. After Installing the 
Ingres database, the locations of this file are fixed. It would be better to 
set the default path related to a variable such as $(ING_HOME). ING_HOME 
variable could be set to II_SYSTEM previously and let it detected 
automatically. If the ING_HOME is set, then Ingres driver will be involved.

# nmake.opt
---------------------------------------------------------------------
   # INGRES Libraries
   # Uncomment the following to enable Ingres format.
   # INGRES_HOME = $(II_SYSTEM)
   !ifdef INGRES_HOME
   INGRES_INC_DIR = $(INGRES_DIR)\files
   INGRES_LIB = $(INGRES_DIR)\lib\iilibapi.lib \
  $(INGRES_DIR)\lib\iilibutil.lib \
 $(INGRES_DIR)\lib\libingres.lib
   !endif
 
2. Building Ingres driver as a plugin library. It is not necessary but provides 
more flexibility especially for Ingres that needs a number of dependent 
libraries.
The Ingres_Plgin flag is set to check which mode would it be. 
  
  # nmake.opt
  ---------------------------------------------------------------------
  # INGRES Libraries
  # Uncomment the following to enable Ingres format.
  # INGRES_HOME = $(II_SYSTEM)
  # Uncomment the following if you prefer to build Ingres support as a plugin.
  # INGRES_PLUGIN = YES
  !ifdef INGRES_HOME
  INGRES_INC_DIR = $(INGRES_DIR)\files
  INGRES_LIB = $(INGRES_DIR)\lib\iilibapi.lib \
  $(INGRES_DIR)\lib\iilibutil.lib \
  $(INGRES_DIR)\lib\libingres.lib
 !endif


Any idea?

zhen

_______________________________________________
gdal-dev mailing list
gdal-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/gdal-dev

Reply via email to