tried to emerge workrave-1.8.0 with kde support. ./configure failed to find kde headers. Reproducible: Always Steps to Reproduce: 1. enable kde flag for workrave (my flags are : -dbus -debug -distribution +gnome +kde +nls +xml2, but same result with only kde flag enabled) 2. unmask workrave-1.8 ~x86 3. emerge workrave Actual Results: configure failed : checking for KDE... configure: error: in the prefix, you've chosen, are no KDE headers installed. This will fail. So, check this please and use another prefix! Expected Results: workrave installed with kde support (kicker applet etc.) workaround : after disabling kde flag workrave builds and installs fine. only kde applet not installed, but does not prevent app from working as intended.
Thanks for your report. I made a mistake including that USE flag without testing it. I removed it for now until someone provides a patch that enables KDE support correctly. I'll have to rely on other people's help in this regard since I can't test things with qt/kde.
I'll add the proper support for the 'kde' flag if someone gives me a hand with it. Resolving as WONTFIX for now.
Created attachment 65503 [details, diff] patch to get kde-applet working Hi, I have been able to compile it with kde-applet. I attach the patch I made to the ebuild. There is still one problem with it, when you emerge workrave you will get the error: open_wr: /usr/qt/3/etc/settings/.qtrc.lock see also http://forums.gentoo.org/viewtopic.php?t=18733 I solved this by removing the /usr/qt/3/etc directory
Thanks for the contribution, I'll look into your patch in a moment.
Created attachment 65506 [details, diff] workrave-1.8.1.ebuild.patch The patch looks fine. A few observations: * Dependencies should be modified accordingly. * Running external commands such as 'kde-config' from the global scope is a bad idea, so I'd move that line to pkg_setup(). * I'm not sure about this, but setting --prefix to the output from `kde-config --prefix` might affect the installation targets for other files that should rely on --prefix=/usr. I suspect it's more appropriate to use --with-qd-dir instead. I'm attaching the resulting patch. Please test it and let me know how it goes. I also have a couple of things I'd like to ask you about before committing a new revision: * What package provides the kde-config binary? It should be included in DEPEND as well. * Does the sandbox issue persist? If it's a real problem I'd like to find the way to fix it so people don't have to use work-arounds such as removing directories before running emerge. Thank you.
I have tried your patch, but if you use '--with-qt-dir' then it fails to find the necessary qt-binaries. I have also tried --with-extra-libs but that doesn't work either. Also the error especially says that in the prefix no KDE errors are installed: > checking for KDE... configure: error: > in the prefix, you've chosen, are no KDE headers installed. This will fail. > So, check this please and use another prefix! So I suppose that --prefix is what we should have. Further: kde-config is in kde-base/kdelibs. The sandbox problem is still there, I don't know what to do about it, other than throwing away the /usr/qt/3/etc dir. I have looked into the eclass kde.eclass, as was suggested in the forum, but I couldn't create a working ebuild with that information.
Created attachment 65587 [details, diff] workrave-1.8.1.ebuild.patch Thank you for the information. I've finally had the chance to do a little testing with KDE packages and I prepared a new ebuild which I think resolves all the remaining problems. Please test it and let me know any problems/suggestions. Thanks.
Yes, it works now. Thanks.
Thank you. Committed now as workrave-1.8.1-r1.
