Patch send upstream, see "URL" above. I have tested all relevant USE flag combinations with the changes I'll attach as patches and it is OK. The dependency occurs when qt4 is in USE and kde-base/kdelibs:4 is installed. When kdelibs is installed and when it's not, USE=qt4 makes libmltqimage.so to be built. Then, if kdelibs is on the system, the library links to libkdecore.so.5 ( :( ). Reproducible: Always
By the time I finished writing this comment above, the patch was merged! :O
Created attachment 330956 [details, diff] patch on ebuild that fixes this The "brackets" change is stylish (avoids a subshell).
Created attachment 330958 [details, diff] patch to be applied on sources
Created attachment 330960 [details, diff] add 'kdenlive' USE flag This adds 'kdenlive' USE flag. This way one can enable linkage to KDE libs and this kdenlive thingy independently. Pretty please.
Created attachment 330966 [details, diff] patch on ebuild that fixes this corrected patch (please see this one first, then the one from comment 4)
(In reply to comment #4) > Created attachment 330960 [details, diff] [details, diff] > add 'kdenlive' USE flag > > This adds 'kdenlive' USE flag. This way one can enable linkage to KDE libs > and this kdenlive thingy independently. Pretty please. By the way, this of course means that kdenlive ebuild should be updated: from media-libs/mlt[kde] to media-libs/mlt[kdenlive].
Regarding bug title change: it's not just about making the dependency optional; it's about solving the automagic dependency issue, and - an extension to fixing this issue - adding a new USE flag to provide choice for two different needs.
+*mlt-0.8.6-r1 (20 Jan 2013) + + 20 Jan 2013; Ben de Groot <yngwin@gentoo.org> + +files/mlt-0.8.6-kdelibs-automagic.patch, +mlt-0.8.6-r1.ebuild, metadata.xml: + Add kdelibs-automagic patch, which has already been accepted upstream. Add + kdenlive useflag, as it is separately configurable from linking to kdelibs. + Fixes bug #445230. Thanks to Enlik for the patches.
thanks :)
