I am working on recipes for KDE 4.1.80 (the KDE 4.2 beta 1 release [1]).
For some reason I found that I could not apply the patch [2] for 4.1.3
to the new release. However I still think the new release needs to
be patched in a similar way. I would like to have a better understanding
of the current patch so i can re-implement it.

The KDE build system assumes that KDE-Libs and KDE-PIM-Libs
will be installed under the same hierarchy. This is probably the case  
under debian
but not on a gobolinux system.

I believe this is what the following part of the patch (along with  
similar parts) addresses:

-  find_library(KDE4_SYNDICATION_LIBRARY NAMES syndication PATHS $ 
{KDE4_LIB_DIR} \
NO_DEFAULT_PATH )
+  find_library(KDE4_SYNDICATION_LIBRARY NAMES syndication PATHS $ 
{LIB_INSTALL_DIR} \

However, I don't understand why the following has been changed:

  if (WIN32)
-  if (KDEPIMLIBS_DIR)
-      set (KDE4_INCLUDE_DIR ${KDE4_INCLUDE_DIR} ${KDEPIMLIBS_DIR}/ 
include)
-      set (KDE4_LIB_DIR ${KDE4_LIB_DIR}  ${KDEPIMLIBS_DIR}/lib)
-  else (KDEPIMLIBS_DIR)
+  if (LIB_INSTALL_DIR)
+      set (KDE4_INCLUDE_DIR ${KDE4_INCLUDE_DIR} ${LIB_INSTALL_DIR}/ 
include)
+      set (LIB_INSTALL_DIR ${KDE4_LIB_DIR}  ${KDEPIMLIBS_DIR}/lib)
+  else (LIB_INSTALL_DIR)
        file(TO_CMAKE_PATH "$ENV{PROGRAMFILES}" _progFiles)
        set (KDE4_INCLUDE_DIR ${KDE4_INCLUDE_DIR} ${_progFiles}/ 
kdepimlibs/include)
-      set (KDE4_LIB_DIR ${KDE4_LIB_DIR} ${_progFiles}/kdepimlibs/lib)
-  endif (KDEPIMLIBS_DIR)
+      set (LIB_INSTALL_DIR ${KDE4_LIB_DIR} ${_progFiles}/kdepimlibs/ 
lib)
+  endif (LIB_INSTALL_DIR)
  endif (WIN32)

Shouldn't the body of this if statement [ if (WIN32) ]  be ignored on  
gobolinux (or any variant of linux
for that matter)? Or am I missing something?

Thanks,

Frank

[1] http://kde.org/info/4.1.80.php
[2] 
http://recipes.gobolinux.org/r/?list=KDE-Libs&ver=4.1.3-r1&file=01-FindKdepimLibs.patch
_______________________________________________
gobolinux-devel mailing list
gobolinux-devel@lists.gobolinux.org
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel

Reply via email to