Enable config.h in MSVC about oce

[dbarbier]

Did you try to add -DHAVE_CONFIG_H to compilation flags? If everything works fine, we can add this flag on Win32 too.

[dbarbier]

ping? I would like to remove #ifdef HAVE_CONFIG_H from inc/*.hxx files (but not src/, this is unnecessary) to let them unconditionnally include oce-config.h, it is extremely annoying to still have to set -DHAVE_CONFIG_H when compiling a program using OCE.

[QbProg]

I'll try this this evening !

2011/7/29 dbarbier
[email protected]:

ping?  I would like to remove #ifdef HAVE_CONFIG_H from inc/*.hxx files (but not src/, this is unnecessary) to let them unconditionnally include oce-config.h, it is extremely annoying to still have to set -DHAVE_CONFIG_H when compiling a program using OCE.

Reply to this email directly or view it on GitHub:
#15 (comment)

[dbarbier]

Please test review/unconditionnaly-include-config-file branch.

[QbProg]

it builds on MSVC.

There are still some things to do before closing the bug:

• not all defines are really defined
• no define is defined in the "installed" config.h
• there are still HAVE_CONFIG_H checks in the cpp files
• the FULL installation path is taken in the OCE_CONFIG.h : this is not really good in windows since the binaries must be allowed to be installed on different paths
  (this is used here and there to define casroot).
  This last point is releated to #2

[dbarbier]

I am closing this issue, changes have been committed recently. I guess that there are still issues, but it is better to file specific bug reports.