94731 2005-06-01 06:00 0000 kdeutils fails compilation if pbbuttonsd >= 0.6.8 is installed 2005-08-04 07:58:20 0000 1 1 1 Unclassified Gentoo Linux KDE unspecified PPC Other RESOLVED FIXED https://bugs.kde.org/show_bug.cgi?id=104517 P2 normal --- 1 josejx@gentoo.org kde@gentoo.org michael.kiermaier@gmx.net ppc@gentoo.org pylon@gentoo.org josejx@gentoo.org 2005-06-01 06:00:07 0000 As I explained in the kde bug above, pbbuttonsd changed its function for starting the ipc. With versions of pbbuttonsd >= 0.6.8, kdeutils fails when compiling powerbook2. The attached patch fixes the problem, but makes kdeutils dependent on pbbuttonsd >= 0.6.8. If you need more details, please let me know. josejx@gentoo.org 2005-06-01 06:01:53 0000 Created an attachment (id=60357) This patch applies cleanly to 3.4.0 and 3.4.1 carlo@gentoo.org 2005-06-01 14:21:24 0000 Moreover I wonder why kdeutils is marked ppc, but the dependency is missing at all. The kde herd is small and x86/amd64 only, we rely on you spotting such minor missing dependencies. When will pbbuttonsd 0.6.8 go stable? carlo@gentoo.org 2005-06-01 14:24:11 0000 s/marked ppc/keyworded ppc josejx@gentoo.org 2005-06-01 21:15:51 0000 It's not a hard dependency. If pbbuttonsd isn't present, it doesn't build the powerbook libraries. At least that's what it looks like from the config files and this would explain the lack of bug reports on it. So, it's perfectly understandable that ppc missed it as a dependency as well. Only portable Apple laptops use this package, and kdeutils builds fine without pbbuttonsd, just without powerbook support. Non-laptop ppc machines should not depend on pbbuttonsd, so we probably need a use flag for it. I don't think that pbbuttonsd-0.6.8 will go stable soon, as we're working on a crash bug with versions 0.6.8 - 0.6.10 and I'm not sure when this will be resolved. I created this bug as an FYI, this is a problem, here is the fix. carlo@gentoo.org 2005-06-02 07:20:52 0000 > It's not a hard dependency. So it needs to become one, optionalised by a use flag. Untracked depedencies are the worst next to boken autodetection code in build scripts, overriding configure flags. Joe, we really need to track _all_ dependencies of _all_ ebuilds in a determinist way, otherwise we'll never see a Portage version with proper reverse dependency tracking. josejx@gentoo.org 2005-06-02 08:55:47 0000 Look, I'm not arguing with you, no need to get all worked up. I don't use KDE. A user pointed out the problem and we're fixing it. I suggested that we use a USE flag in my previous post, I'm *agreeing* with you. carlo@gentoo.org 2005-06-02 17:02:44 0000 O.k., o.k. ;) Is a "powerbook" use flag fine then? Or does it work with iBooks, too and e.g. "apple-laptop" would be more appropriate? for reference: https://bugs.kde.org/show_bug.cgi?id=104517 josejx@gentoo.org 2005-06-02 19:39:33 0000 Well, I'd probably use "pbbuttonsd" as there is still support for pmu (Apple's version of apm) without pbbuttonsd. pbbuttonsd support allows the use of the special buttons, such as volume and sound. The powerbook2 module provides visual feedback to the user when these buttons are pressed in KDE. flameeyes@gentoo.org 2005-06-03 00:47:21 0000 pbbuttonsd should be good, as ibooks users usually use that, too. carlo@gentoo.org 2005-07-03 08:56:07 0000 *** Bug 97802 has been marked as a duplicate of this bug. *** josejx@gentoo.org 2005-07-03 09:51:44 0000 FYI the upstream patch has support for all versions of pbbuttonsd. If we include this patch instead and add the pbbuttonsd useflag we can close this bug now. josejx@gentoo.org 2005-07-03 09:56:52 0000 Created an attachment (id=62553) Updated patch carlo@gentoo.org 2005-07-03 10:37:56 0000 We can. But I don't like to name the use flag "pbbuttonsd". Imho use flags should be as abstract as possible, so they can be reused. We have already too much crappy named and even redundant use flags, making Portage far more complex than necessary. Fear the bloated Gentoo! It won't be the most speedy penguin anymore. :) josejx@gentoo.org 2005-07-03 11:05:05 0000 Well, what do you suggest then? I don't want to make it conditional on ppc since non laptop users don't need pbbuttonsd at all. We can't use powerbook/ibook because both use pbbuttonsd. If there was an existing laptop use flag, I would suggest use ppc && use laptop but since there isn't, it would be just as wasteful to add one. josejx@gentoo.org 2005-07-26 07:50:48 0000 Carsten, would you like me to just apply the patch myself? greg_g@gentoo.org 2005-07-26 09:18:40 0000 At this point we can just wait a couple of days for kde 3.4.2, which includes the patch for pb_monitor.cpp and will be also patched to have a --with-powerbook/--without-powerbook switch. Then you can add there the optional dependency on pbbuttonsd with a USE flag to be decided. greg_g@gentoo.org 2005-07-30 02:56:40 0000 kde 3.4.2 is in portage now, feel free to add an USE flag and a dependency on pbbuttonsd to kde-base/kdeutils and kde-base/kmilo. josejx@gentoo.org 2005-08-04 07:58:20 0000 Fixed in CVS, thanks! 60357 2005-06-01 06:01 0000 This patch applies cleanly to 3.4.0 and 3.4.1 kdeutils-3.4.0-pbbuttonsd-fix.patch text/plain LS0tIGttaWxvL3Bvd2VyYm9vazIvcGJfbW9uaXRvci5jcHAuYmFrCTIwMDUtMDUtMjcgMDk6MjA6 MDguMDAwMDAwMDAwIC0wNDAwCisrKyBrbWlsby9wb3dlcmJvb2syL3BiX21vbml0b3IuY3BwCTIw MDUtMDUtMjcgMDk6MTk6NTcuMDAwMDAwMDAwIC0wNDAwCkBAIC02NSw3ICs2NSw3IEBACiAKIGJv b2wgUG93ZXJCb29rTW9uaXRvcjo6aW5pdCgpCiB7Ci0JcmV0dXJuICggaXBjX2luaXQoIExJQk1P REVfQ0xJRU5ULCAxICkgPT0gMCApOworCXJldHVybiAoIGlwY19pbml0KCJQb3dlckJvb2tNb25p dG9yIiwgTElCTU9ERV9DTElFTlQsIDEgKSA9PSAwICk7CiB9CiAKIAo= 62553 2005-07-03 09:56 0000 Updated patch pbbutonsd.patch text/plain LS0tIGttaWxvL3Bvd2VyYm9vazIvcGJfbW9uaXRvci5jcHAuYmFrCTIwMDUtMDctMDMgMTI6NTQ6 MTguMDAwMDAwMDAwIC0wNDAwCisrKyBrbWlsby9wb3dlcmJvb2syL3BiX21vbml0b3IuY3BwCTIw MDUtMDctMDMgMTI6NTU6MjcuMDAwMDAwMDAwIC0wNDAwCkBAIC02NSw3ICs2NSwxMiBAQAogCiBi b29sIFBvd2VyQm9va01vbml0b3I6OmluaXQoKQogeworLy9wYmJ1dHRvbnMgMC42Ljcgb3IgbmV3 ZXIKKyNpZmRldiBDTElFTlRfUkVHSVNURVIKKwlyZXR1cm4gKCBpcGNfaW5pdCggImttaWxvIiwg TElCTU9ERV9DTElFTlQsIENMSUVOVF9SRUdJU1RFUiApID09IDAgKTsKKyNlbHNlCiAJcmV0dXJu ICggaXBjX2luaXQoIExJQk1PREVfQ0xJRU5ULCAxICkgPT0gMCApOworI2VuZGlmCiB9CiAKIAo=