[Frugalware-devel] Re: [PATCH] cache loading infolevel

[VMiklos]

On Sun, Oct 09, 2005 at 12:01:30AM +0200, Aurelien Foret <aurelien at archlinux.org> wrote:
> Please note that with this patch, the library is no more compatible with
> existing sync db archives.
> It means that sync db archives generated by pacman 2.9.x scripts (and
> thus the ones available on archlinux servers) should not be used anymore for tests.
> This is the reason why it was not yet implemented...

yes. but any "old" pacman can upgrade itself using the new repo format,
so that users who do the ususal pacman -Syu should not notice anything

> >a question: maybe we should export infolevel in alpm_db_readpkg() and
> >alpm_db_getpkgcache()? i'm not sure about it
> 
> I don't think so.
> 
> The whole idea is that the cache should remain not visible from the
> frontend.

ok

> To sum-up, the complexity is hidden within the library, which is using
> its cache if data is available in it to answer the queries from the
> frontend. Otherwise, the lib performs some disk access to read the
> database content and feeds the cache with it, before aswering the library.
> This way, things remain as simple as possible for the frontend.
> 
> I've just rewrote a little bit the cache handling: now, it only loads DEPENDS info by 
> default instead of DESC+DEPENDS previously.
> 
> DEPENDS is really needed in almost all library operations, so it really makes sense to 
> have it always loaded.
> I understand the aim of this patch, but I think it is now pointless.

thank you for describing that, then i think so :)

udv / greetings,
VMiklos

-- 
Developer of Frugalware Linux, to make things frugal - http://frugalware.org