[pacman-dev] [PATCH] move db lock/unlock to the transaction functions

VMiklos vmiklos at frugalware.org
Sun Jan 29 07:04:23 EST 2006


On Sat, Jan 28, 2006 at 10:32:11PM +0100, Aurelien Foret <aurelien at archlinux.org> wrote:
> Indeed, I noticed it right after applying your lock/unlock move patch.
> 
> In fact, the trans_release call has always been missing, even before 
> your patch (a transaction must be released, even if it happens to succeed!).
> 
> Note that the same also applied to pacman_add, pacman_sync and 
> pacman_deptest too.

ok, but why don't you removed the unlock stuff from alpm_release()?

without removing that the lockfile will be removed by pacman's cleanup()
even if an anouther pacman is already running

udv / greetings,
VMiklos

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




More information about the pacman-dev mailing list