On Fri, Aug 24, 2007 at 10:40:41PM -0400, Travis Willard wrote:
I don't much like it, to be honest - I'd prefer to separate the concept of the local repo from the sync repos entirely, but that would break the current interface into libalpm, as alpm_db_register("local") would, ideally, register a sync repo named "local", and we'd have to add another method like alpm_localdb_register() or something of that sort to register the local database.
I could do the less-dirty fix, but I quite imagine the current API is supposed to remain decently stable, so here's one that works with the current setup.
I'm not aware of any projects using it. Which ones are there ? Imo it's better to try improving the interface now than waiting :) Since both registering and unregistering the local databases require a special handling already in these functions : if (strcmp(name, "local") == 0) { blabla } I think it might be a good idea to split them, if it doesn't duplicate code too much.