branches/work/kdepim-3.5.5+/kpilot/conduits

[Alexander Neundorf]

Hi Adriaan,

cmake stuff for KDE 3, if I unserstand this correctly ?
Nice :-)

On Wednesday 27 December 2006 15:01, Adriaan de Groot wrote:
> SVN commit 616950 by adridg:
>
> Only try building the stuff that depends on KDE PIM headers if the headers
> are there. I can't seem to find a 'find include' macro that obeys whatever
> include paths are in effect (ie. taking the settings from
> include_directories), 

That's right, that doesn't exist. I guess the idea is to keep system 
introspection and building strictly separated.

> so look in the single KDE3 include dir. This
> precludes installing PIM in a different prefix, though.
>
> (CC -buildsystem for advice in pulling in the right include dir)
>
> CCMAIL: kde-buildsystem at kde.org
>
>
>  M  +8 -1      branches/work/kdepim-3.5.5+/kpilot/conduits/CMakeLists.txt
>
>
> --- branches/work/kdepim-3.5.5+/kpilot/conduits/CMakeLists.txt
> #616949:616950 @@ -13,8 +13,15 @@
>  add_subdirectory(popmail)
>  add_subdirectory(sysinfoconduit)
>  add_subdirectory(timeconduit)
> -add_subdirectory(vcalconduit)
>
> +FIND_PATH( HAVE_CALENDARLOCAL_H "libkcal/calendarlocal.h"
> ${KDE3_INCLUDE_DIR} ) +
> +IF (HAVE_CALENDARLOCAL_H)
> +	add_subdirectory(vcalconduit)
> +ELSE (HAVE_CALENDARLOCAL_H)
> +	MESSAGE(STATUS "No KDE PIM development headers were found.")
> +ENDIF (HAVE_CALENDARLOCAL_H)
> +

The patch looks good IMO.

Bye
Alex
-- 
Work: alexander.neundorf AT jenoptik.com - http://www.jenoptik-los.de
Home: neundorf AT kde.org                - http://www.kde.org
      alex AT neundorf.net               - http://www.neundorf.net