On Wed, 2011-04-06 at 15:03 -0400, seth vidal wrote:
> On Wed, 2011-04-06 at 14:34 -0400, James Antill wrote:
> > ---
> > yum/yumRepo.py | 12 +++++++++++-
> > 1 files changed, 11 insertions(+), 1 deletions(-)
> >
> > diff --git a/yum/yumRepo.py b/yum/yumRepo.py
> > index 6ba058c..2246aec 100644
> > --- a/yum/yumRepo.py
> > +++ b/yum/yumRepo.py
> > @@ -544,7 +544,17 @@ class YumRepository(Repository, config.RepoConf):
> > """make the necessary dirs, if possible, raise on failure"""
> >
> > cachedir = os.path.join(self.basecachedir, self.id)
> > - persistdir = os.path.join(self.base_persistdir, self.id)
> > + if self.base_persistdir and self.base_persistdir[0] == '/':
> > + persistdir = self.base_persistdir
> > + else:
> > + # This sucks, because cakeys can't work ... but it's better
> > + # than creating dirs. in the currect dir.
> > + if tmpdir is None:
> > + tmpdir = os.getenv('TMPDIR')
> > + if tmpdir is None: # Note that TMPDIR isn't exported by
> > default :(
> > + tmpdir = '/var/tmp'
> > + persistdir = misc.getCacheDir(tmpdir=tmpdir)
> > + persistdir = os.path.join(persistdir, self.id)
> > pkgdir = os.path.join(cachedir, 'packages')
> > hdrdir = os.path.join(cachedir, 'headers')
> > self.setAttribute('_dir_setup_cachedir', cachedir)
>
>
> So is the situation here that the root user has not done anything to
> make the persistdirs?
No, it's that some random piece of code "creates" a repo. ... but it
doesn't set base_persistdir ... so it's the default of ''. The above
turns that into 'foobar', and then yum creates that dir. in $CWD and
everyone is unhappy :).
_______________________________________________
Yum-devel mailing list
Yum-devel@lists.baseurl.org
http://lists.baseurl.org/mailman/listinfo/yum-devel
