Review Request: Fix FindXine.cmake to use pkg-config instead of xine-config

Rolf Eike Beer kde at opensource.sf-tec.de
Thu Jan 5 13:15:12 GMT 2012


>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103631/
> -----------------------------------------------------------
>
> Review request for kdelibs.

If it is a CMake-related issue it's always a wise idea to explicitely add
Alex Neundorf as he is by default guilty of all CMake bugs ;) Alex,
explicitely added you to CC now. Are you on k-c-d?

> Description
> -------
>
> This is a gentoo downstream patch, see bug
> https://bugs.gentoo.org/show_bug.cgi?id=397595 for cause.

While this patch may "just work", especially for 1.2, I don't like the way
it is done. The first question is: does the pkg-config approach also work
with 1.1.0 (as that is the minimum required version)?

And if yes: then please use the FindPkgConfig module instead of open
coding such things.

Eike




More information about the kde-core-devel mailing list