KDE_EXPORT in kdelibs and winposix

Brad King brad.king at kitware.com
Thu Feb 9 22:15:12 CET 2006


Brad King wrote:
> Ralf Habacker wrote:
>>Brad King schrieb:
>>>I suggest that the winposix_export.h versions be renamed to 
>>>WINPOSX_EXPORT and WINPOSX_IMPORT
>>I agree
> Thanks for making the changes, but there is one problem.  Now nothing 
> defines KDE_IMPORT.  I think it should be defined in kdemacros.h next to 
> KDE_EXPORT.

There is also an inconsistency in the definition of KICE_EXPORT:

// dcop/KDE-ICE/KICE.h
#ifndef KICE_EXPORT
# ifdef MAKE_DCOP_LIB
# define KICE_EXPORT KDE_EXPORT
# else
# define KICE_EXPORT KDE_IMPORT
# endif
#endif

// kdelibs_export_win.h
#ifndef KICE_EXPORT
# ifdef MAKE_KICE_LIB
#  define KICE_EXPORT KDE_EXPORT
# else
#  define KICE_EXPORT KDE_IMPORT
# endif
#endif

The patch below works-around the problem the same way bksys does but I 
think the KICE.h version should be moved to kdelibs_export_win.h (and 
later kdelibs_export.h).

-Brad


Index: dcop/CMakeLists.txt
===================================================================
--- dcop/CMakeLists.txt (revision 507708)
+++ dcop/CMakeLists.txt (working copy)
@@ -87,7 +87,7 @@
     set(LINK_WITH_kICE kICE)
  else(UNIX)
     add_definitions(-DICE_t -DTRANS_SERVER -DTRANS_CLIENT -DANSICPP)
-   add_definitions(-DSOCKCONN -DTCPCONN)
+   add_definitions(-DSOCKCONN -DTCPCONN -DKICE_EXPORT=DCOP_EXPORT)
     set(DCOP_LIB_SRCS ${DCOP_LIB_SRCS} ${kICE_STAT_SRCS})
     set(LINK_WITH_kICE )
  endif(UNIX)


More information about the Kde-buildsystem mailing list