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

[James Antill]

On Fri, 2012-01-13 at 03:17 -0500, Zdenek Pavlas wrote:
> > > -            # 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
> 
> I still think it's not necessary, as _getRepoXML() 
> calls _commonRetrieveDataMD() and there's unconditional
> _groupCheckDataMDValid() which does the decompression.

 _groupCheckDataMDValid() doesn't do the decompression (_when_ it calls
decompress it does so with fn_only=True, which doesn't do
decompression), it also doesn't do the xml => sqlite conversion.

 IIRC the problem is that for both of those operations you want to have
the callbacks setup, so the user sees something is happening, but that
can't easily happen at _getRepoXML() time. But it's been a while, so it
might be possible to change ... I do know it hasn't changed, yet,
though.