[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