playground/base/nepomuk-kde/ontologyimportclient

[Alexander Neundorf]

On Friday 21 August 2009, Michael Jansen wrote:
> SVN commit 1014078 by mjansen:
>
> The file is installed by kdebase/runtime/nepomuk/ontologies/CMakeLists.txt
> into ${DBUS_INTERFACES_INSTALL_DIR} .
>
> For us that means KDE4_DBUS_INTERFACE_DIR since we don't support installing
> kdelibs and kdebase into different prefixes. 

This sounds like a bug. If you (i.e. ontologyimportclient) don't support 
kdelibs and kdebase in different directories, then it is broken and will not 
build on such systems (e.g. my system and there are more).
There are two options: either keep searching ENV XDG_DATA_DIRS, or also export 
the kdebase interface dir when installing kdebase/runtime/ (this is e.g. 
already done for kdebase/workspace, there is a 
KDE4WORKSPACE_DBUS_INTERFACES_DIR in KDE4WorkspaceConfig.cmake.in)

Is is maybe a problem if things depend on stuff from kdebase/runtime/ for 
building ?

> No need to search in
> XDG_DATA_DIRS.
>
> @Sebastian: Is there a use case for having to look for the interface
> somewehere else but the place kdebase/runtime put it?
>
> CCMAIL: sebastian at trueg.de
> CCMAIL: kde-buildsystem at kde.org
>
>  M  +1 -2      CMakeLists.txt
>
>
> --- trunk/playground/base/nepomuk-kde/ontologyimportclient/CMakeLists.txt
> #1014077:1014078 @@ -13,8 +13,7 @@
>  find_file(
>    ONTOLOGY_MANAGER_INTERFACE
>    org.kde.nepomuk.OntologyManager.xml
> -  PATH_SUFFIXES "/dbus-1/interfaces" "/"
> -  PATHS ${DBUS_INTERFACES_INSTALL_DIR} ENV XDG_DATA_DIRS
> +  PATHS ${KDE4_DBUS_INTERFACES_DIR}
>  )
>  qt4_add_dbus_interface(ontoimportclient_BIN_SRCS
> ${ONTOLOGY_MANAGER_INTERFACE} ontologyloaderinterface)

Alex