On Sun, Dec 25, 2005 at 04:21:00PM -0800, Judd Vinet <jvinet@zeroflux.org> wrote:
I don't think it's necessary with -S, because of the way resolvedeps() processes targets. Or am I missing something?
resolvedeps will not add the dependency if it's already in the target list (see libalpm/deps.c:629). so if you do a pacman -S bash glibc, the order will be bash glibc instead of glibc bash
Also, I think there is some leakage in there. Some list entries left behind due to the reassigning of trans->packages and the sortbydeps() call.
hm, yes, i forgot to FREELISTPTR() trans->packages. here is an updated patch: http://frugalware.org/~vmiklos/patches/libpacman-proposed/sync_prepare_deps2... sortbydeps() will modify the list in place. how could this cause a memleak?
ps: Happy Holidays to all pacman hackers! Don't work too hard over the holidays. :)
same to you! btw it's over now ;) udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org