- Pacman-dev - lists.archlinux.org

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.2.2-204-g8ebc077
by dan＠archlinux.org 16 Jul '09

16 Jul '09

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 8ebc07744ac58ed1e42af3a52d998587fca85f8b (commit) via 2e043aae36bdb8a7591646db00a95dd21f1f368f (commit) from 41a55d4effecfb2914e96ed4a99eb621557c6d2f (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 8ebc07744ac58ed1e42af3a52d998587fca85f8b Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Wed Jul 8 14:00:25 2009 +0200 Offer to clean up non-package files in cache directory. This implements FS#15142. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 2e043aae36bdb8a7591646db00a95dd21f1f368f Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Wed Jul 8 11:08:32 2009 +0200 Run ldconfig inside chroot. This fixes FS#15294. The code to run a command inside a chroot was refactored from the _alpm_runscriptlet function to _alpm_run_chroot. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/add.c | 1 - lib/libalpm/remove.c | 1 - lib/libalpm/trans.c | 88 +------------------------------------------ lib/libalpm/util.c | 103 ++++++++++++++++++++++++++++++++++++++++++++++++- lib/libalpm/util.h | 1 + src/pacman/sync.c | 14 ++++--- 6 files changed, 110 insertions(+), 98 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] questions about pmhandle and pmtrans
by Xavier 16 Jul '09

16 Jul '09

The handle contains the current transaction and the local database : typedef struct _pmhandle_t { /* internal usage */ pmdb_t *db_local; /* local db pointer */ alpm_list_t *dbs_sync; /* List of (pmdb_t *) */ FILE *logstream; /* log file stream pointer */ int lckfd; /* lock file descriptor if one exists */ pmtrans_t *trans; Yet we pass the trans or db_local param to many many functions : $ egrep "pmtrans_t|db_local" lib/libalpm/*.h lib/libalpm/add.h:int _alpm_add_loadtarget(pmtrans_t *trans, pmdb_t *db, char *name); lib/libalpm/add.h:int _alpm_upgrade_packages(pmtrans_t *trans, pmdb_t *db); lib/libalpm/alpm.h:typedef struct __pmtrans_t pmtrans_t; lib/libalpm/conflict.h:alpm_list_t *_alpm_db_find_fileconflicts(pmdb_t *db, pmtrans_t *trans, lib/libalpm/handle.h: pmdb_t *db_local; /* local db pointer */ lib/libalpm/handle.h: pmtrans_t *trans; lib/libalpm/remove.h:int _alpm_remove_loadtarget(pmtrans_t *trans, pmdb_t *db, char *name); lib/libalpm/remove.h:int _alpm_remove_prepare(pmtrans_t *trans, pmdb_t *db, alpm_list_t **data); lib/libalpm/remove.h:int _alpm_remove_commit(pmtrans_t *trans, pmdb_t *db); lib/libalpm/remove.h:int _alpm_remove_packages(pmtrans_t *trans, pmdb_t *db); lib/libalpm/sync.h:int _alpm_sync_sysupgrade(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync, int enable_downgrade); lib/libalpm/sync.h:int _alpm_sync_addtarget(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync, char *name); lib/libalpm/sync.h:int _alpm_sync_prepare(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync, alpm_list_t **data); lib/libalpm/sync.h:int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data); lib/libalpm/trans.h:struct __pmtrans_t { lib/libalpm/trans.h:pmtrans_t *_alpm_trans_new(void); lib/libalpm/trans.h:void _alpm_trans_free(pmtrans_t *trans); lib/libalpm/trans.h:int _alpm_trans_init(pmtrans_t *trans, pmtransflag_t flags, lib/libalpm/trans.h:int _alpm_trans_prepare(pmtrans_t *trans, alpm_list_t **data); lib/libalpm/trans.h:int _alpm_trans_commit(pmtrans_t *trans, alpm_list_t **data); lib/libalpm/trans.h: const char *oldver, pmtrans_t *trans); So what is the preferred way here? Since we use handle->trans or handle->db_local in some places, why not everywhere? Another thing I am wondering, why these 3 callbacks are part of pmtrans : alpm_trans_cb_event cb_event; alpm_trans_cb_conv cb_conv; alpm_trans_cb_progress cb_progress; and the other callbacks are in pmhandle : /* callback functions */ alpm_cb_log logcb; /* Log callback function */ alpm_cb_download dlcb; /* Download callback function */ alpm_cb_totaldl totaldlcb; /* Total download callback function */ Why not having all callbacks in the handle? At least in pacman frontend, cb_event, cb_conv and cb_progress are the same for all transactions.

1 1

[pacman-dev] nautilus extension for packages
by K. Piche 15 Jul '09

15 Jul '09

Hi, Aaron suggested I send this here to put in contrib. Haven't made any changes since the original announcement on arch-dev-public. Regards, k -- K. Piche <kpiche(a)rogers.com>

2 2

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.2.2-202-g41a55d4
by dan＠archlinux.org 15 Jul '09

15 Jul '09

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 41a55d4effecfb2914e96ed4a99eb621557c6d2f (commit) via 5dc0b80c2683551c10b96157ba34b277db25b82d (commit) via 104daa16a69708c9b16fc59ac895f2932ea509d4 (commit) via caa0f2205a62344c4df6e286e14a0d89cd1d10f2 (commit) via 68c10690eae369928b6cdc2d658588ad06e4b1a5 (commit) via 8d7764abaecb89a5baa6b4afe25f4db8e8a40465 (commit) via 6092dda1775a3d887d557c0e4961645a4ed67064 (commit) via e72cce352a4f43e371998dc8985b5fa2e0102216 (commit) via 621aa26e26fcf578221051e896d6440a0095b8a3 (commit) via b312c820c8c8ed98a8b8f927b948bca8a3a22632 (commit) via 617e7d512ff6fb7d6c5e5232091b528148cc40c7 (commit) from c72b4543b69f5a80d254a3f408748e19db758947 (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 41a55d4effecfb2914e96ed4a99eb621557c6d2f Author: Volodia Macovei <blog(a)volodia.ro> Date: Wed Jul 1 11:06:02 2009 -0500 Update Romanian translation Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5dc0b80c2683551c10b96157ba34b277db25b82d Author: Allan McRae <allan(a)archlinux.org> Date: Sun Jul 12 17:10:11 2009 +1000 makepkg: clean up BUILDSCRIPT usage FS#15448 (which is made worse by the "fix" for FS#14727...), highlighted some deficiencies in the usage of the BUILDSCRIPT variable. In particular, only relative paths worked with "-p" and some output was very strange in combination with the "-p" flag or reading from /dev/stdin. e.g. "Please add a license line to your /dev/stdin!". This patch adds a new variable, BUILDFILE, which contains the full path to the BUILDSCRIPT. This defaults to $startdir/$BUILDSCRIPT. Also, fix a missed quoting of $BUILD{SCRIPT->FILE} and remove warning about missing BUILDSCRIPT definition in makepkg.conf as the default BUILDSCRIPT value is now specified during configure. Add check that BUILDFILE is writable before updating VCS PKGBUILDs. When making a source package, the BUILDSCRIPT always gets given the default name, regardless of what it was originally called. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 104daa16a69708c9b16fc59ac895f2932ea509d4 Author: Allan McRae <allan(a)archlinux.org> Date: Sun Jul 12 01:07:46 2009 +1000 makepkg: allow spaces in source file names The download command failed with sources that contained spaces. Remainder of fix for FS#15323. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit caa0f2205a62344c4df6e286e14a0d89cd1d10f2 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 1 02:31:59 2009 -0500 makepkg: fix breakage with '%' in source filenames Ensure we don't pass a bare filename to printf that might contain a lookalike '%' escape sequence. Fixes part of FS#15323. Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 68c10690eae369928b6cdc2d658588ad06e4b1a5 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 1 02:26:27 2009 -0500 makepkg: quote filenames when extracting We currently fall apart on files with spaces in the names. Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 8d7764abaecb89a5baa6b4afe25f4db8e8a40465 Author: Allan McRae <allan(a)archlinux.org> Date: Fri Jul 10 00:31:20 2009 +1000 Quote values in the arch array in example PKGBUILD Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 6092dda1775a3d887d557c0e4961645a4ed67064 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Jul 4 03:16:14 2009 +1000 makepkg: check for package functions when package splitting Makes sure the required package functions are present when using package splitting. Also moves setting of pkgbase variable outside the check_sanity function to somewhere more appropriate. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit e72cce352a4f43e371998dc8985b5fa2e0102216 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Jul 4 02:28:03 2009 +1000 makepkg: fix pkgdesc restoration with split packaging A pkgdesc with spaces in it would get restored to an array and thus only the first word would be restored (FS#15210). Convert that array back to a string. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 621aa26e26fcf578221051e896d6440a0095b8a3 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Jul 4 01:52:28 2009 +1000 makepkg: Exit on failure within build() or package() functions Errors in build() functions were only fatal, if "--log" was enabled. Errors in package() functions were never fatal. Piping these functions through "cat -" triggers error trapping. This prevents the need for "|| return 1" usage in PKGBUILDs. Original-patch-by: Juergen Hoetzel <juergen(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit b312c820c8c8ed98a8b8f927b948bca8a3a22632 Author: Allan McRae <allan(a)archlinux.org> Date: Thu Jul 2 13:53:29 2009 +1000 makepkg: fix check for previously built packages with package splitting Checks if some or all packages are built before overwriting/installing. Adds some new strings for translation. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 617e7d512ff6fb7d6c5e5232091b528148cc40c7 Author: Allan McRae <allan(a)archlinux.org> Date: Wed Jul 1 13:52:52 2009 +1000 makepkg: clean up moving PKGINFO creation to a function Someone forgot to commit this when pulling in the original patch to his working branch... Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/PKGBUILD-example.txt | 2 +- po/ro.po | 8 +- scripts/makepkg.sh.in | 137 ++++++++++++++++++++++++++++++---------------- 3 files changed, 95 insertions(+), 52 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] makepkg: clean up BUILDSCRIPT usage
by Allan McRae 12 Jul '09

12 Jul '09

FS#15448 (which is made worse by the "fix" for FS#14727...), highlighted some deficiencies in the usage of the BUILDSCRIPT variable. In particular, only relative paths worked with "-p" and some output was very strange in combination with the "-p" flag or reading from /dev/stdin. e.g. "Please add a license line to your /dev/stdin!". This patch adds a new variable, BUILDFILE, which contains the full path to the BUILDSCRIPT. This defaults to $startdir/$BUILDSCRIPT. Also, fix a missed quoting of $BUILD{SCRIPT->FILE} and remove warning about missing BUILDSCRIPT definition in makepkg.conf as the default BUILDSCRIPT value is not specified during configure. Add check that BUILDFILE is writeable before udpating VCS PKGBUILDs. When making a source package, the BUILDSCRIPT always gets given the default name, regardless of what it was originally called. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 40 ++++++++++++++++++++-------------------- 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 8998e0d..50cf54d 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1032,7 +1032,7 @@ create_srcpackage() { mkdir "${srclinks}"/${pkgbase} msg2 "$(gettext "Adding %s...")" "$BUILDSCRIPT" - ln -s "${startdir}/${BUILDSCRIPT}" "${srclinks}/${pkgbase}/" + ln -s "${BUILDFILE}" "${srclinks}/${pkgbase}/${BUILDSCRIPT}" if [ -n "$install" ]; then if [ -f $install ]; then @@ -1192,8 +1192,9 @@ devel_check() { newpkgver="" # Do not update pkgver if --holdver is set, when building a source package, - # or when reading PKGBUILD from pipe - if [ "$HOLDVER" -eq 1 -o "$SOURCEONLY" -ne 0 -o ! -f "./$BUILDSCRIPT" ]; then + # when reading PKGBUILD from pipe (-f), or if we cannot write to the file (-w) + if [ "$HOLDVER" -eq 1 -o "$SOURCEONLY" -ne 0 -o ! -f "$BUILDFILE" \ + -o ! -w "$BUILDFILE" ]; then return fi @@ -1262,10 +1263,10 @@ devel_update() { # if [ -n "$newpkgver" ]; then if [ "$newpkgver" != "$pkgver" ]; then - if [ -f "./$BUILDSCRIPT" ]; then - @SEDINPLACE@ "s/^pkgver=[^ ]*/pkgver=$newpkgver/" "./$BUILDSCRIPT" - @SEDINPLACE@ "s/^pkgrel=[^ ]*/pkgrel=1/" "./$BUILDSCRIPT" - source "$BUILDSCRIPT" + if [ -f "$BUILDFILE" -a -w "$BUILDFILE" ]; then + @SEDINPLACE@ "s/^pkgver=[^ ]*/pkgver=$newpkgver/" "$BUILDFILE" + @SEDINPLACE@ "s/^pkgrel=[^ ]*/pkgrel=1/" "$BUILDFILE" + source "$BUILDFILE" fi fi fi @@ -1480,7 +1481,7 @@ while true; do -L|--log) LOGGING=1 ;; -m|--nocolor) USE_COLOR='n' ;; -o|--nobuild) NOBUILD=1 ;; - -p) shift; BUILDSCRIPT=$1 ;; + -p) shift; BUILDFILE=$1 ;; -r|--rmdeps) RMDEPS=1 ;; -R|--repackage) REPKG=1 ;; --source) SOURCEONLY=1 ;; @@ -1565,11 +1566,6 @@ if [ "$CLEANCACHE" -eq 1 ]; then fi fi -if [ -z "$BUILDSCRIPT" ]; then - error "$(gettext "BUILDSCRIPT is undefined! Ensure you have updated %s.")" "$MAKEPKG_CONF" - exit 1 -fi - if [ "$INFAKEROOT" -eq 0 ]; then if [ $EUID -eq 0 -a "$ASROOT" -eq 0 ]; then # Warn those who like to live dangerously. @@ -1616,23 +1612,27 @@ unset pkgname pkgbase pkgver pkgrel pkgdesc url license groups provides unset md5sums replaces depends conflicts backup source install build unset makedepends optdepends options noextract -if [ ! -f "$BUILDSCRIPT" ]; then +BUILDFILE=${BUILDFILE:-$BUILDSCRIPT} +if [ ! -f "$BUILDFILE" ]; then if [ -t 0 ]; then - error "$(gettext "%s does not exist.")" "$BUILDSCRIPT" + error "$(gettext "%s does not exist.")" "$BUILDFILE" exit 1 else # PKGBUILD passed through a pipe - BUILDSCRIPT=/dev/stdin - source "$BUILDSCRIPT" + BUILDFILE=/dev/stdin + source "$BUILDFILE" fi else - crlftest=$(file $BUILDSCRIPT | grep -F 'CRLF' || true) + crlftest=$(file "$BUILDFILE" | grep -F 'CRLF' || true) if [ -n "$crlftest" ]; then - error "$(gettext "%s contains CRLF characters and cannot be sourced.")" "$BUILDSCRIPT" + error "$(gettext "%s contains CRLF characters and cannot be sourced.")" "$BUILDFILE" exit 1 fi - source ./"$BUILDSCRIPT" + if [ "${BUILDFILE:0:1}" != "/" ]; then + BUILDFILE="$startdir/$BUILDFILE" + fi + source "$BUILDFILE" fi if [ "$GENINTEG" -eq 1 ]; then -- 1.6.3.3

1 0

[pacman-dev] makepkg security
by Thomas Bächler 11 Jul '09

11 Jul '09

This continues a thread on arch-general: Thomas Bächler schrieb: >>> I agree. The question is not about makepkg security, but about sudo >>> security. And frankly, sudo is a security desaster in its default >>> configuration. >> >> Any suggestions for changing / shipping a better default config file? >> I know little about the security implications of this, but I think we >> should ship a decent default if possible. > > Our policy is usually to ship whatever upstream ships. IMO, a good > default would be to set sudo to require the root password (not the user > password) and not cache any passwords at all. > > Also, I think instead of using sudo in makepkg, we should use su by > default (with an option to enable sudo). su always has a good default > configuration requiring the root password (it's also possible to set it > to allow password-less su in the pam configuration, but everyone who > does that is crazy anyway). The original complaint was that when using makepkg -sic, the sudo password is cached after dependency installation and malicious sudo commands might be executed during build() as the password is cached. My opinion on this is that we should not encourage people to use sudo, Aaron suggested to move it here for further discussion. What do you think?

5 4

[pacman-dev] [PATCH] Quote values in the arch array in example PKGBUILD
by Allan McRae 09 Jul '09

09 Jul '09

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- doc/PKGBUILD-example.txt | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/doc/PKGBUILD-example.txt b/doc/PKGBUILD-example.txt index 0477cb1..7245228 100644 --- a/doc/PKGBUILD-example.txt +++ b/doc/PKGBUILD-example.txt @@ -4,7 +4,7 @@ pkgname=patch pkgver=2.5.4 pkgrel=3 pkgdesc="A utility to apply patch files to original sources" -arch=(i686 x86_64) +arch=('i686' 'x86_64') url="http://www.gnu.org/software/patch/patch.html" license=('GPL') groups=('base-devel') -- 1.6.3.3

2 3

[pacman-dev] [PATCH] New sync070.py pactest
by Nagy Gabor 09 Jul '09

09 Jul '09

From cfcaa50b83d6ce09a026e8275f19ce0665365e31 Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Wed, 2 Jul 2008 17:04:29 +0200 Subject: [PATCH] New sync070.py pactest This pactest tests the cooperation between front-end and back-end in case of "-S provision" operation. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- pactest/tests/sync070.py | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) create mode 100644 pactest/tests/sync070.py diff --git a/pactest/tests/sync070.py b/pactest/tests/sync070.py new file mode 100644 index 0000000..2177ea7 --- /dev/null +++ b/pactest/tests/sync070.py @@ -0,0 +1,15 @@ +self.description = "Advanced -S provision test" + +sp1 = pmpkg("pkg", "1.0-1") +self.addpkg2db("sync2", sp1) + +sp2 = pmpkg("pkg", "2.0-1") +sp2.provides = [ "provision" ] +sp2.conflicts = [ "pkg1=1.0-1" ] +self.addpkg2db("sync1", sp2) + +self.args = "-S provision" + +self.addrule("PACMAN_RETCODE=0") +self.addrule("PKG_EXIST=pkg") +self.addrule("PKG_VERSION=pkg|2.0-1") -- 1.5.6.1

2 6

[pacman-dev] Idea: Merge -U and -S (half universal transaction)?
by Nagy Gabor 09 Jul '09

09 Jul '09

Hi! After killing pmsyncpkg_t, there is not much difference between -S and -U transactions. The only difference is that trans->packages come from pkgcache (-S) or they are loaded from file (-U). And of course, with -S we have an extra step, we have to actually download the packages. However, -U is much more "stupid" than -S atm. Now the removes field is attached to pmpkg_t, so we could implement FS#3492 without pain, the conflict resolving part from sync.c can be reused without any modification*. Or if we allow "mixed" trans->packages list (pkg->origin can indicate that we have a file[-U] or repo[-S] package), even the implementaion of FS#5798 should be also straightforward. The only difference between PKG_FROM_CACHE and PKG_FROM_FILE packages, that PKG_FROM_CACHE needs to be downloaded. Then -U would become a sync transaction (internally), and add_commit would be a helper function only. *: I started to work on the implementation of mixed target list, and I realised that the most annoying "difference" is that PKG_FROM_CACHE packages are not duplicated, so they are mustn't be freed (the new _alpm_pkg_free_trans should be used), but PKG_FROM_FILE packages must be freed. How much memory can we spare by not duplicating sync packages (not calling _alpm_pkg_dup when adding them to transaction target)? The whole base repo is <0.5 MB atm. Does this hack "worth"? With the old behaviour, implementing this idea (and so FS#3492 and FS#5798) would be really nice. With the new behaviour it would not. (And I don't really like the need for _alpm_pkg_free_trans neither.) Bye

3 3

[pacman-dev] MirrorBrain for mirrors?
by Tim G 03 Jul '09

03 Jul '09

Hey everyone, I am not sure if it has been brought up before, but has anyone thought about providing a MirrorBrain http repository? OpenSUSE has one at http://download.opensuse.org/ and it works quite well. If we are able to provide something like this, it would mean that we could provide transparent mirroring. MirrorBrain even looks after creating mirror lists http://download.opensuse.org/distribution/11.1/iso/openSUSE-11.1-KDE4-LiveCD -i686.iso?mirrorlist Anyways, have a look at it, http://mirrorbrain.org/features -- Tim Groeneveld (Application Developer) Digital Pacific www.digitalpacific.com.au <http://www.digitalpacific.com.au/>

5 5

[pacman-dev] [PATCH] makepkg: check for package functions when package splitting
by Allan McRae 03 Jul '09

03 Jul '09

Makes sure the required package functions are present when using package splitting. Also moves setting of pkgbase variable outside the check_sanity function to somewhere more appropriate. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 14 +++++++++++--- 1 files changed, 11 insertions(+), 3 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 922187a..66287d0 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1134,9 +1134,6 @@ check_sanity() { if [ "$arch" = 'any' ]; then CARCH='any' fi - - pkgbase=${pkgbase:-${pkgname[0]}} - if ! in_array $CARCH ${arch[@]}; then if [ "$IGNOREARCH" -eq 0 ]; then error "$(gettext "%s is not available for the '%s' architecture.")" "$pkgbase" "$CARCH" @@ -1178,6 +1175,15 @@ check_sanity() { return 1 fi + if [ "${#pkgname[@]}" -gt "1" ]; then + for pkg in ${pkgname[@]}; do + if [ "$(type -t package_${pkg})" != "function" ]; then + error "$(gettext "missing package function for split package '%s'")" "$pkg" + return 1 + fi + done + fi + return 0 } @@ -1653,6 +1659,8 @@ if [ "${#pkgname[@]}" -gt "1" ]; then SPLITPKG=1 fi +pkgbase=${pkgbase:-${pkgname[0]}} + if [ "$SPLITPKG" -eq 0 ]; then if [ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ -a "$FORCE" -eq 0 -a "$SOURCEONLY" -eq 0 -a "$NOBUILD" -eq 0 ]; then -- 1.6.3.3

1 0

[pacman-dev] [PATCH] makepkg: fix pkgdesc restoration with split packaging
by Allan McRae 03 Jul '09

03 Jul '09

A pkgdesc with spaces in it would get restored to an array and thus only the first word would be restored (FS#15210). Convert that array back to a string. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- Not sure if there is a better way to do this? scripts/makepkg.sh.in | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 34c92c7..922187a 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1280,6 +1280,11 @@ restore_package_variables() { unset ${var} fi done + + # pkgdesc gets restored as an array - convert back to a string + local pkgdesc_backup="${pkgdesc[@]}" + unset pkgdesc + pkgdesc=${pkgdesc_backup} } # getopt like parser -- 1.6.3.3

1 0

[pacman-dev] [PATCH] makepkg: Exit on failure within build() or package() functions
by Allan McRae 03 Jul '09

03 Jul '09

Errors in build() functions were only fatal, if "--log" was enabled. Errors in package() functions were never fatal. Piping these functions through "cat -" triggers error trapping. This prevents the need for "|| return 1" usage in PKGBUILDs. Original-patch-by: Juergen Hoetzel <juergen(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 8 +++++--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 91e2518..34c92c7 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -717,8 +717,10 @@ run_build() { build 2>&1 | tee "$BUILDLOG"; ret=${PIPESTATUS[0]} else - build 2>&1 || ret=$? + # piping to cat ensures makepkg fails on any build error + build 2>&1 | cat -; ret=${PIPESTATUS[0]} fi + # reset our shell options eval "$shellopts" @@ -771,12 +773,12 @@ run_package() { exec 3>&1 tee "$BUILDLOG" < "$logpipe" & exec 1>"$logpipe" 2>"$logpipe" - $pkgfunc 2>&1 || ret=$? + $pkgfunc 2>&1 | cat -; ret=${PIPESTATUS[0]} sync exec 1>&3 2>&3 3>&- rm "$logpipe" else - $pkgfunc 2>&1 || ret=$? + $pkgfunc 2>&1 | cat -; ret=${PIPESTATUS[0]} fi if [ $ret -gt 0 ]; then -- 1.6.3.3

1 0

[pacman-dev] [PATCH] Ensure build failure if a single build() command fails
by Jürgen Hötzel 03 Jul '09

03 Jul '09

Hi, I doubt this was by intention: errors in build() functions are only fatal, if "--log" is enabled. I just made a buggy pkg because some "install ..." commands where not handled by "|| return 1". Our PKGBUILDs are cluttert full of "|| return 1". Failing commands in build functions should always result in an build error. Signed-off-by: Juergen Hoetzel <juergen(a)archlinux.org> --- scripts/makepkg.sh.in | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index f46b7f8..84d4599 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -703,6 +703,7 @@ run_build() { local ret=0 if [ "$LOGGING" -eq 1 ]; then BUILDLOG="${startdir}/${pkgbase}-${pkgver}-${pkgrel}-${CARCH}-build.log" + BUILDLOG_CMD="tee $BUILDLOG" if [ -f "$BUILDLOG" ]; then local i=1 while true; do @@ -714,11 +715,11 @@ run_build() { done mv "$BUILDLOG" "$BUILDLOG.$i" fi - - build 2>&1 | tee "$BUILDLOG"; ret=${PIPESTATUS[0]} else - build 2>&1 || ret=$? + BUILDLOG_CMD="cat -" fi + + build 2>&1 | ${BUILDLOG_CMD}; ret=${PIPESTATUS[0]} # reset our shell options eval "$shellopts" -- 1.6.3.2

4 7

[pacman-dev] Patches for srcpac
by Loui Chang 03 Jul '09

03 Jul '09

This seems like the most appropriate mailing list for this. Cheers!

3 7

[pacman-dev] delta package sugestions...
by Cochise - Quixote de Coração e Alma 02 Jul '09

02 Jul '09

hi guys. i'm interessed in delta packages. I'm a brazilian with a poor english, then sorry by the kill of the languege. the delta packages need 3 things in my vision... 1 - rebuild instaled packages for the upgrade no needing a permanet cacheThis huge and ugly shell line mke this. pacman -Ql kdemod-shaman | awk '{if ($1 == "kdemod-shaman") print $2}' | xargs ls -ld | grep -v '^d' | awk '{ print $9 }' | xargs tar cf shaman.pkg.tar 3 - not expend lots ofspace in the server with delta files life Fedora and SUSE. The rsync protocol update a file downoading the changed parts of then. the server unly store the lastet version, like today. 3 - Updae the rebuilded package with any server. them need use http and ftp protocols This prograns provides this. http://zsync.moria.org.uk/ Then... Its necessary in the creation of package use gzip with the --rsyncable flag and create a zsync file. When upgrading a instaled package the pacman needs an alias to rebuild the package with the --rsyncable flag too and download then with the zsync. the xdelta is more eficient in bandwidth, but the zsync is it in storage space. Bye folks.

1 0

Re: [pacman-dev] [PATCH] makepkg: fix check for previously built packages with package splitting
by Marc - A. Dahlhaus [ Administration | Westermann GmbH ] 02 Jul '09

02 Jul '09

Am Donnerstag, den 02.07.2009, 16:24 +1000 schrieb Allan McRae: > Marc - A. Dahlhaus wrote: > > Is there a way to rebuild only a subset of split packages? > > No. Mainly because I don't know of a case where this is useful. If you > adjust something in one of the package function, the repackage option > will just run the packaging steps which will be very fast. > Hello Allan, the php-mysql module is a good example if mysql-lib changes to a new so-filename you need to rebuild it and nothing alse from php. but i think in this case we would need to change the pkgver also before the check if the package exists runs. I'll better go for a separate package for this kind of split-packages... Thanks, Marc

1 0

[pacman-dev] [PATCH] makepkg: fix check for previously built packages with package splitting
by Allan McRae 02 Jul '09

02 Jul '09

Checks if some or all packages are built before overwriting/installing. Adds two new translation strings Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 41 ++++++++++++++++++++++++++++++++--------- 1 files changed, 32 insertions(+), 9 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 2bb7eb9..61ae037 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1646,15 +1646,38 @@ if [ "${#pkgname[@]}" -gt "1" ]; then SPLITPKG=1 fi -if [ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ - -a "$FORCE" -eq 0 -a "$SOURCEONLY" -eq 0 -a "$NOBUILD" -eq 0 ]; then - if [ "$INSTALL" -eq 1 ]; then - warning "$(gettext "A package has already been built, installing existing package...")" - install_package - exit $? - else - error "$(gettext "A package has already been built. (use -f to overwrite)")" - exit 1 +if [ "$SPLITPKG" -eq 0 ]; then + if [ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ + -a "$FORCE" -eq 0 -a "$SOURCEONLY" -eq 0 -a "$NOBUILD" -eq 0 ]; then + if [ "$INSTALL" -eq 1 ]; then + warning "$(gettext "A package has already been built, installing existing package...")" + install_package + exit $? + else + error "$(gettext "A package has already been built. (use -f to overwrite)")" + exit 1 + fi + fi +else + local allpkgbuilt=1 + local somepkgbuilt=0 + for pkg in ${pkgname[@]}; do + if [ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" ]; then + somepkgbuilt=1 + else + allpkgbuilt=0 + fi + done + if [ -f "$FORCE" -eq 0 -a "$SOURCEONLY" -eq 0 -a "$NOBUILD" -eq 0 ]; then + if [ "$INSTALL" -eq 1 -a "$allpkgbuilt" -eq 1 ]; then + warning "$(gettext "The package group has already been built, installing existing packages...")" + install_package + exit $? + fi + if [ "$somepkgbuilt" -eq 1 ]; then + error "$(gettext "The package group has already been (partially) built. (use -f to overwrite)")" + exit 1 + fi fi fi -- 1.6.3.3

3 7

[pacman-dev] Package signing again....
by Gerhard Brauer 01 Jul '09

01 Jul '09

Hello, i would like to push this again, as a remainder... Maybe there are now more pacman Devs with the time to continue the work on GPG signed packages. We have the threads in 12/2008 here: http://www.archlinux.org/pipermail/pacman-dev/2008-December/007761.html http://www.archlinux.org/pipermail/pacman-dev/2008-December/007808.html I'd have the impression we're on a good way there - but lack of man power... Regards Gerhard

7 7

[pacman-dev] [TRANSLATION FIXED] Romanian translation for pacman and libalpm FIXED
by volodia macovei 01 Jul '09

01 Jul '09

FIXED the errors in my translation. Now I understand how to check for errors, I think... hope to be OK now... Thanks a lot. Volodia

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.2.2-191-gc72b454
by dan＠archlinux.org 01 Jul '09

01 Jul '09

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 c72b4543b69f5a80d254a3f408748e19db758947 (commit) via be7266155fb726bbb62b6c86857273465209b586 (commit) via a21d1f99b874ebdee58b283119dd447771af0958 (commit) via 6af163dd79f21283aaa879f40fa53d415a9c1a9c (commit) via a89eae99f6b01c30a57fdbd9e4e541afee9e84af (commit) from 6d2930cc5ab95d843686b787910c6db8b6336248 (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 c72b4543b69f5a80d254a3f408748e19db758947 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 1 02:08:33 2009 -0500 Update copyright headers and messages Signed-off-by: Dan McGee <dan(a)archlinux.org> commit be7266155fb726bbb62b6c86857273465209b586 Author: Dan McGee <dan(a)archlinux.org> Date: Sat Jun 6 12:05:55 2009 -0500 Allow remove to accept 'local/' prefix See FS#14642- this allows -Qs output to be fed back into pacman without problems or having to strip off the 'local/' prefix manually. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a21d1f99b874ebdee58b283119dd447771af0958 Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Tue Jun 30 16:21:45 2009 +0200 Update NEWS for 3.3 Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 6af163dd79f21283aaa879f40fa53d415a9c1a9c Author: Volodia Macovei <blog(a)volodia.ro> Date: Tue Jun 30 23:34:18 2009 -0500 Add new Romanian translation Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a89eae99f6b01c30a57fdbd9e4e541afee9e84af Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jun 30 23:14:18 2009 -0500 Fix compile warning fail on older versions of GCC Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: NEWS | 25 +- TRANSLATORS | 4 + contrib/gensync | 4 +- doc/index.txt | 7 +- lib/libalpm/add.c | 3 +- lib/libalpm/add.h | 3 +- lib/libalpm/alpm.c | 3 +- lib/libalpm/alpm.h | 3 +- lib/libalpm/alpm_list.c | 3 +- lib/libalpm/alpm_list.h | 3 +- lib/libalpm/backup.c | 1 + lib/libalpm/backup.h | 3 +- lib/libalpm/be_package.c | 3 +- lib/libalpm/cache.c | 3 +- lib/libalpm/cache.h | 3 +- lib/libalpm/conflict.c | 3 +- lib/libalpm/conflict.h | 3 +- lib/libalpm/db.c | 3 +- lib/libalpm/db.h | 3 +- lib/libalpm/delta.c | 3 +- lib/libalpm/delta.h | 3 +- lib/libalpm/deps.c | 3 +- lib/libalpm/deps.h | 3 +- lib/libalpm/dload.c | 3 +- lib/libalpm/dload.h | 3 +- lib/libalpm/error.c | 3 +- lib/libalpm/graph.h | 3 +- lib/libalpm/group.c | 3 +- lib/libalpm/group.h | 3 +- lib/libalpm/handle.c | 3 +- lib/libalpm/handle.h | 3 +- lib/libalpm/log.c | 3 +- lib/libalpm/log.h | 3 +- lib/libalpm/md5.c | 2 +- lib/libalpm/package.c | 3 +- lib/libalpm/package.h | 3 +- lib/libalpm/po/LINGUAS | 1 + lib/libalpm/po/libalpm.pot | 6 +- lib/libalpm/po/{en_GB.po => ro.po} | 282 ++++++------ lib/libalpm/remove.c | 17 +- lib/libalpm/remove.h | 3 +- lib/libalpm/sync.c | 3 +- lib/libalpm/sync.h | 3 +- lib/libalpm/trans.c | 3 +- lib/libalpm/trans.h | 3 +- lib/libalpm/util.c | 3 +- lib/libalpm/util.h | 3 +- po/LINGUAS | 1 + po/pacman.pot | 8 +- po/{cs.po => ro.po} | 926 ++++++++++++++++++------------------ scripts/makepkg.sh.in | 6 +- scripts/pacman-optimize.sh.in | 6 +- scripts/rankmirrors.py.in | 6 +- src/pacman/callback.c | 3 +- src/pacman/callback.h | 3 +- src/pacman/conf.c | 3 +- src/pacman/conf.h | 3 +- src/pacman/deptest.c | 3 +- src/pacman/package.c | 3 +- src/pacman/package.h | 3 +- src/pacman/pacman.c | 5 +- src/pacman/pacman.h | 3 +- src/pacman/query.c | 3 +- src/pacman/remove.c | 3 +- src/pacman/sync.c | 3 +- src/pacman/upgrade.c | 3 +- src/pacman/util.c | 3 +- src/pacman/util.h | 3 +- src/util/vercmp.c | 1 + 69 files changed, 776 insertions(+), 685 deletions(-) mode change 100644 => 100755 NEWS copy lib/libalpm/po/{en_GB.po => ro.po} (52%) copy po/{cs.po => ro.po} (51%) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] Remove unused string.h from src/pacman/remove.c
by Gerardo Exequiel Pozzi 30 Jun '09

30 Jun '09

This header was used in the code for the function strdup() that is not used anymore. Signed-off-by: Gerardo Exequiel Pozzi <vmlinuz386(a)yahoo.com.ar> --- src/pacman/remove.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/pacman/remove.c b/src/pacman/remove.c index 11bc157..1968744 100644 --- a/src/pacman/remove.c +++ b/src/pacman/remove.c @@ -21,7 +21,6 @@ #include <stdlib.h> #include <stdio.h> -#include <string.h> #include <alpm.h> #include <alpm_list.h> -- 1.6.3.1

2 5

[pacman-dev] [PATCH] NEWS file updates for makepkg and contrib
by Allan McRae 30 Jun '09

30 Jun '09

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- We should really enforce that people do this when they submit patches... NEWS | 20 ++++++++++++++++++++ 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index f0a88fd..c2eb8af 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,25 @@ VERSION DESCRIPTION ----------------------------------------------------------------------------- +3.3.0 - makepkg: + - package splitting support! - see PKGBUILD-split.proto + - limit fakeroot usage with addition of package() function + - info pages are handled like man pages and not other documentation + - configuration option for man/info page directories + - added ability to automatically remove files from package + - configuration option for default LDFLAGS + - specify alternative configuration file with --config flag + - check all integrity checksums provided in PKGBUILD + - fix pkgver/pkgrel updating in SCM packages + - BUILDSCRIPT option removed from makepkg.conf. Now specified + during configure + - enforce no ">" or "<" in provides array + - package compression autodetection + - check PKGBUILD for CRLF line endings + - fix reading PKGBUILD from pipe + - various fixes to increase compatibility with BSDs & Mac OSX + - contrib + - pacscripts - print install scripts for a package + 3.2.2 - log pacsave warnings to pacman.log (FS#12531) - separate local DB creation and writing (FS#12263) - pacman-optimize: rewrite and refresh (FS#11767) -- 1.6.3.1

3 4

[pacman-dev] [TRANSLATION] Romanian translation for pacman and libalpm
by volodia macovei 30 Jun '09

30 Jun '09

I hope now is in good health and complete... two archive with files pacman_ro_RO.po and libalpm_ro_RO.po ... pls tell me if is i order...

5 7

[pacman-dev] Mime-type filtering?
by Aaron Griffin 29 Jun '09

29 Jun '09

On Mon, Mar 23, 2009 at 5:11 PM, Dan McGee <dpmcgee(a)gmail.com> wrote: > 2009/3/23 Samed Beyribey <ras0ir(a)eventualis.org>: >> Well i just noticed that, (probably) mailman rejects my attachment (it >> was 9.5k) >> so i've put it on my web site: >> http://eventualis.org/pacman/0001-fixed-typos-and-alignment-problem-in-Turk… >> >> This is my last mail about this patch, again, i feel so sorry. > > This isn't your fault. > > Aaron, do we not allow gzipped and bzipped files through? I guess not. What's the mime-type for these? Considering mailman uses a whitelist, would anyone be able to construct a decent list that covers pretty much all attachment types we'd want?

3 5