148419 2006-09-20 14:11 0000 kmplayer 0.9.3 with binary mplayer on amd64 needs testing 2006-10-09 10:49:24 0000 1 1 1 Unclassified Gentoo Linux KDE unspecified All Linux RESOLVED WORKSFORME P2 normal --- 1 deathwing00@gentoo.org amd64@gentoo.org kde@gentoo.org deathwing00@gentoo.org 2006-09-20 14:11:11 0000 Please, test that the patch supplied works correctly (test all kmplayer functionalities) when using mplayer-bin under amd64. I have reported upstream that these harcoded mplayer entries are becoming annoying. Hope they do something about it. http://bugs.kde.org/show_bug.cgi?id=134418 deathwing00@gentoo.org 2006-09-20 14:23:26 0000 Nevermind, upstream confirmed that mplayer can be set from within the configuration menu from this version and on. Do the amd64 users still want this patch on this version or should I drop it and add an einfo? deathwing00@gentoo.org 2006-09-20 14:25:44 0000 Can you guys still try that the feature works correctly before I perform any change? ------- Additional Comment #2 From Koos Vriezen 2006-09-20 23:18 ------- Have you tried the MPlayer config tab in kmplayer's configure dialog? As of this 0.9.3 release, you should be able to set MPlayer's binary name (and even it's complete path) blubb@gentoo.org 2006-10-09 10:12:02 0000 Ioannis, can you just go on with the change? I haven't tested it (load of kde deps) but if it doesn't work out, we'll soon notice it. It is testing after all, users can expect minor breakage :) deathwing00@gentoo.org 2006-10-09 10:16:47 0000 It's already in the tree. I just wanted to be sure that it worked for everyone. If no one has complained to the moment, maybe things are alright anyway. MEMORANDUM: Do not create mplayer-bin patch for amd64 on next version. blubb@gentoo.org 2006-10-09 10:23:57 0000 Sure? kmplayer-0.9.3.ebuild: src_unpack() { kde_src_unpack if use mplayer && use amd64 && ! has_version media-video/mplayer; then epatch "${FILESDIR}/${P}-use32bitbin.diff" fi } deathwing00@gentoo.org 2006-10-09 10:31:09 0000 I was thinking in allowing the patch in there, but if you insist... :) deathwing00@gentoo.org 2006-10-09 10:49:24 0000 Fixed in CVS.