Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 71563 - app-cdr/kiso uses einstall instead of `make install DESTDIR=$D`
Summary: app-cdr/kiso uses einstall instead of `make install DESTDIR=$D`
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-17 09:19 UTC by SpanKY
Modified: 2005-07-02 08:43 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SpanKY gentoo-dev 2004-11-17 09:19:58 UTC
einstall is a kludge that is supposed to be used only when DESTDIR is not supported
Comment 1 SpanKY gentoo-dev 2004-11-17 09:20:59 UTC
also, rather than doing an 'addwrite' on the qt settings dir, the ebuild should use the functions from the kde/qt eclasses

also, it looks upstream packaged it incorrectly ... the ebuild should run `make distclean` in src_unpack() to clean up all the generated crap
Comment 2 Simone Gotti (RETIRED) gentoo-dev 2004-11-17 09:58:52 UTC
Take a look at kiso-0.70, I've already fixed it to a right version. If needed I'll update also the old ebuilds.

I don't understand the "make distclean" needs, as they are things not installed.
Comment 3 SpanKY gentoo-dev 2004-11-17 11:04:33 UTC
it looks like just 0.6 was packaged incorrectly

0.7 does resolve all the issues mentioned here, so feel free to close once the other two ebuilds are removed from portage

it doesnt matter if the files are installed or not, `make distclean` is required for the 0.6 package since those files are per-host specific
Comment 4 Dan Armak (RETIRED) gentoo-dev 2005-07-01 11:05:42 UTC
ppc, could you keyword kiso-0.8.1 stable so we can remove 0.4.2 from  
portage?  
Comment 5 Joe Jezak (RETIRED) gentoo-dev 2005-07-02 07:15:03 UTC
Sure we can. Fixed in CVS.
Comment 6 Carsten Lohrke (RETIRED) gentoo-dev 2005-07-02 08:43:17 UTC
Thanks Joe.