92710 2005-05-15 08:54 0000 xxdiff-3.0.2 dependancy on KDE isnt handled 100% correctly 2005-05-17 08:00:45 0000 1 1 1 Unclassified Gentoo Linux Ebuilds unspecified All Linux RESOLVED FIXED P2 normal --- 1 groepaz@gmx.net kde@gentoo.org groepaz@gmx.net 2005-05-15 08:54:56 0000 i somehow ended up with a /usr/kde/3.4/share/doc/HTML/en/kompare/common/ directory (maybe another bug that should be handled seperatly?) and nothing else of kde3.4 installed... this causes the xxdiff ebuild to think that i am using kde3.4, and the build fails. what i did was changing the detection to look for the /include subdirectory in the kde tree instead of just the kde directory, resulting in it detecting the correct version of kde. src_unpack() { unpack ${A} if use kde; then cd ${S}/src cp ${FILESDIR}/kdesupport.patch . # this one is not 100% #sed -e "s:/usr/kde/3.1:`ls -d /usr/kde | tail -n 1`:g" -i kdesupport.patch # this one works perfectly fine sed -e "s:/usr/kde/3.1:`ls -d /usr/kde/*/include | sed -e "s:/include::g" | tail -n 1`:g" -i kdesupport.patch epatch kdesupport.patch fi } mmmh...maybe there should be a function dedicated to this so the kde version detection can be addressed in a centralized way? Reproducible: Always Steps to Reproduce: 1. 2. 3. groepaz@gmx.net 2005-05-15 08:58:28 0000 Created an attachment (id=58956) fixed ebuild greg_g@gentoo.org 2005-05-17 08:00:45 0000 The detection of kde was indeed clumsy. I changed the ebuild to properly use the kde eclasses in 3.0.2-r1. Thanks. 58956 2005-05-15 08:58 0000 fixed ebuild xxdiff-3.0.2.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDA1IEdlbnRvbyBGb3VuZGF0aW9uCiMgRGlzdHJpYnV0ZWQgdW5k ZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5zZSB2MgojICRIZWFk ZXI6IC92YXIvY3Zzcm9vdC9nZW50b28teDg2L2Rldi11dGlsL3h4ZGlmZi94eGRpZmYtMy4wLjIu ZWJ1aWxkLHYgMS4xMyAyMDA1LzA1LzA4IDIyOjMxOjQyIGFncmlmZmlzIEV4cCAkCgppbmhlcml0 IGV1dGlscwoKREVTQ1JJUFRJT049IkEgZ3JhcGhpY2FsIGZpbGUgYW5kIGRpcmVjdG9yaWVzIGNv bXBhcmF0b3IgYW5kIG1lcmdlIHRvb2wuIgpTUkNfVVJJPSJtaXJyb3I6Ly9zb3VyY2Vmb3JnZS8k e1BOfS8ke1B9LnRhci5neiIKUkVTVFJJQ1Q9Im5vbWlycm9yIgpIT01FUEFHRT0iaHR0cDovL3h4 ZGlmZi5zb3VyY2Vmb3JnZS5uZXQvIgoKREVQRU5EPSI+PXgxMS1saWJzL3F0LTMuMC4wCgk9ZGV2 LXV0aWwvdG1ha2UtMS44KgoJa2RlPyAoID49a2RlLWJhc2Uva2RlbGlicy0zLjEuMCApIgoKUkRF UEVORD0ic3lzLWFwcHMvZGlmZnV0aWxzIgoKU0xPVD0iMCIKTElDRU5TRT0iR1BMLTIiCktFWVdP UkRTPSJhbHBoYSB+YW1kNjQgaWE2NCB+cHBjIH5zcGFyYyB+eDg2IgpJVVNFPSJrZGUiCgpzcmNf dW5wYWNrKCkKewoJdW5wYWNrICR7QX0KCWlmIHVzZSBrZGU7IHRoZW4KCQljZCAke1N9L3NyYwoJ CWNwICR7RklMRVNESVJ9L2tkZXN1cHBvcnQucGF0Y2ggLgoJCXNlZCAtZSAiczovdXNyL2tkZS8z LjE6YGxzIC1kIC91c3Iva2RlLyovaW5jbHVkZSB8IHNlZCAtZSAiczovaW5jbHVkZTo6ZyIgfCB0 YWlsIC1uIDFgOmciIC1pIGtkZXN1cHBvcnQucGF0Y2gKCQllcGF0Y2gga2Rlc3VwcG9ydC5wYXRj aAoJZmkKfQoKc3JjX2NvbXBpbGUoKSB7CgljZCBzcmMKCXRtYWtlIC1vIE1ha2VmaWxlIHh4ZGlm Zi5wcm8KCgkjIyMgYm9ycm93ZWQgZnJvbSBrZGUuZWNsYXNzICMKCSMKCSMgZml4IHRoZSBzYW5k Ym94IGVycm9ycyAiY2FuJ3Qgd3JpdCB0byAua2RlIG9yIC5xdCIgcHJvYmxlbXMuCgkjIHRoaXMg aXMgYSBmYWtlIGhvbWVkaXIgdGhhdCBpcyB3cml0ZWFibGUgdW5kZXIgdGhlIHNhbmRib3gsIHNv IHRoYXQgdGhlIGJ1aWxkCgkjIHByb2Nlc3MgY2FuIGRvIGFueXRoaW5nIGl0IHdhbnRzIHdpdGgg aXQuCglSRUFMSE9NRT0iJEhPTUUiCglta2RpciAtcCAkVC9mYWtlaG9tZS8ua2RlCglta2RpciAt cCAkVC9mYWtlaG9tZS8ucXQKCWV4cG9ydCBIT01FPSIkVC9mYWtlaG9tZSIKCWFkZHdyaXRlICIk e1FURElSfS9ldGMvc2V0dGluZ3MiCgoJIyB0aGluZ3MgdGhhdCBzaG91bGQgYWNjZXNzIHRoZSBy ZWFsIGhvbWVkaXIKCVsgLWQgIiRSRUFMSE9NRS8uY2NhY2hlIiBdICYmIGxuIC1zZiAiJFJFQUxI T01FLy5jY2FjaGUiICIkSE9NRS8iCgoJZW1ha2UgfHwgZGllCn0KCnNyY19pbnN0YWxsICgpIHsK CWRvYmluIGJpbi94eGRpZmYgYmluL3h4ZGlmZi1jdnMtZGlmZiBiaW4veHhkaWZmLWVuY3J5cHRl ZCBiaW4veHhkaWZmLWZpbmQtZ3JlcC1zZWQgYmluL3h4ZGlmZi1tYXRjaAoJZG9tYW4gc3JjL3h4 ZGlmZi4xCglkb2RvYyBSRUFETUUgQ09QWUlORyBDSEFOR0VTIFRPRE8KfQo=