GDAL failed to link when compiled with OGDI support on Windows

[emmanueld]

Target platforms: Windows 32 & 64 bits
Cygwin: CYGWIN_NT-5.2-WOW64, version 1.7.1
MS Visual C++:

• (32 bits) 15.00.30729.01 for 80x86 (Visual C++ .Net 2008 Express SP1)
  
• (64 bits) 15.00.30729.01 for x64 (Visual C++ .Net 2008 Express SP1)
  

OGDI version: 3.1.5
GDAL version: 1.7.1

When OGDI support is active, the GDAL dll fail to link. It seems it tries to link against zlib_ogdi31.dll (which lacks inflateCopy) instead its own internal zlib. To correct the problem, I had to

1) modify nmake.opt : remove zlib_ogdi$(OGDIVER).lib from the OGDILIB variable.

Current code:

#OGDILIB =	$(OGDIDIR)\lib\$(TARGET)\ogdi$(OGDIVER).lib \
#		$(OGDIDIR)\lib\$(TARGET)\zlib_ogdi$(OGDIVER).lib

Proposed code:

#OGDILIB =	$(OGDIDIR)\lib\$(TARGET)\ogdi$(OGDIVER).lib

2) modify frmts/makefile.vc :

Current code:

!IFDEF OGDIDIR
EXTRAFLAGS	=	$(EXTRAFLAGS) -DFRMT_ogdi
!ELSE
EXTRAFLAGS	=	$(EXTRAFLAGS) -DFRMT_zlib
!ENDIF

Proposed code:

!IFDEF OGDIDIR
EXTRAFLAGS	=	$(EXTRAFLAGS) -DFRMT_ogdi
!ENDIF
EXTRAFLAGS	=	$(EXTRAFLAGS) -DFRMT_zlib

(To make sure that FRMT_zlib is always compiled in GDAL).

These changes will not disallow OGDI to use itw own version of zlib.

[Jukka Rahkonen]

Heads-up: Suggested edits have not been done. No bad words about them either.

[Even Rouault]

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to ​https://github.com/OSGeo/gdal/issues if it is not already reported there.