- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] fix for incorrect checking of return code, which causes syntax errors
by Dieter Plaetinck 16 May '10

16 May '10

--- scripts/makepkg.sh.in | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index e8aaa8b..9741879 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -693,10 +693,10 @@ extract_sources() { local ret=0 msg2 "$(gettext "Extracting %s with %s")" "$file" "$cmd" if [[ $cmd = bsdtar ]]; then - $cmd -xf "$file" || ret=? + $cmd -xf "$file" || ret=$? else rm -f "${file%.*}" - $cmd -dcf "$file" > "${file%.*}" || ret=? + $cmd -dcf "$file" > "${file%.*}" || ret=$? fi if (( ret )); then error "$(gettext "Failed to extract %s")" "$file" -- 1.7.1

3 4

[pacman-dev] [PATCH] Fix a serious bug in the download code
by Nagy Gabor 15 May '10

15 May '10

After commit df99495b82 pacman downloaded files from the first repo only, and reported corrupted packages for all files from other repos. The download_size was set to 0 for _all_ transaction packages after downloading some files from the first repo. This code-block was moved to its correct place. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- lib/libalpm/sync.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 6bc0b37..6b625ed 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -857,12 +857,6 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data) EVENT(trans, PM_TRANS_EVT_RETRIEVE_START, current->treename, NULL); errors = _alpm_download_files(files, current->servers, cachedir); - for(j = trans->add; j; j = j->next) { - pmpkg_t *pkg = j->data; - pkg->infolevel &= ~INFRQ_DSIZE; - pkg->download_size = 0; - } - if (errors) { _alpm_log(PM_LOG_WARNING, _("failed to retrieve some files from %s\n"), current->treename); @@ -875,6 +869,12 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data) } } + for(j = trans->add; j; j = j->next) { + pmpkg_t *pkg = j->data; + pkg->infolevel &= ~INFRQ_DSIZE; + pkg->download_size = 0; + } + /* clear out value to let callback know we are done */ if(handle->totaldlcb) { handle->totaldlcb(0); -- 1.7.1

3 2

[pacman-dev] [PATCH] rankmirrors: Add a --repo option to target a specific repo
by Daenyth 15 May '10

15 May '10

Signed-off-by: Daenyth <Daenyth%2BArch(a)gmail.com> --- scripts/rankmirrors.sh.in | 19 ++++++++++++++----- 1 files changed, 14 insertions(+), 5 deletions(-) diff --git a/scripts/rankmirrors.sh.in b/scripts/rankmirrors.sh.in index d5cbadd..9b4e973 100644 --- a/scripts/rankmirrors.sh.in +++ b/scripts/rankmirrors.sh.in @@ -35,6 +35,7 @@ usage() { echo " -t, --times only output mirrors and their response times" echo " -u, --url test a specific url" echo " -v, --verbose be verbose in ouptut" + echo " -r, --repo specify a specific repo name instead of guessing" exit 0 } @@ -69,13 +70,18 @@ ARCH="$(uname -m)" getfetchurl() { local strippedurl="${1%/}" - local replacedurl="${strippedurl//'$repo'/core}" - replacedurl="${replacedurl//'$arch'/$ARCH}" + local replacedurl="${replacedurl//'$arch'/$ARCH}" + if [[ ! $TARGETREPO ]]; then + replacedurl="${strippedurl//'$repo'/core}" + local tmp="${replacedurl%/*}" + tmp="${tmp%/*}" - local tmp="${replacedurl%/*}" - tmp="${tmp%/*}" + local reponame="${tmp##*/}" + else + replacedurl="${strippedurl//'$repo'/$TARGETREPO}" + local reponame="$TARGETREPO" + fi - local reponame="${tmp##*/}" if [[ -z $reponame || $reponame = $replacedurl ]]; then echo "fail" else @@ -119,6 +125,7 @@ while [[ $1 ]]; do times) TIMESONLY=1 ; shift ;; verbose) VERBOSE=1 ; shift ;; url) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; shift 2;; + repo) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; shift 2;; *) err "\`$1' is an invalid argument." esac elif [[ ${1:0:1} = - ]]; then @@ -136,6 +143,7 @@ while [[ $1 ]]; do t) TIMESONLY=1 ;; v) VERBOSE=1 ;; u) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; snum=2;; + r) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; snum=2;; n) [[ $2 ]] || err "Must specify number." ; NUM="$2" ; snum=2;; *) err "\`-$1' is an invald argument." ;; esac @@ -182,6 +190,7 @@ for line in "${linearray[@]}"; do # Getting values and times and such server="${line#*= }" + server="${server%%#*}" url="$(getfetchurl "$server")" [[ $url = fail ]] && err "url \`$URL' is malformed." time=$(gettime "$url") -- 1.7.1

1 0

[pacman-dev] NEWS for 3.4.0 - very, very draft
by Allan McRae 15 May '10

15 May '10

A very draft NEWS item based on "git shortlog --no-merges master --not v3.3.0". May be a bit makepkg-centric, but numbers of patches for makepkg is quite high this time and many pacman patches are the "not-newsworthy" type... Anyway, cut/add entries and I can flash it out some more. pacman: -D --database option show required by in -Sii output Architecture --arch option / check package arch / $arch --print/--print-format only extract new DB entiries -Su foo util: deltacleanup rankmirror -> bash repo-add - can create "empty" repo when removing last package makepkg: new packaging variable - changelog Allow overriding pkgver, pkgrel, arch in split packages Can specify subsets of packages to build from a split package warn about references to build root in package deprecate repackaging without a package function can only use package() function strip options are configurable auto abort on any packaging error SRCPKGDEST --skipinteg syntax checking for backup and optdepends entries contrib: pactree - reverse tree

5 5

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.3.3-179-g25cd6c2
by dan＠archlinux.org 15 May '10

15 May '10

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 25cd6c2e8da4d0659d259a38f00d512f97dbc962 (commit) from f03f09011f878ad0b3aea05602a1cb0a11d839e1 (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 25cd6c2e8da4d0659d259a38f00d512f97dbc962 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Fri May 14 21:58:31 2010 +0200 Fix a serious bug in the download code After commit df99495b82 pacman downloaded files from the first repo only, and reported corrupted packages for all files from other repos. The download_size was set to 0 for _all_ transaction packages after downloading some files from the first repo. This code-block was moved to its correct place. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/sync.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.3.3-178-gf03f090
by dan＠archlinux.org 14 May '10

14 May '10

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 f03f09011f878ad0b3aea05602a1cb0a11d839e1 (commit) via 10b0acfc20589abd92b5d7517529c92e48cfcc7c (commit) via 07b2ba251f7956b887f00dadc0e2cff91777326b (commit) via df833f6c64033bc4e00e47213a536219bc1611b4 (commit) from 7f02f7cb9fba13a8e9762d30d7e0648b257ec091 (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 f03f09011f878ad0b3aea05602a1cb0a11d839e1 Author: Dan McGee <dan(a)archlinux.org> Date: Thu May 13 18:11:50 2010 -0500 doc: fix up description of where example PKGBUILD is located As Allan pointed out, this actually ships with pacman (at least with Arch) and not necessarily with ABS or any other package. Also fix the language dealing with the prototype install files. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 10b0acfc20589abd92b5d7517529c92e48cfcc7c Author: Dan McGee <dan(a)archlinux.org> Date: Thu May 13 17:46:30 2010 -0500 PKGBUILD.vim: only allow hex characters in checksums Of course, we still have only md5 and sha1 hardcoded here but I resisted the urge to copy paste for the rest of our supported checksums in hope that someone knows how to do it a better way. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 07b2ba251f7956b887f00dadc0e2cff91777326b Author: Dan McGee <dan(a)archlinux.org> Date: Thu May 13 17:26:54 2010 -0500 Add 'pkgbase' highlighting to vim syntax file Simple change and probably a bit too copy/paste, but works for now. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit df833f6c64033bc4e00e47213a536219bc1611b4 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Thu May 13 14:36:59 2010 +0200 PKGBUILD.vim : update valid licenses Updated list with : echo $(pacman -Ql licenses | grep "/usr/share/licenses/common/.*/$" | cut -d'/' -f6 ) Maybe PKGBUILD.vim could do this at runtime ? Dan: you forgot the symlinks; readded FDL, GPL, LGPL. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: PKGBUILD-split.proto | 2 +- contrib/PKGBUILD.vim | 21 +++++++++++++++------ doc/Makefile.am | 3 +++ doc/PKGBUILD.5.txt | 15 ++++++--------- 4 files changed, 25 insertions(+), 16 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] PKGBUILD man page
by Allan McRae 13 May '10

13 May '10

More as a reminder to me that I need to fix this: --begin quote-- Note If you are using Arch Linux and have a local copy of the Arch Build System (ABS) tree on your computer, or are using another distribution that provides a similar tree of build files, you can copy the provided PKGBUILD.proto file to a new package build directory and make customizations to suit your needs. An up to date prototype file can also be found in the source distribution of this package. --end quote-- The protoype is provided by the pacman package and not ABS.

3 3

[pacman-dev] [PATCH] rankmirrors: Add a --repo option to target a specific repo
by Daenyth Blank 13 May '10

13 May '10

--- scripts/rankmirrors.sh.in | 19 ++++++++++++++----- 1 files changed, 14 insertions(+), 5 deletions(-) diff --git a/scripts/rankmirrors.sh.in b/scripts/rankmirrors.sh.in index d5cbadd..54b04e0 100644 --- a/scripts/rankmirrors.sh.in +++ b/scripts/rankmirrors.sh.in @@ -35,6 +35,7 @@ usage() { echo " -t, --times only output mirrors and their response times" echo " -u, --url test a specific url" echo " -v, --verbose be verbose in ouptut" + echo " -r, --repo specify a specific repo name instead of guessing" exit 0 } @@ -69,13 +70,18 @@ ARCH="$(uname -m)" getfetchurl() { local strippedurl="${1%/}" - local replacedurl="${strippedurl//'$repo'/core}" - replacedurl="${replacedurl//'$arch'/$ARCH}" + local replacedurl="${replacedurl//'$arch'/$ARCH}" + if [[ $TARGETREPO ]]; then + replacedurl="${strippedurl//'$repo'/core}" + local tmp="${replacedurl%/*}" + tmp="${tmp%/*}" - local tmp="${replacedurl%/*}" - tmp="${tmp%/*}" + local reponame="${tmp##*/}" + else + replacedurl="${strippedurl//'$repo'/$TARGETREPO}" + local reponame="$TARGETREPO" + fi - local reponame="${tmp##*/}" if [[ -z $reponame || $reponame = $replacedurl ]]; then echo "fail" else @@ -119,6 +125,7 @@ while [[ $1 ]]; do times) TIMESONLY=1 ; shift ;; verbose) VERBOSE=1 ; shift ;; url) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; shift 2;; + repo) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; shift 2;; *) err "\`$1' is an invalid argument." esac elif [[ ${1:0:1} = - ]]; then @@ -136,6 +143,7 @@ while [[ $1 ]]; do t) TIMESONLY=1 ;; v) VERBOSE=1 ;; u) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; snum=2;; + r) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; snum=2;; n) [[ $2 ]] || err "Must specify number." ; NUM="$2" ; snum=2;; *) err "\`-$1' is an invald argument." ;; esac @@ -182,6 +190,7 @@ for line in "${linearray[@]}"; do # Getting values and times and such server="${line#*= }" + server="${server%%#*}" url="$(getfetchurl "$server")" [[ $url = fail ]] && err "url \`$URL' is malformed." time=$(gettime "$url") -- 1.7.0

3 9

[pacman-dev] [PATCH 1/2] Add 'pkgbase' highlighting to vim syntax file
by Dan McGee 13 May '10

13 May '10

Simple change and probably a bit too copy/paste, but works for now. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- PKGBUILD-split.proto | 2 +- contrib/PKGBUILD.vim | 11 ++++++++++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/PKGBUILD-split.proto b/PKGBUILD-split.proto index 0cbd563..484cb1a 100644 --- a/PKGBUILD-split.proto +++ b/PKGBUILD-split.proto @@ -5,7 +5,7 @@ # Maintainer: Your Name <youremail(a)domain.com> pkgname=('pkg1' 'pkg2') -pkgbase="" +pkgbase=BASE pkgver=VERSION pkgrel=1 pkgdesc="" diff --git a/contrib/PKGBUILD.vim b/contrib/PKGBUILD.vim index a1112e5..90b628e 100644 --- a/contrib/PKGBUILD.vim +++ b/contrib/PKGBUILD.vim @@ -25,6 +25,13 @@ syn match pbValidPkgname /$[[:alnum:]]\|+\|-\|_${,32}/ contained contains=pbIl syn match pbIllegalPkgname /[[:upper:]]\|[^[:alnum:]-+_=]\|=.*=\|=['"]\?.\{33,\}['"]\?/ contained syn match pbPkgnameGroup /^pkgname=.*/ contains=pbIllegalPkgname,pb_k_pkgname,shDoubleQuote,shSingleQuote +" pkgbase +" FIXME if '=' is in pkgbase/pkgname/pkgver, it highlights whole string, not just '=' +syn keyword pb_k_pkgbase pkgbase contained +syn match pbValidPkgbase /$[[:alnum:]]\|+\|-\|_${,32}/ contained contains=pbIllegalPkgbase +syn match pbIllegalPkgbase /[[:upper:]]\|[^[:alnum:]-+_=]\|=.*=\|=['"]\?.\{33,\}['"]\?/ contained +syn match pbPkgbaseGroup /^pkgbase=.*/ contains=pbIllegalPkgbase,pb_k_pkgbase,shDoubleQuote,shSingleQuote + " pkgver syn keyword pb_k_pkgver pkgver contained syn match pbValidPkgver /$[[:alnum:]]\|\.\|+\|_$/ contained contains=pbIllegalPkgver @@ -133,7 +140,6 @@ syn match pbDerefEmulation /\$[{]\?[[:alnum:]_]*[}]\?/ contained hi def link pbDerefEmulation PreProc " md5sums - syn keyword pb_k_md5sums md5sums contained syn match pbIllegalMd5sums /[^='"()\/ ]/ contained contains=pbValidMd5sums syn match pbValidMd5sums /[[:alnum:]]\{32\}/ contained @@ -187,6 +193,9 @@ hi def link pbTodo Todo hi def link pbIllegalPkgname Error hi def link pb_k_pkgname pbKeywords +hi def link pbIllegalPkgbase Error +hi def link pb_k_pkgbase pbKeywords + hi def link pbIllegalPkgver Error hi def link pb_k_pkgver pbKeywords -- 1.7.1

1 1

[pacman-dev] [PATCH] PKGBUILD.vim : update valid licenses
by Xavier Chantry 13 May '10

13 May '10

Updated list with : echo $(pacman -Ql licenses | grep "/usr/share/licenses/common/.*/$" | cut -d'/' -f6 ) Maybe PKGBUILD.vim could do this at runtime ? Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> --- contrib/PKGBUILD.vim | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/contrib/PKGBUILD.vim b/contrib/PKGBUILD.vim index af0f981..55a5161 100644 --- a/contrib/PKGBUILD.vim +++ b/contrib/PKGBUILD.vim @@ -54,7 +54,7 @@ syn match pbUrlGroup /^url=.*/ contains=pbValidUrl,pb_k_url,pbIllegalUrl,shDoubl " license syn keyword pb_k_license license contained -syn keyword pbLicense APACHE CDDL EPL FDL GPL LGPL MPL PHP RUBY ZLIB ISC MIT BSD contained +syn keyword pbLicense APACHE CCPL CDDL CPL EPL FDL1.2 FDL1.3 GPL2 GPL3 LGPL2.1 LGPL3 LPPL MPL PHP PSF PerlArtistic RALINK RUBY ZPL contained syn match pbLicenseCustom /custom$:[[:alnum:]]*$*/ contained syn match pbIllegalLicense /[^='"() ]/ contained contains=pbLicenseCustom,pbLicense syn region pbLicenseGroup start=/^license=(/ end=/)/ contains=pb_k_license,pbLicenseCustom,pbLicense,pbIllegalLicense -- 1.7.1

2 1