KDE/kdegraphics/okular/generators/ooo

[Manuel "Sput" Nickschas]

On Sunday 16 November 2008 22:35:28 Alexander Neundorf wrote:

> > reavertm sent me a patch [1], with the following comments:
> >
> > <reavertm> this is only 1st problem
> > <reavertm> the second is ... list expansion - QCA2_LIBRARY_DIRS is list -
> > not single entry so it don't work when ytou just add it to find_library
> > as aegument
>
> I think he's wrong here.
> If it looks like "/usr/lib;/usr/local/lib" then it should be perfectly fine
> to add it using a single variable to the find_library() call.
>
> And I think prefixing the variables coming from pkgconfig() with PC_ is
> something we should do, so we don't need code like that:
> set (QCA2_LIBRARIES QCA_LIBRARIES-NOTFOUND)
> and it becomes more obvious where the stuff comes from.

Well, whatever you say, you're the expert :) I'm not very familiar with 
pkgconfig and the peculiarities of KDE's buildsystem.

But in any case, this should be fixed rather soon, since some important 
packages (okular, kopete) have been failing for several days now. If you need 
somebody to test something, I'd be glad to do it - 

Based on your suggestion, I've created and attached a patch. With that, okular 
builds fine on my system with QCA support.

If it's ok that way, I can commit it to the repo.

Greetings,
~ Sput
-- 
Manuel "Sputnick" Nickschas ("Sput" on Freenode)                  |  (o<
Member of the Quassel IRC Project - http://quassel-irc.org        |  //\
Come visit us in #quassel!                                        |  V_/_
-------------- next part --------------
A non-text attachment was scrubbed...
Name: FindQCA2.cmake.patch
Type: text/x-patch
Size: 1219 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kde-buildsystem/attachments/20081117/58c8bc30/attachment.patch