Summary: | incorrect dependencies for DirectFB | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Adam M. <adamm> |
Component: | New packages | Assignee: | Seemant Kulleen (RETIRED) <seemant> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | High | ||
Version: | 1.1a | ||
Hardware: | x86 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | my proposed patch |
Description
Adam M.
2002-04-20 14:05:59 UTC
I can't reproduce this problem on my machine. Would you send me the full output of the emerge which caused the problem? eg. emerge DirectFB &>/tmp/emerge-DirectFB.log and emerge xine-lib &>/tmp/emerge-xine-lib.log I was able to reproduce the bug by doing the following: emerge -C libmpeg3; emerge DirectFB the build fails because libmpeg3 is not found. Note that I have "mpeg" in my USE string. I believe the bug to be the following (taken from the DirectFB .ebuild): There is this block: use mpeg \ && myconf="${myconf} --with-libmpeg3=/usr/include/libmpeg3" \ || myconf="${myconf} --disable-libmpeg3" But there is no corresponding entry in the DEPENDS string. Thus, the problem would not be reproduced unless you also had "mpeg" in your USE string. Adding the following line to the DEPENDS section resolves the problem: mpeg? ( media-libs/libmpeg3 ) It then correctly detects libmpeg3 as a dependency and builds it prior to attempting to build DirectFB. I wouldn't be surprised, however, if the current version of DirectFB only worked with a particular version of libmpeg3, so I can't say that the addition of that line is entirely correct. It might be prudent to research the specific version and use >=media-libs/libmpeg3-VERSION instead. Hope this helps! I would have attached my proposed patch, but for some reason Bugzilla keeps saying that I haven't attached a file, or it's empty... should be simple to fix, though. See bug #71 for a attachment work around. i'd still like to se your diff. Created attachment 634 [details, diff]
my proposed patch
Nothing major...
I hope I did that correctly... first time using diff to create a patch :) okay thanks matt, trade you this for an abiword buf report You know what, when I upgraded to DirectFB-0.9.9, I added that, so the ebuild has actually been correct for a while. Meanwhile, I have added DirectFB-0.9.10 into portage as well, but it is masked. I don't suppose you'd be willing to test it, would you? |