> > +        default_grabber.opts.timedhosts = self._conf.cachedir +
> > '/timedhosts'
> 
>  Don't you need a hasattr() here?

We're assigning, so this should not break even with
old urlgrabber that does not set the default value.

> it fails in weird ways if someone has a repo. called "timedhosts" ...
> we could just move it to <cachedir>/main/timedhosts ... or put it
> in /var/lib/yum ... meh.

fastestmirror uses '/var/cache/yum/timedhosts.txt', so I just
removed the extension.. 'timedhosts.yum' might be a better name.

> However make sure it behaves with multiple updaters
> .. ie. write to new file and rename.

Added that.
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to