August 2014 - Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] paccache: adding the possibility for multiple cachedirs
by silent boatman 25 Aug '14

25 Aug '14

Either by specifing several -c parameters or only one -c parameter with a comma separated value. One charedir could for example be for the official arch packages and one for the aur. Signed-off-by: Maxim Andersson <thesilentboatman(a)gmail.com> --- contrib/paccache.sh.in | 89 +++++++++++++++++++++++++++++--------------------- 1 file changed, 51 insertions(+), 38 deletions(-) diff --git a/contrib/paccache.sh.in b/contrib/paccache.sh.in index 039ac8a..2170d76 100644 --- a/contrib/paccache.sh.in +++ b/contrib/paccache.sh.in @@ -23,9 +23,9 @@ shopt -s extglob declare -r myname='paccache' declare -r myver='@PACKAGE_VERSION@' -declare -a candidates=() cmdopts=() whitelist=() blacklist=() +declare -a cachedirs=() candidates=() cmdopts=() whitelist=() blacklist=() declare -i delete=0 dryrun=0 filecount=0 move=0 needsroot=0 totalsaved=0 verbose=0 -declare cachedir=@localstatedir@/cache/pacman/pkg delim=$'\n' keep=3 movedir= scanarch= +declare delim=$'\n' keep=3 movedir= scanarch= USE_COLOR='y' @@ -159,25 +159,25 @@ A flexible pacman cache cleaning utility. Usage: ${myname} <operation> [options] [targets...] Operations: - -d, --dryrun perform a dry run, only finding candidate packages. - -m, --move <dir> move candidate packages to "dir". - -r, --remove remove candidate packages. + -d, --dryrun perform a dry run, only finding candidate packages. + -m, --move <dir> move candidate packages to "dir". + -r, --remove remove candidate packages. Options: - -a, --arch <arch> scan for "arch" (default: all architectures). - -c, --cachedir <dir> scan "dir" for packages. - (default: @localstatedir@/cache/pacman/pkg). - -f, --force apply force to mv(1) and rm(1) operations. - -h, --help display this help message and exit. - -i, --ignore <pkgs> ignore "pkgs", comma-separated. Alternatively, specify - "-" to read package names from stdin, newline- - delimited. - -k, --keep <num> keep "num" of each package in the cache (default: 3). - --nocolor remove color from output. - -u, --uninstalled target uninstalled packages. - -v, --verbose increase verbosity. specify up to 3 times. - -z, --null use null delimiters for candidate names (only with -v - and -vv). + -a, --arch <arch> scan for "arch" (default: all architectures). + -c, --cachedir <dirs> scan "dirs" for packages, comma-separated. + (default: @localstatedir@/cache/pacman/pkg). + -f, --force apply force to mv(1) and rm(1) operations. + -h, --help display this help message and exit. + -i, --ignore <pkgs> ignore "pkgs", comma-separated. Alternatively, + pecify "-" to read package names from stdin, newline- + delimited. + -k, --keep <num> keep "num" of each package in the cache (default: 3). + --nocolor remove color from output. + -u, --uninstalled target uninstalled packages. + -v, --verbose increase verbosity. specify up to 3 times. + -z, --null use null delimiters for candidate names (only with -v + and -vv). EOF } @@ -203,7 +203,9 @@ while :; do scanarch=$2 shift ;; -c|--cachedir) - cachedir=$2 + IFS=',' read -r -a cdrs <<< "$2" + cachedirs+=("${cdrs[@]}") + unset cdrs shift ;; -d|--dryrun) dryrun=1 ;; @@ -256,6 +258,9 @@ done m4_include(../scripts/library/term_colors.sh) +# setting default cachedir +cachedirs=${cachedirs:-@localstatedir@/cache/pacman/pkg} + # remaining args are a whitelist whitelist=("$@") @@ -265,28 +270,36 @@ case $(( dryrun+delete+move )) in [^1]) die "only one operation may be used at a time" ;; esac -[[ -d $cachedir ]] || - die "cachedir '%s' does not exist or is not a directory" "$cachedir" +for cachedir in "${cachedirs[@]}" +do + [[ -d $cachedir ]] || + die "cachedir '%s' does not exist or is not a directory" "$cachedir" -[[ $movedir && ! -d $movedir ]] && - die "destination directory '%s' does not exist or is not a directory" "$movedir" + [[ $movedir && ! -d $movedir ]] && + die "destination directory '%s' does not exist or is not a directory" "$movedir" + + if (( move || delete )); then + # make it an absolute path since we're about to chdir + [[ ${movedir:0:1} != '/' ]] && movedir=$PWD/$movedir + [[ ! -w $cachedir || ( $movedir && ! -w $movedir ) ]] && needsroot=1 + fi + + # unlikely that this will fail, but better make sure + cd "$cachedir" >/dev/null || die "failed to chdir to '%s'" "$cachedir" + + # note that these results are returned in an arbitrary order from awk, but + # they'll be resorted (in summarize) iff we have a verbosity level set. + IFS=$'\n' read -r -d '' -a tmpcandidates < \ + <(printf '%s\n' *.pkg.tar?(.+([^.])) | pacsort --files | + pkgfilter "$keep" "$scanarch" \ + "${#whitelist[*]}" "${whitelist[@]}" \ + "${#blacklist[*]}" "${blacklist[@]}") + + candidates+=(${tmpcandidates[@]}) +done -if (( move || delete )); then - # make it an absolute path since we're about to chdir - [[ ${movedir:0:1} != '/' ]] && movedir=$PWD/$movedir - [[ ! -w $cachedir || ( $movedir && ! -w $movedir ) ]] && needsroot=1 -fi -# unlikely that this will fail, but better make sure -cd "$cachedir" >/dev/null || die "failed to chdir to '%s'" "$cachedir" -# note that these results are returned in an arbitrary order from awk, but -# they'll be resorted (in summarize) iff we have a verbosity level set. -IFS=$'\n' read -r -d '' -a candidates < \ - <(printf '%s\n' *.pkg.tar?(.+([^.])) | pacsort --files | - pkgfilter "$keep" "$scanarch" \ - "${#whitelist[*]}" "${whitelist[@]}" \ - "${#blacklist[*]}" "${blacklist[@]}") if (( ! ${#candidates[*]} )); then msg 'no candidate packages found for pruning' -- 2.0.3

2 2

[pacman-dev] Bug with remove conflicting packages and their depends
by Allan McRae 22 Aug '14

22 Aug '14

[2014-08-21 21:27] [PACMAN] Running 'pacman -Syu vim-minimal' ... [2014-08-22 09:17] [ALPM] removed vim (7.4.404-1) .. [2014-08-22 09:17] [ALPM] upgraded vim-runtime (7.4.404-1 -> 7.4.410-1) [2014-08-22 09:17] [ALPM] installed vim-minimal (7.4.410-1) ... [2014-08-22 09:17] [ALPM] installed ruby (2.1.2-1) [2014-08-22 09:17] [ALPM] installed lua (5.2.3-1) Ruby and lua are dependencies of vim which is removed. They were orphans after this transaction. Here is debug output: ... [09:38:49] debug: pulling dependency ruby (needed by vim) [09:38:49] debug: started resolving dependencies [09:38:49] debug: checkdeps: package lua-5.2.3-1 [09:38:49] debug: finished resolving dependencies [09:38:49] debug: pulling dependency lua (needed by vim) [09:38:49] debug: finished resolving dependencies looking for conflicting packages... [09:38:49] debug: looking for conflicts [09:38:49] debug: check targets vs targets [09:38:49] debug: check targets vs targets [09:38:49] debug: package vim-minimal conflicts with vim (by vim) [09:38:49] debug: package vim conflicts with vim-minimal (by vim-minimal) [09:38:49] debug: conflicting packages in the sync list: 'vim-minimal' <-> 'vim' [09:38:49] warning: removing 'vim' from target list because it conflicts with 'vim-minimal' [09:38:49] debug: check targets vs db and db vs targets [09:38:49] debug: check targets vs db [09:38:49] debug: package vim-minimal conflicts with vim (by vim) [09:38:49] debug: check db vs targets [09:38:49] debug: package 'vim-minimal' conflicts with 'vim' :: vim-minimal and vim are in conflict. Remove vim? [y/N] So I guess the fix is to consider explicitly stated packages first. I'll file a bug report later. Allan

1 0

[pacman-dev] [PATCH] fix improper FREELIST calls
by Andrew Gregory 20 Aug '14

20 Aug '14

* conflicts need to be freed with alpm_conflict_free * sync dbs need to be unregistered and are handled by alpm_release Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- lib/libalpm/conflict.c | 8 ++++++-- lib/libalpm/handle.c | 1 - 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/conflict.c b/lib/libalpm/conflict.c index 961e3a7..db07102 100644 --- a/lib/libalpm/conflict.c +++ b/lib/libalpm/conflict.c @@ -449,7 +449,9 @@ alpm_list_t *_alpm_db_find_fileconflicts(alpm_handle_t *handle, conflicts = add_fileconflict(handle, conflicts, path, p1, p2); if(handle->pm_errno == ALPM_ERR_MEMORY) { - FREELIST(conflicts); + alpm_list_free_inner(conflicts, + (alpm_list_fn_free) alpm_conflict_free); + alpm_list_free(conflicts); alpm_list_free(common_files); return NULL; } @@ -622,7 +624,9 @@ alpm_list_t *_alpm_db_find_fileconflicts(alpm_handle_t *handle, if(!resolved_conflict) { conflicts = add_fileconflict(handle, conflicts, path, p1, NULL); if(handle->pm_errno == ALPM_ERR_MEMORY) { - FREELIST(conflicts); + alpm_list_free_inner(conflicts, + (alpm_list_fn_free) alpm_conflict_free); + alpm_list_free(conflicts); alpm_list_free(tmpfiles); return NULL; } diff --git a/lib/libalpm/handle.c b/lib/libalpm/handle.c index fdd269b..7652d1f 100644 --- a/lib/libalpm/handle.c +++ b/lib/libalpm/handle.c @@ -81,7 +81,6 @@ void _alpm_handle_free(alpm_handle_t *handle) FREE(handle->lockfile); FREE(handle->arch); FREE(handle->gpgdir); - FREELIST(handle->dbs_sync); FREELIST(handle->noupgrade); FREELIST(handle->noextract); FREELIST(handle->ignorepkg); -- 2.1.0

1 0

[pacman-dev] [PATCH v2] Extend database upgrade script to handle alpm db version 9
by Allan McRae 20 Aug '14

20 Aug '14

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- v2: Use awk to do symlink to actual directory replacement As an example of what changes get made: diff -Naur local.tmp/glibc-2.18-4/files local/glibc-2.18-4/files --- local.tmp/glibc-2.18-4/files 2013-09-18 18:45:35.747068765 +1000 +++ local/glibc-2.18-4/files 2013-09-18 18:51:29.633643945 +1000 @@ -4,8 +4,8 @@ etc/locale.gen etc/nscd.conf etc/rpc -lib/ -lib/libc.a +usr/lib/ +usr/lib/libc.a usr/ usr/bin/ usr/bin/catchsegv Note that now usr/lib/ is present twice in the file (may not always be the case) and these entries are out of alphabetical order. Both these are cosmetic to pacman. scripts/pacman-db-upgrade.sh.in | 59 ++++++++++++++++++++++++++++++++++------- 1 file changed, 49 insertions(+), 10 deletions(-) diff --git a/scripts/pacman-db-upgrade.sh.in b/scripts/pacman-db-upgrade.sh.in index a1630c5..d6f4169 100644 --- a/scripts/pacman-db-upgrade.sh.in +++ b/scripts/pacman-db-upgrade.sh.in @@ -109,18 +109,57 @@ fi # do not let pacman run while we do this touch "$lockfile" -# pacman-3.4 to 3.5 upgrade - merge depends into desc -if [[ $(find "$dbroot"/local -name depends) ]]; then - msg "$(gettext "Pre-3.5 database format detected - upgrading...")" - for i in "$dbroot"/local/*; do - if [[ -f "$i"/depends ]]; then - cat "$i"/depends >> "$i"/desc - rm "$i"/depends - fi - done - msg "$(gettext "Done.")" +if [[ -f "${dbroot}"/local/.alpm_db_version ]]; then + db_version=$(cat "${dbroot}"/local/.alpm_db_version) fi +if [[ -z "$db_version" ]]; then + # pacman-3.4 to 3.5 upgrade - merge depends into desc + if [[ $(find "$dbroot"/local -name depends) ]]; then + msg "$(gettext "Pre-3.5 database format detected - upgrading...")" + for i in "$dbroot"/local/*; do + if [[ -f "$i"/depends ]]; then + cat "$i"/depends >> "$i"/desc + rm "$i"/depends + fi + done + msg "$(gettext "Done.")" + fi + + # pacman 4.1 to 4.2 upgrade - remove directory symlink support + dirlist=() + + unset GREP_OPTIONS + while IFS= read -r dir; do + dirlist+=("/${dir%/}") + done < <(grep -h '/$' "$dbroot"/local/*/files | sort -u) + + mapfile -t dirlist < <(find "${dirlist[@]}" -maxdepth 0 -type l) + + if [[ ${#dirlist[@]} != 0 ]]; then + msg "$(gettext "Pre-4.2 database format detected - upgrading...")" + for dir in "${dirlist[@]}"; do + realdir="$(cd $dir; pwd -P)" + for f in "$dbroot"/local/*/files; do + awk -v "dir=${dir#/}/" -v "realdir=${realdir#/}/" ' + BEGIN { + i = length(dir) + 1 + } + { + if (index($0, dir) == 1) { + printf("%s%s\n", realdir, substr($0, i)) + } else { + print + } + }' $f > $f.tmp + mv $f.tmp $f + done + done + fi +fi + +echo "9" > "$dbroot"/local/.alpm_db_version + # remove the lock file rm -f "$lockfile" -- 1.8.4

2 9

[pacman-dev] [PATCH] pacman.c: Add -gg, -ii, and -yy to help summary
by Johannes Löthberg 14 Aug '14

14 Aug '14

The sync help summaries was missing the information about: * -gg: View all groups and members * -ii: View extended information * -yy: Force refresh even if DBs are up to date Original-work-by: Earnestly <zibeon(a)gmail.com> Signed-off-by: Johannes Löthberg <johannes(a)kyriasis.com> --- src/pacman/pacman.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 345fb0a..f3b5672 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -153,14 +153,16 @@ static void usage(int op, const char * const myname) printf("%s: %s {-S --sync} [%s] [%s]\n", str_usg, myname, str_opt, str_pkg); printf("%s:\n", str_opt); addlist(_(" -c, --clean remove old packages from cache directory (-cc for all)\n")); - addlist(_(" -g, --groups view all members of a package group\n")); - addlist(_(" -i, --info view package information\n")); + addlist(_(" -g, --groups view all members of a package group\n" + " (-gg to view all groups and members)\n")); + addlist(_(" -i, --info view package information (-ii for extended information)\n")); addlist(_(" -l, --list <repo> view a list of packages in a repo\n")); addlist(_(" -q, --quiet show less information for query and search\n")); addlist(_(" -s, --search <regex> search remote repositories for matching strings\n")); addlist(_(" -u, --sysupgrade upgrade installed packages (-uu allows downgrade)\n")); addlist(_(" -w, --downloadonly download packages but do not install/upgrade anything\n")); - addlist(_(" -y, --refresh download fresh package databases from the server\n")); + addlist(_(" -y, --refresh download fresh package databases from the server\n" + " (-yy to force a refresh even if up to date)\n")); addlist(_(" --needed do not reinstall up to date packages\n")); } else if(op == PM_OP_DATABASE) { printf("%s: %s {-D --database} <%s> <%s>\n", str_usg, myname, str_opt, str_pkg); -- 2.0.4

1 1

[pacman-dev] [PATCH 1/4] paccache: read default cachedir from pacman.conf
by Dave Reisner 11 Aug '14

11 Aug '14

Implements FS#40738. --- contrib/paccache.sh.in | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/contrib/paccache.sh.in b/contrib/paccache.sh.in index 039ac8a..57c0458 100644 --- a/contrib/paccache.sh.in +++ b/contrib/paccache.sh.in @@ -25,7 +25,7 @@ declare -r myver='@PACKAGE_VERSION@' declare -a candidates=() cmdopts=() whitelist=() blacklist=() declare -i delete=0 dryrun=0 filecount=0 move=0 needsroot=0 totalsaved=0 verbose=0 -declare cachedir=@localstatedir@/cache/pacman/pkg delim=$'\n' keep=3 movedir= scanarch= +declare cachedir= delim=$'\n' keep=3 movedir= scanarch= USE_COLOR='y' @@ -37,6 +37,19 @@ die() { exit 1 } +get_cachedir_from_config() { + local key value + + while IFS=$'= \t' read -r key value _; do + if [[ $key = CacheDir ]]; then + echo "$value" + return 0 + fi + done <"$1" + + return 1 +} + # reads a list of files on stdin and prints out deletion candidates pkgfilter() { # there's whitelist and blacklist parameters passed to this @@ -165,8 +178,7 @@ Usage: ${myname} <operation> [options] [targets...] Options: -a, --arch <arch> scan for "arch" (default: all architectures). - -c, --cachedir <dir> scan "dir" for packages. - (default: @localstatedir@/cache/pacman/pkg). + -c, --cachedir <dir> scan "dir" for packages. (default: $cachedir). -f, --force apply force to mv(1) and rm(1) operations. -h, --help display this help message and exit. -i, --ignore <pkgs> ignore "pkgs", comma-separated. Alternatively, specify @@ -191,6 +203,8 @@ OPT_SHORT=':a:c:dfhi:k:m:rsuVvz' OPT_LONG=('arch:' 'cachedir:' 'dryrun' 'force' 'help' 'ignore:' 'keep:' 'move' 'nocolor' 'remove' 'uninstalled' 'version' 'verbose' 'null') +cachedir=$(get_cachedir_from_config "@sysconfdir@/pacman.conf") + if ! parseopts "$OPT_SHORT" "${OPT_LONG[@]}" -- "$@"; then exit 1 fi -- 2.0.4

2 5

[pacman-dev] [PATCH 1/3] makepkg: break out check_checksums to reasonably sized functions
by Dave Reisner 11 Aug '14

11 Aug '14

--- scripts/makepkg.sh.in | 95 +++++++++++++++++++++++++++++---------------------- 1 file changed, 54 insertions(+), 41 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 4b23a5b..cb5ded9 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1111,53 +1111,66 @@ generate_checksums() { done } -check_checksums() { - (( SKIPCHECKSUMS )) && return 0 - (( ! ${#source[@]} )) && return 0 +verify_integrity_one() { + local source_name=$1 integ=$2 expectedsum=$3 - local correlation=0 - local integ required - for integ in "${known_hash_algos[@]}"; do - local sumname="${integ}sums[@]" - local integrity_sums=("${!sumname}") - if (( ${#integrity_sums[@]} == ${#source[@]} )); then - msg "$(gettext "Validating source files with %s...")" "${integ}sums" - correlation=1 - local idx errors=0 - for (( idx = 0; idx < ${#source[*]}; idx++ )); do - local file="$(get_filename "${source[idx]}")" - printf ' %s ... ' "$file" >&2 - - if [[ ${integrity_sums[idx]} = 'SKIP' ]]; then - printf '%s\n' "$(gettext "Skipped")" >&2 - continue - fi + local file="$(get_filename "$source_name")" + printf ' %s ... ' "$file" >&2 - if ! file="$(get_filepath "$file")"; then - printf '%s\n' "$(gettext "NOT FOUND")" >&2 - errors=1 - continue - fi + if [[ $expectedsum = 'SKIP' ]]; then + printf '%s\n' "$(gettext "Skipped")" >&2 + return + fi - local expectedsum="${integrity_sums[idx],,}" - local realsum="$(openssl dgst -${integ} "$file")" - realsum="${realsum##* }" - if [[ $expectedsum = "$realsum" ]]; then - printf '%s\n' "$(gettext "Passed")" >&2 - else - printf '%s\n' "$(gettext "FAILED")" >&2 - errors=1 - fi - done + if ! file="$(get_filepath "$file")"; then + printf '%s\n' "$(gettext "NOT FOUND")" >&2 + return 1 + fi - if (( errors )); then - error "$(gettext "One or more files did not pass the validity check!")" - exit 1 # TODO: error code - fi - elif (( ${#integrity_sums[@]} )); then - error "$(gettext "Integrity checks (%s) differ in size from the source array.")" "$integ" + local realsum="$(openssl dgst -${integ} "$file")" + realsum="${realsum##* }" + if [[ ${expectedsum,,} = "$realsum" ]]; then + printf '%s\n' "$(gettext "Passed")" >&2 + else + printf '%s\n' "$(gettext "FAILED")" >&2 + return 1 + fi + + return 0 +} + +verify_integrity_sums() { + local integ=$1 integrity_sums + + array_build integrity_sums "${integ}sums" + + if (( ${#integrity_sums[@]} == ${#source[@]} )); then + msg "$(gettext "Validating source files with %s...")" "${integ}sums" + local idx errors=0 + for (( idx = 0; idx < ${#source[*]}; idx++ )); do + verify_integrity_one "${source[idx]}" "$integ" "${integrity_sums[idx]}" || errors=1 + done + + if (( errors )); then + error "$(gettext "One or more files did not pass the validity check!")" exit 1 # TODO: error code fi + elif (( ${#integrity_sums[@]} )); then + error "$(gettext "Integrity checks (%s) differ in size from the source array.")" "$integ" + exit 1 # TODO: error code + else + return 1 + fi +} + +check_checksums() { + (( SKIPCHECKSUMS )) && return 0 + (( ! ${#source[@]} )) && return 0 + + local correlation=0 + local integ + for integ in "${known_hash_algos[@]}"; do + verify_integrity_sums "$integ" && correlation=1 done if (( ! correlation )); then -- 2.0.4

2 4

[pacman-dev] [PATCH] pacman-key: stricter parsing for -verify
by Allan McRae 09 Aug '14

09 Aug '14

Prevents trust being spoofed by using TRUST_FULLY in the signatory's name or in an added notation. Fixes FS#41147. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/pacman-key.sh.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in index 82340f9..ba8d02e 100644 --- a/scripts/pacman-key.sh.in +++ b/scripts/pacman-key.sh.in @@ -482,7 +482,7 @@ verify_sig() { local ret=0 for sig; do msg "Checking %s ..." "$sig" - if ! "${GPG_PACMAN[@]}" --status-fd 1 --verify "$sig" | grep -qE 'TRUST_(FULLY|ULTIMATE)'; then + if ! "${GPG_PACMAN[@]}" --status-fd 1 --verify "$sig" | grep -qE '^\[GNUPG:\] TRUST_(FULLY|ULTIMATE)$'; then error "$(gettext "The signature identified by %s could not be verified.")" "$sig" ret=1 fi -- 2.0.4

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.1.2-406-g3e19cd3
by allan＠archlinux.org 09 Aug '14

09 Aug '14

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 3e19cd366add23390ce4d0c9531b2c07e009a217 (commit) via ab07dfdeb9b4ecc443aa25f40fa530a730f65cde (commit) via 37634d22e501538e5a4b12105a0b56542841e71f (commit) via db14815f463c5128a2fc25df8d7570293987e18f (commit) via 99a5017f5591d17784ac2456b1401628f2749eea (commit) via 17ed9eb7342670a18337516312a25e6cc0b8936c (commit) via 2b556d89de0d3f10466b4db4d8915a3ffb1163de (commit) via cbd6c300b5bdf61b7df80f42cf06c69a6e4fde60 (commit) via 8d5d7f676196c9ec56dddc39de260eb40a51e843 (commit) via 8a02abcf194a7d4479c39cf0ce2dc03db9b199e1 (commit) via b694d8455453f06158f51a276c61f9d90ecc10fe (commit) via 1431d244f741b5ea2c7fc16324174d59d13ffc33 (commit) via 574e45df28769b4919c0c7e5c7cb10d521066adc (commit) via 7c75564af933ad5258b2e2bee6ce63b9bcbf6561 (commit) via 2025279eb64c397e408baab7ff686624604f415a (commit) via d981f93f18449e2cb7b67bfb417aac4737c84b57 (commit) via 4ccf49d3e74bf2af010fc90d9cd6fbeb71402f99 (commit) via 0bf4ae726d6a98142a176b9ac0b9ed8f00389e67 (commit) via 03b9bf08ac3b1e3f2c3b950ce995654af2c40503 (commit) via 7f1360b44089d69bae9a793741c0292fe91e32e4 (commit) via f4992960ad4a325184802a47dc23d53d15be452f (commit) via 0c43198ee0817d7acea47ea5d15b4704db78308f (commit) via 22e478d20322a1d224fc41866d018d4b8a774057 (commit) via 85c80542a5c52311fdde71459a924d87ec95cc93 (commit) via 32413ad44b874d622034ebd63da57159446a91d3 (commit) via dce82f9d198aed6ee1700048cd49dd3f95bdc9a7 (commit) via 7e87614665f667cc550acab3626ab6ad655541c7 (commit) from fbb0945bfb6a9535dc42a120466b44c0b39c915d (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 3e19cd366add23390ce4d0c9531b2c07e009a217 Author: Jason St. John <jstjohn(a)purdue.edu> Date: Thu Aug 7 00:43:21 2014 -0400 pkgdelta(8): Fix broken example of --min-pkg-size with human-readable values The example for human-readable values must not contain a space between the value and the unit; otherwise, pkgdelta will not recognize the command options and will error out. In prose, however, there should be a space between the value and the unit. Signed-off-by: Jason St. John <jstjohn(a)purdue.edu> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit ab07dfdeb9b4ecc443aa25f40fa530a730f65cde Author: Jason St. John <jstjohn(a)purdue.edu> Date: Thu Aug 7 00:43:20 2014 -0400 man: Improve grammar and add missing single quotes around command options Signed-off-by: Jason St. John <jstjohn(a)purdue.edu> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 37634d22e501538e5a4b12105a0b56542841e71f Author: Jason St. John <jstjohn(a)purdue.edu> Date: Thu Aug 7 00:43:19 2014 -0400 man: Use uniform line spacing between sections Remove blank lines immediately following section headings. Ensure two blank lines before the start of a new section. Signed-off-by: Jason St. John <jstjohn(a)purdue.edu> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit db14815f463c5128a2fc25df8d7570293987e18f Author: Jason St. John <jstjohn(a)purdue.edu> Date: Thu Aug 7 00:43:18 2014 -0400 pacman(8): Improve grammar and quoting Notable changes: * Add double quotes around printf-like formatting options for "--print-format" * Add a missing backslash in '--force' Signed-off-by: Jason St. John <jstjohn(a)purdue.edu> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 99a5017f5591d17784ac2456b1401628f2749eea Author: Olivier Brunel <jjk(a)jjacky.com> Date: Sat Aug 9 00:41:35 2014 +0200 Fix loading invalid package In 5acfa65f when adding a new variable (hit_mtree), the initialization of a variable (config) was dropped, which could lead to loading invalid package files, i.e. files that aren't archive (no metadata loaded) would return a new alpm_pkg_t with everything set to 0/NULL. Depending on the operation/use of the package, this could lead to segfault. Signed-off-by: Olivier Brunel <jjk(a)jjacky.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 17ed9eb7342670a18337516312a25e6cc0b8936c Author: Dave Reisner <dreisner(a)archlinux.org> Date: Sat Aug 2 20:28:05 2014 -0400 makepkg: replace bare eval with var extraction functions commit 2b556d89de0d3f10466b4db4d8915a3ffb1163de Author: Dave Reisner <dreisner(a)archlinux.org> Date: Fri Jul 25 14:01:48 2014 -0400 PKGBUILD: handle arch specific attributes This introduces support for architecture-specific conflicts, depends, optdepends, makedepends, replaces, and conflicts by appending "_$CARCH" to the array name. For example, in the global section: arch=('i686' 'x86_64') depends=('foo') depends_x86_64=('bar') This will generate depends of 'foo' and 'bar' on x86_64, but only 'foo' on i686. Moreover, this is supported in the package functions with the same heuristics as the generic names, e.g. ... arch=('i686' 'x86_64') depends=('foo') ... package_somepkg() { depends_x86_64=('bar') ... } Again, will cause x86_64 to have depends of 'foo' and 'bar', but only 'foo' for i686. commit cbd6c300b5bdf61b7df80f42cf06c69a6e4fde60 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Fri Aug 1 16:59:58 2014 -0400 makepkg: refactor check_sanity, give it some sanity of its own Break apart each of the blocks into their own separate functions. And, instead of the hand crafted eval statements, reuse the logic from pkgbuild-introspection[0] to abstract away the complexities of parsing bash. This commit fixes at least 3 bugs in check_sanity: 1) The wrong variable is shown for the error which would be thrown when, e.g. pkgname=('foopkg' 'bar^pkg') 2) The "arch" variable is not sanity checked when the PKGBUILD has an arch override, but only one output package. 3) https://bugs.archlinux.org/task/40361 Lastly, there's some string changes here which should help to clarify a few errors emitted in the linting process. [0] https://github.com/falconindy/pkgbuild-introspection commit 8d5d7f676196c9ec56dddc39de260eb40a51e843 Author: Allan McRae <allan(a)archlinux.org> Date: Fri Aug 8 13:41:27 2014 +1000 Update PKGBUILD documentation for removed override options Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 8a02abcf194a7d4479c39cf0ce2dc03db9b199e1 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Mon Aug 4 08:48:05 2014 -0400 makepkg: disallow pkgver/pkgrel/epoch overrides in packages This is a confusing feature, and no one uses it. commit b694d8455453f06158f51a276c61f9d90ecc10fe Author: Dave Reisner <dreisner(a)archlinux.org> Date: Thu Aug 7 19:22:21 2014 -0400 remove --destination-dir argument from a2x options This option is only for HTML output, and we're building manpages here. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 1431d244f741b5ea2c7fc16324174d59d13ffc33 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Aug 4 15:34:01 2014 +1000 Add Andrew to maintainers list. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 574e45df28769b4919c0c7e5c7cb10d521066adc Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Sun Apr 6 11:04:18 2014 -0400 trans_commit: log beginning and end of transaction This makes it clear whether a transaction successfully completed and allows log parsers to group related actions. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 7c75564af933ad5258b2e2bee6ce63b9bcbf6561 Author: Ryo Munakata <ryomnktml(a)gmail.com> Date: Mon Jul 14 19:15:40 2014 +0900 filecache_find_url: dereference the 'filebase' pointer Signed-off-by: Ryo Munakata <ryomnktml(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 2025279eb64c397e408baab7ff686624604f415a Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:54 2014 -0700 replace strdup with STRDUP Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit d981f93f18449e2cb7b67bfb417aac4737c84b57 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:53 2014 -0700 sync.c: fix style violations Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 4ccf49d3e74bf2af010fc90d9cd6fbeb71402f99 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:52 2014 -0700 pacman.c: simplify stdin parsing Incorporate memory exhaustion and end-of-stream checks into the main loop. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 0bf4ae726d6a98142a176b9ac0b9ed8f00389e67 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:51 2014 -0700 handle_unlock: log error when lock file is missing Problems due to concurrent running instances of pacman can be difficult to diagnose. Log a warning to make it more obvious that that's what happened, that it's a bad idea, and hopefully encourage people who do things like removing the lock file to run pacman from an install script to at least be courteous enough to put it back when they're done. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 03b9bf08ac3b1e3f2c3b950ce995654af2c40503 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:50 2014 -0700 handle_unlock: log lock removal failure Rather than have individual callers log failure, just do it directly in _alpm_handle_unlock. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 7f1360b44089d69bae9a793741c0292fe91e32e4 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:49 2014 -0700 signing.c: remove useless error check CHECK_ERR checks gpg_err which is a local variable. Calling gpg_op_import_result cannot modify it. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit f4992960ad4a325184802a47dc23d53d15be452f Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:48 2014 -0700 deps.c: use alpm_list_find_ptr Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 0c43198ee0817d7acea47ea5d15b4704db78308f Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:47 2014 -0700 dep_graph_init: filter ignored packages by name Ignored packages are from the transaction remove list which consists of duplicated packages so a direct pointer comparison is not appropriate. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 22e478d20322a1d224fc41866d018d4b8a774057 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:46 2014 -0700 pactest: delay test object creation The actual test object is only used to run the test. Storing test cases as strings limits the test object scope and allows it to be garbage collected, reducing memory usage when multiple tests are run. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 85c80542a5c52311fdde71459a924d87ec95cc93 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:45 2014 -0700 pactest: only snapshot needed files Only a few of our tests need file snapshots at all and most of them only need a few files. Taking snapshots of the entire test environment for every single test is a massive waste. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 32413ad44b874d622034ebd63da57159446a91d3 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Aug 1 14:19:44 2014 -0700 handle: factor out string list option handling Consolidates repeated code and replaces dangerous strdup calls with STRDUP. Also fix a couple variables named "pkg" that refer to file paths. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit dce82f9d198aed6ee1700048cd49dd3f95bdc9a7 Author: Allan McRae <allan(a)archlinux.org> Date: Sun Jun 29 23:23:09 2014 +1000 makepkg: skip dependency checking with --verifysource Dependencies are now handled with --nobuild unless specificially skipped. Using --verifysource will skip dependency checks unless --syncdeps is specified. Fixes FS#35057 and FS#36999. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 7e87614665f667cc550acab3626ab6ad655541c7 Author: Allan McRae <allan(a)archlinux.org> Date: Sun Jun 29 22:36:22 2014 +1000 makepkg: ensure vcs download tool are installed when required Add an array VCSCLIENTS to makepkg.conf that matches vcs source protocols to the package containing the software needed for handling the source. Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/Makefile.am | 3 +- doc/PKGBUILD.5.txt | 33 ++- doc/footer.txt | 1 + doc/makepkg-template.1.txt | 7 +- doc/makepkg.8.txt | 16 +- doc/makepkg.conf.5.txt | 10 +- doc/pacman-key.8.txt | 4 +- doc/pacman.8.txt | 14 +- doc/pacman.conf.5.txt | 13 +- doc/pactree.8.txt | 15 +- doc/pkgdelta.8.txt | 7 +- doc/repo-add.8.txt | 11 +- doc/submitting-patches.txt | 16 +- doc/translation-help.txt | 27 +-- doc/vercmp.8.txt | 7 +- lib/libalpm/alpm.h | 8 +- lib/libalpm/be_package.c | 4 +- lib/libalpm/be_sync.c | 6 +- lib/libalpm/deps.c | 13 +- lib/libalpm/diskspace.c | 6 +- lib/libalpm/dload.c | 2 +- lib/libalpm/handle.c | 115 ++++----- lib/libalpm/package.c | 2 +- lib/libalpm/signing.c | 1 - lib/libalpm/trans.c | 17 +- scripts/makepkg.sh.in | 577 ++++++++++++++++++++++++++++++++++----------- src/pacman/pacman.c | 27 +-- src/pacman/sync.c | 12 +- test/pacman/pmenv.py | 6 +- test/pacman/pmrule.py | 6 + test/pacman/pmtest.py | 17 +- 31 files changed, 667 insertions(+), 336 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH 1/4] pacman(8): Improve grammar and quoting
by Jason St. John 09 Aug '14

09 Aug '14

Notable changes: * Add double quotes around printf-like formatting options for "--print-format" * Add a missing backslash in '--force' Signed-off-by: Jason St. John <jstjohn(a)purdue.edu> --- doc/pacman.8.txt | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/doc/pacman.8.txt b/doc/pacman.8.txt index ccc0a01..28a12cf 100644 --- a/doc/pacman.8.txt +++ b/doc/pacman.8.txt @@ -123,7 +123,7 @@ Options Specify an alternative database location (a typical default is +{localstatedir}/lib/pacman+). This should not be used unless you know what you are doing. - *NOTE*: if specified, this is an absolute path, and the root path is + *NOTE*: If specified, this is an absolute path, and the root path is not automatically prepended. *-r, \--root* <path>:: @@ -176,7 +176,7 @@ Options Bypass any and all ``Are you sure?'' messages. It's not a good idea to do this unless you want to run pacman from a script. -Transaction Options (apply to '-S', '-R' and '-U') +Transaction Options (apply to '-S', '-R', and '-U') -------------------------------------------------- *-d, \--nodeps*:: Skips dependency version checks. Package names are still checked. Normally, @@ -203,8 +203,8 @@ Transaction Options (apply to '-S', '-R' and '-U') *\--print-format* <format>:: Specify a printf-like format to control the output of the '\--print' - operation. The possible attributes are: %n for pkgname, %v for pkgver, - %l for location, %r for repo, and %s for size. Implies '\--print'. + operation. The possible attributes are: "%n" for pkgname, "%v" for pkgver, + "%l" for location, "%r" for repository, and "%s" for size. Implies '\--print'. Upgrade Options (apply to '-S' and '-U')[[UO]] -------------------------------------------- @@ -212,7 +212,7 @@ Upgrade Options (apply to '-S' and '-U')[[UO]] Bypass file conflict checks and overwrite conflicting files. If the package that is about to be installed contains files that are already installed, this option will cause all those files to be overwritten. - Using '--force' will not allow overwriting a directory with a file or + Using '\--force' will not allow overwriting a directory with a file or installing packages with conflicting files and directories. This option should be used with care, ideally not at all. @@ -406,7 +406,7 @@ select sync packages whose versions do not match with the local versions. This can be useful when the user switches from a testing repository to a stable one. + Additional targets can also be specified manually, so that '-Su foo' will do a -system upgrade and install/upgrade the foo package in the same operation. +system upgrade and install/upgrade the "foo" package in the same operation. *-w, \--downloadonly*:: Retrieve all packages from the server, but do not install/upgrade anything. -- 2.0.4

2 4