Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 262191 - request: app-portage/gentoolkit eclean option for distfiles path
Summary: request: app-portage/gentoolkit eclean option for distfiles path
Status: CONFIRMED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Tools (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Portage Tools Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-11 20:06 UTC by Juan
Modified: 2010-04-01 06:29 UTC (History)
0 users

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 Juan 2009-03-11 20:06:43 UTC
Is it possible to add the option to pass explicitly the path to the DISTDIR directory (ussually /usr/portage/distfiles). The reason is that if we can pass a path we can use it for local repositories, when using it with http-replicator for instance. It is quite easy to modify the eclean program and I did it for my personal use (just one line), but it would be nice if we just use the distribution's eclean and do not worry about updating.
Comment 1 michael@smith-li.com 2009-04-22 02:41:38 UTC
(In reply to comment #0)
> It is quite easy to modify the eclean program and I did it for my
> personal use (just one line), but it would be nice if we just use the
> distribution's eclean and do not worry about updating.

Please submit your patch!

Thanks :)

Comment 2 michael@smith-li.com 2009-05-03 19:25:30 UTC
eclean already respects DISTDIR and PKGDIR in the environment, like so:

$ DISTDIR="/home/michael/foo" eclean-dist

or

$ export PKGDIR="/home/michael/bar"
$ eclean-pkg

Is it really necessary to add a commandline option for this as well?
Comment 3 Brian Dolbec (RETIRED) gentoo-dev 2010-04-01 06:29:59 UTC
I am just completing a modular re-write of eclean, including a number of unit-tests.  One of the things that I did was to make the different functions get the distdir from it's input parameters.  This makes it much easier to test and also makes it possible for it to work on any distfiles or multiple directories in the future with less re-coding.

I will inquire with the pkg maintainer whether to add a cmd line option.  On a slightly different approach to your suggestion.  I also plan to add a network enabling feature that will work with a NFS shared or http-replicator shared distfiles that will correctly obtain info from the clients to decide what it should clean in the served distfiles.  It will make use of a hosts file for the list of clients to access for it's info.  It may also be advantageous to put a config option in eclean-net variant for the distfiles location.