This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, master has been updated via 83c05e71bc6e614f6f3d190d5e752a0c5239a2d1 (commit) via 1463a9aa3646a0886b7ec892f4a436ea5bb4749a (commit) via ad8d3ceb898716cf2642fb71963c0ef5a490621e (commit) via 775b94e64903057a75071491670013975fc3ecb2 (commit) via 497501d1c751b340a115657084402bb3899396c4 (commit) via d1af9b70c8ed76d38c5cb6205de6c9349b6d8a09 (commit) via e0acf2f144404e613a75d5aafea2a909989544e9 (commit) via 9a58d5c6c5a69ebf7a920c357820a59c19d5b641 (commit) via 83f99bfb39841b254046f6c8a210b6a91aca22ee (commit) from a8ed39ce05b68f5c9f0e29d86bf9bb0ac81b0ffd (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 83c05e71bc6e614f6f3d190d5e752a0c5239a2d1 Author: lolilolicon <lolilolicon@gmail.com> Date: Thu Sep 29 10:45:20 2011 +0800 paccache: allow strictly integer for -k option Verify the argument to -k is a non-negative integer. Leading zeros are simply stripped. 'declare -i keep' allowed the argument to -k to be any arithmetic evaluation expression. The simple assignment 'keep=$OPTARG' triggers arithmetic evaluation implicitly, which can either consume a huge amount of resources with input such as '2**2**32' or immediately produce an error on invalid input. Instead, we simply 'declare -- keep' and avoid all that. Signed-off-by: lolilolicon <lolilolicon@gmail.com> Signed-off-by: Dave Reisner <dreisner@archlinux.org> commit 1463a9aa3646a0886b7ec892f4a436ea5bb4749a Author: Allan McRae <allan@archlinux.org> Date: Thu Sep 29 21:07:48 2011 +1000 Remove redundant "removal" output in pure remove operation Printing "[removal]" beside all package names is redundant when all packages are being removed (i.e. when using -R). Signed-off-by: Allan McRae <allan@archlinux.org> commit ad8d3ceb898716cf2642fb71963c0ef5a490621e Author: Dave Reisner <dreisner@archlinux.org> Date: Thu Sep 29 12:17:16 2011 -0400 move prevprogress onto payload handle This is a poor place for it, and it will likely move again in the future, but it's better to have it here than as a static variable. Initialization of this variable is now no longer necessary as its zeroed on creation of the payload struct. Signed-off-by: Dave Reisner <dreisner@archlinux.org> Signed-off-by: Dan McGee <dan@archlinux.org> commit 775b94e64903057a75071491670013975fc3ecb2 Author: lolilolicon <lolilolicon@gmail.com> Date: Thu Sep 29 03:05:03 2011 +0800 pkgdelta: proper quoting in [[ expression ]] Always quote the right-hand side of expression when the == or != operator is used, unless intended as a pattern. Signed-off-by: lolilolicon <lolilolicon@gmail.com> commit 497501d1c751b340a115657084402bb3899396c4 Author: lolilolicon <lolilolicon@gmail.com> Date: Wed Sep 28 13:06:03 2011 -0500 makepkg: proper quoting in [[ expression ]] Always quote the righthand side of expression when the == or != operator is used, unless intended as a pattern. Quoting bash(1): When the == and != operators are used, the string to the right of the operator is considered a pattern. Any part of the pattern may be quoted to force it to be matched as a string. Signed-off-by: lolilolicon <lolilolicon@gmail.com> commit d1af9b70c8ed76d38c5cb6205de6c9349b6d8a09 Author: Dan McGee <dan@archlinux.org> Date: Wed Sep 28 12:55:27 2011 -0500 Refactor display_targets to not be recursive This also fixes a memory leak and makes the dual-purpose "rows" variable go away in favor of storing the rows and non-verbose names separately. This also fixes some potential memory leaks and/or wrong behavior due to the config->verbosepkglists flag being flipped, which we should never be doing. Signed-off-by: Dan McGee <dan@archlinux.org> commit e0acf2f144404e613a75d5aafea2a909989544e9 Author: Dan McGee <dan@archlinux.org> Date: Wed Sep 28 12:58:43 2011 -0500 Refactor download payload reset and free This was done to squash a memory leak in the sync database download code. When we downloaded a database and then reused the payload struct, we could find ourselves calling get_fullpath() for the signatures and overwriting non-freed values we had left over from the database download. Refactor the payload_free function into a payload_reset function that we can call that does NOT free the payload itself, so we can reuse payload structs. This also allows us to move the payload to the stack in some call paths, relieving us of the need to alloc space. Signed-off-by: Dan McGee <dan@archlinux.org> commit 9a58d5c6c5a69ebf7a920c357820a59c19d5b641 Author: Dan McGee <dan@archlinux.org> Date: Wed Sep 28 12:06:36 2011 -0500 Initialize cURL library on first use Rather than always initializing it on any handle creation. There are several frontend operations (search, info, etc.) that never need the download code, so spending time initializing this every single time is a bit silly. This makes it a bit more like the GPGME code init path. Signed-off-by: Dan McGee <dan@archlinux.org> commit 83f99bfb39841b254046f6c8a210b6a91aca22ee Author: Dan McGee <dan@archlinux.org> Date: Wed Sep 28 04:47:28 2011 -0500 Add more valgrind suppressions Signed-off-by: Dan McGee <dan@archlinux.org> ----------------------------------------------------------------------- Summary of changes: contrib/paccache.in | 8 ++- lib/libalpm/alpm.c | 5 -- lib/libalpm/be_sync.c | 42 ++++++++------- lib/libalpm/dload.c | 138 +++++++++++++++++++++++++----------------------- lib/libalpm/dload.h | 4 +- lib/libalpm/handle.h | 1 - lib/libalpm/sync.c | 5 +- scripts/makepkg.sh.in | 6 +- scripts/pkgdelta.sh.in | 6 +- src/pacman/util.c | 62 ++++++++++----------- valgrind.supp | 25 +++++++++ 11 files changed, 164 insertions(+), 138 deletions(-) hooks/post-receive -- The official pacman repository