cmake issue: includes

[Alexander Neundorf]

On Friday 24 March 2006 14:13, David Faure wrote:
> On Friday 24 March 2006 14:02, William A. Hoffman wrote:
> > At 06:05 AM 3/24/2006, Adriaan de Groot wrote:
> > >Building kdebase against yesterday's kdelibs:
> > >
> > >adridg at kde$ gmake
> > >Building CXX object libkonq/CMakeFiles/konq.dir/konq_operations.o
> > >/mnt/src/kde/kde-4.0/kdebase/libkonq/konq_operations.cc:65:18:
> > > X11/Xlib.h: No such file or directory
> > >
> > >2) Why isn't the X11 include path added to $(all_includes) or whatever
> > > cmake calls it? How can I add this in the CMakefiles?
> >
> > It should be something like this:
> >
> > include_directories(${X11_INCLUDE_DIR})
> >
> > Assuming find_package(X11) has been called earlier.
>
> And if you do it in the toplevel CMakeFiles.txt it will affect all of
> kdebase, which is probably what we want. Well, at least in workspace/, but
> libkonq isn't X11-independent yet...

In FindKDE4Internal.cmake there are two variables: 
* KDE4_INCLUDE_DIR (which is /opt/kde4/include) and 
* KDE4_INCLUDE_DIRS, which is KDE4_INCLUDE_DIR and more include dirs, as 
QT_INCLUDES and on UNIX except OS X also X11_INCLUDE_DIR.

So KDE4_INCLUDE_DIRS should be used instead of KDE4_INCLUDE_DIR to get 
everything.

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