Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 257616

Summary: app-text/kchmviewer-4.0 installs into 4.1 prefix
Product: Gentoo Linux Reporter: Tomas Caithaml <tomas.caithaml>
Component: [OLD] KDEAssignee: Gentoo KDE team <kde>
Status: RESOLVED FIXED    
Severity: normal CC: pva
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: kchmviewer-4.0.ebuild.patch
File list from proposed patch.
Original filelist
kchmviewer-4.0.ebuild.patch
kchmviewer-4.0.ebuild.patch
kde4-base.eclass.patch
kchmviewer-4.0.ebuild.patch
File list from proposed patch.

Description Tomas Caithaml 2009-02-04 14:25:59 UTC
I have upgraded to KDE 4.2 with kdeprefix USE flag enabled but kchmviewer was installed into /usr/kde/4.1 instead of 4.2.
Comment 1 Peter Volkov (RETIRED) gentoo-dev 2009-02-07 09:28:02 UTC
Created attachment 181226 [details, diff]
kchmviewer-4.0.ebuild.patch

Thank you for report, Tomas. Could you try attached patch and report if it works for you. I don't have kdeprefix'ed installation at the moment so I can check by myself.
Comment 2 Tomas Caithaml 2009-02-07 13:25:01 UTC
Created attachment 181248 [details]
File list from proposed patch.

(In reply to comment #1)
> Created an attachment (id=181226) [edit]
> kchmviewer-4.0.ebuild.patch
> 
> Thank you for report, Tomas. Could you try attached patch and report if it
> works for you. I don't have kdeprefix'ed installation at the moment so I can
> check by myself.
> 

Thank for quick response. Unfortunately the patch does not work properly. Now kchmviewer is not prefixed (even if kdeprefix USE flag is set), emerge even creates toplevel /share directory (see attached filelist)
Comment 3 Tomas Caithaml 2009-02-07 13:27:34 UTC
Created attachment 181250 [details]
Original filelist

This is how files were installed in original version.
Comment 4 Peter Volkov (RETIRED) gentoo-dev 2009-02-09 20:30:40 UTC
Created attachment 181475 [details, diff]
kchmviewer-4.0.ebuild.patch

Thank you for feed back. Please, try this patch.
Comment 5 Peter Volkov (RETIRED) gentoo-dev 2009-02-09 20:42:05 UTC
Created attachment 181481 [details, diff]
kchmviewer-4.0.ebuild.patch

And even better this, patch.
Comment 6 Peter Volkov (RETIRED) gentoo-dev 2009-02-09 21:48:32 UTC
Created attachment 181490 [details, diff]
kde4-base.eclass.patch

Tomas, after discussion with kde team it was decided to go another way. Could you try it out. You need to patch both /usr/portage/eclass/kde4-base.eclass and kchmviewer-4.0.ebuild. So please apply this patch.
Comment 7 Peter Volkov (RETIRED) gentoo-dev 2009-02-09 21:49:12 UTC
Created attachment 181491 [details, diff]
kchmviewer-4.0.ebuild.patch

And this one and test, please. :)
Comment 8 Tomas Caithaml 2009-02-10 09:39:56 UTC
Created attachment 181531 [details]
File list from proposed patch.

Well, this one is closer. kchmviewer is now installed directly in /usr/kde (with no prefix).
Comment 9 Tomáš Chvátal (RETIRED) gentoo-dev 2009-02-10 10:33:10 UTC
(In reply to comment #8)
> Created an attachment (id=181531) [edit]
> File list from proposed patch.
> 
> Well, this one is closer. kchmviewer is now installed directly in /usr/kde
> (with no prefix).
> 

Heya,
you might think about droping on #gentoo.cs and we can talk about it, i can produce some patch for the eclass, i hate working throught bug, it is too slow :]
Comment 10 Peter Volkov (RETIRED) gentoo-dev 2009-02-10 21:13:53 UTC
Tomas we've commited what we think should work to the tree and hope bug is fixed. Please, try kchmviewer-4.0-r1 and if it fails reopen bug please. Thank you for you help.