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