KDE/kdelibs/cmake/modules

[Thiago Macieira]

On Tuesday 20 May 2008 16:59:40 Allen Winter wrote:
> be extra careful about libICE and libSM prerequisites since many
> distros have separate development packages for these.

> --- trunk/KDE/kdelibs/cmake/modules/FindX11.cmake #810371:810372
> @@ -295,6 +295,14 @@
>       SET(X11_SM_FOUND TRUE)
>    ENDIF(X11_SM_LIB AND X11_SM_INCLUDE_PATH)
>
> +  IF(NOT X11_ICE_FOUND)
> +    MESSAGE(FATAL_ERROR "\nThe X11 Inter-Client Exchange (ICE) Library
> could not be found.\nCannot continue. Please install libICE.\n") + 
> ENDIF(NOT X11_ICE_FOUND)
> +
> +  IF(NOT X11_SM_FOUND)
> +    MESSAGE(FATAL_ERROR "\nThe X11 Session Management (SM) Library could
> not be found.\nCannot contineu. Please install libSM.\n") +  ENDIF(NOT
> X11_SM_FOUND)
> +
>    # Deprecated variable for backwards compatibility with CMake 1.4
>    IF (X11_X11_INCLUDE_PATH AND X11_LIBRARIES)
>      SET(X11_FOUND 1)

I don't think FindX11.cmake is the place to put this. It should be in the KDE 
code that requires X11 in the first place. In fact, it should be in the code 
that requires ICE and/or SM, since other parts of KDE may be just fine 
compiling with X11, but not those two.

PS: "continue" is misspelt in the second message

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kde-buildsystem/attachments/20080520/7b21a7ea/attachment.pgp