On Mon, Jan 19, 2009 at 1:12 PM, Xavier <[email protected]> wrote:
> On Mon, Jan 19, 2009 at 12:57 AM, Nagy Gabor <[email protected]> wrote:
>> >From 884155e03bf20f1cca880f39848423091729f6cd Mon Sep 17 00:00:00 2001
>> From: Nagy Gabor <[email protected]>
>> Date: Mon, 19 Jan 2009 00:35:45 +0100
>> Subject: [PATCH] db_register: Unlink DbPath/database, if it is not a 
>> directory
>>
>> We had a check for this case, but we didn't handle it at all (neither error,
>> nor fix), we also had a TODO in the code.
>>
>> For example, if user has an "unstable" regular file instead of "unstable"
>> directory (FS#12148), from now on pacman will remove that file (and print
>> a warning message). Before this patch pacman just stopped with "could not
>> register database" error.
>>
>> Signed-off-by: Nagy Gabor <[email protected]>
>
> Signed-off-by: Xavier Chantry <[email protected]>
>

I liked the change made to the check, but I didn't like this check in
the first place (especially the place where it was done).
So I just sent a patch to the ML which moves this code : Delay the
creation of local and sync db dir
So I integrated your changes in there.
_______________________________________________
pacman-dev mailing list
[email protected]
http://archlinux.org/mailman/listinfo/pacman-dev

Reply via email to