[Yum-devel] [PATCH] Don't fail when pre-caching filelists. BZ 769864.

[James Antill]

On Thu, 2012-01-12 at 10:17 -0500, Zdenek Pavlas wrote:
> >  On the other hand I can't think of a reason to not change makecache to
> > just call:
> > 
> >             base.repos.populateSack(mdtype='all', cacheonly=1)
> > 
> > ...and then we don't have to worry about it.
> 
> Also 'makecache' sets mdpolicy to "group:all", so there seems 
> to be no need to explicitly load metadata/filelists/otherdata.

 Yeh, there is ... read the comment ...

> And there's base.repos.doSetup(), so I don't know why it touches
> every repo.repoXML.  The following works in HEAD:

 doSetup() just sets up the callbacks, it doesn't touch repo.repoXML
which needs to be touched to trigger the repomd.xml/etc. downloads

> --- a/yumcommands.py
> +++ b/yumcommands.py
> @@ -962,17 +962,6 @@ class MakeCacheCommand(YumCommand):
> -            # These convert the downloaded data into usable data,
> -            # we can't remove them until *LoadRepo() can do:
> -            # 1. Download a .sqlite.bz2 and convert to .sqlite
> -            # 2. Download a .xml.gz and convert to .xml.gz.sqlite