Bug 257616 – app-text/kchmviewer-4.0 installs into 4.1 prefix

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[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) 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) 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) 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) 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) 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) 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) 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.