[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.0.0rc2-42-g83c05e7

Thu Sep 29 14:30:43 EDT 2011

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 at 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 at gmail.com>
    Signed-off-by: Dave Reisner <dreisner at archlinux.org>

commit 1463a9aa3646a0886b7ec892f4a436ea5bb4749a
Author: Allan McRae <allan at 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 at archlinux.org>

commit ad8d3ceb898716cf2642fb71963c0ef5a490621e
Author: Dave Reisner <dreisner at 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 at archlinux.org>
    Signed-off-by: Dan McGee <dan at archlinux.org>

commit 775b94e64903057a75071491670013975fc3ecb2
Author: lolilolicon <lolilolicon at 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 at gmail.com>

commit 497501d1c751b340a115657084402bb3899396c4
Author: lolilolicon <lolilolicon at 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 at gmail.com>

commit d1af9b70c8ed76d38c5cb6205de6c9349b6d8a09
Author: Dan McGee <dan at 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 at archlinux.org>

commit e0acf2f144404e613a75d5aafea2a909989544e9
Author: Dan McGee <dan at 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 at archlinux.org>

commit 9a58d5c6c5a69ebf7a920c357820a59c19d5b641
Author: Dan McGee <dan at 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 at archlinux.org>

commit 83f99bfb39841b254046f6c8a210b6a91aca22ee
Author: Dan McGee <dan at archlinux.org>
Date:   Wed Sep 28 04:47:28 2011 -0500

    Add more valgrind suppressions

    Signed-off-by: Dan McGee <dan at 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