- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH 00/11] autofool cleanup
by Dave Reisner 02 Apr '12

02 Apr '12

I've been working on sorting out some crappiness in our build system, in part evidenced by an Ubuntu user attempting to compile pacman. This sort of snowballed and I took the liberty of cleaning up a few other things as well. Basically, the changes fall into a few categories: * use pkg-config for more accurate library detection * avoid touching LDFLAGS, CFLAGS, and CPPFLAGS * move files out of the repo root The last patch is none of the above, but it's good juju anyways. I definitely won't call myself an autotools guru and will point out that what I've done here is based on what I've gleaned from some other autotools-based projects. That said, I think these changes all make sense, and the Ubuntu user was certainly happy that the configure script was doing a better job of library autodetection. Dave Reisner (11): buildsys: add pkg-config m4 macros buildsys: use pkg-config for openssl detection buildsys: use pkg-config for libarchive detection buildsys: use pkg-config for libcurl detection buildsys: handle gpgme libs and cflags separately buildsys: use AC_DEFINE for CYGWIN macro buildsys: define warning CFLAGS in separate var buildsys: cleanup libtool and autoconf initialization buildsys: move acinclude.m4 to m4/ buildsys: move autotools bloat to build-aux autoclean.sh: abide by POSIX shebang autoclean.sh | 24 +-- config.guess => build-aux/config.guess | 0 config.rpath => build-aux/config.rpath | 0 config.sub => build-aux/config.sub | 0 depcomp => build-aux/depcomp | 0 install-sh => build-aux/install-sh | 0 ltmain.sh => build-aux/ltmain.sh | 0 missing => build-aux/missing | 0 mkinstalldirs => build-aux/mkinstalldirs | 0 configure.ac | 63 ++++---- lib/libalpm/Makefile.am | 19 ++- acinclude.m4 => m4/acinclude.m4 | 0 m4/libcurl.m4 | 250 ------------------------------ m4/pkg.m4 | 159 +++++++++++++++++++ src/pacman/Makefile.am | 2 +- src/util/Makefile.am | 4 +- 16 files changed, 220 insertions(+), 301 deletions(-) rename config.guess => build-aux/config.guess (100%) rename config.rpath => build-aux/config.rpath (100%) rename config.sub => build-aux/config.sub (100%) rename depcomp => build-aux/depcomp (100%) rename install-sh => build-aux/install-sh (100%) rename ltmain.sh => build-aux/ltmain.sh (100%) rename missing => build-aux/missing (100%) rename mkinstalldirs => build-aux/mkinstalldirs (100%) rename acinclude.m4 => m4/acinclude.m4 (100%) delete mode 100644 m4/libcurl.m4 create mode 100644 m4/pkg.m4 -- 1.7.9.5

1 11

[pacman-dev] Fwd: Re: Unable to compile pacman in ubuntu-11.10
by Dave Reisner 02 Apr '12

02 Apr '12

----- Forwarded message from Dave Reisner <d(a)falconindy.com> ----- > Date: Fri, 30 Mar 2012 14:44:05 -0400 > From: Dave Reisner <d(a)falconindy.com> > To: Talespin Kit <talespin_kit(a)yahoo.com> > Subject: Re: [pacman-dev] Unable to compile pacman in ubuntu-11.10 > > On Fri, Mar 30, 2012 at 05:12:11AM -0700, Talespin Kit wrote: > > Finally managed to compile in ubuntu-11.10. > > > > Seems the problem was the Makefile generated by the configure script for the > > **libalpm ** > > does not add the **libcrypto** as dependency in the LIBS variable. > > > > Below is the steps. > > > > I have used the git commit e6f72c61a098b52ea29e54b8eb4739a2ff81e6b0 in > > repo git://projects.archlinux.org/pacman.git > > > > 1) $ git clean -xfd; ./autogen.sh && ./configure --prefix=/usr --sysconfdir=/ > > etc --localstatedir=/var \ > > --enable-git-version --enable-debug --without-gpgme > > > > 2) cd lib/libalpm > > > > 3) In the Makefile replace line > > > > LIBS = -lssl -larchive -lm > > with > > LIBS = -lssl -larchive -lm -lcrypto > > 4) cd ../../ && make > > > > top-posted HTML... awesome... I've made some alterations to the build > system and dropped them in a branch in my repo: > > git clone git://code.falconindy.com/pacman.git buildsys > > If you could check out this branch and see if you have any more luck > with auto configuration, it'd be appreciated. I tested this on my 10.04 > LTS install with: > > ./autogen.sh > ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var > --enable-git-version --enable-debug > > curl and ssl were properly detected for the build. If it does succeed, > I'd like to see the output of: > > readelf -d ./lib/libalpm/.libs/libalpm.so > > Thanks, > dave > ----- End forwarded message ----- Forwarding to list because I didnt reply-all... d

2 4

[pacman-dev] [PATCH 1/4] pacman-key: lsign keys in --populate without prompting
by Pierre Schmitz 31 Mar '12

31 Mar '12

There is no gain in security when we ask the user to type in "y" on every single key. It also makes scripting harder. Signed-off-by: Pierre Schmitz <pierre(a)archlinux.de> --- 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 482b56d..32c70dc 100644 --- a/scripts/pacman-key.sh.in +++ b/scripts/pacman-key.sh.in @@ -275,7 +275,7 @@ populate_keyring() { msg "$(gettext "Locally signing trusted keys in keyring...")" for key_id in "${!trusted_ids[@]}"; do msg2 "$(gettext "Locally signing key %s...")" "${key_id}" - "${GPG_PACMAN[@]}" --quiet --lsign-key "${key_id}" + "${GPG_PACMAN[@]}" --quiet --batch --yes --lsign-key "${key_id}" done msg "$(gettext "Importing owner trust values...")" for keyring in "${KEYRINGIDS[@]}"; do -- 1.7.9.4

2 8

[pacman-dev] Unable to compile pacman in ubuntu-11.10
by Talespin Kit 30 Mar '12

30 Mar '12

Hi, I have tried to do the Offline Installation as described in the ArchLinux wiki, but it did not worked out as i have mentioned in the thread http://stackoverflow.com/posts/comments/12588344. Hence i have planned to compile the pacman in a machine which has internet connection and then download the packages using the compiled pacman binary and then transfer it to offline ArchLinux Machine. While i tried to compile the pacman in Ubuntu-11.10 machine i got the following errors $ make # after ./autogen.sh && ./configure make all-recursive make[1]: Entering directory `/home/talespin/archlinux/pacman' Making all in lib/libalpm make[2]: Entering directory `/home/talespin/archlinux/pacman/lib/libalpm' Making all in po make[3]: Entering directory `/home/talespin/archlinux/pacman/lib/libalpm/po' test ! -f ./libalpm.pot || \ test -z "ca.gmo cs.gmo da.gmo de.gmo el.gmo en_GB.gmo es.gmo fi.gmo fr.gmo hu.gmo it.gmo kk.gmo lt.gmo nb.gmo pl.gmo pt.gmo pt_BR.gmo ro.gmo ru.gmo sk.gmo sr.gmo sr(a)latin.gmo sv.gmo tr.gmo uk.gmo zh_CN.gmo zh_TW.gmo" || make ca.gmo cs.gmo da.gmo de.gmo el.gmo en_GB.gmo es.gmo fi.gmo fr.gmo hu.gmo it.gmo kk.gmo lt.gmo nb.gmo pl.gmo pt.gmo pt_BR.gmo ro.gmo ru.gmo sk.gmo sr.gmo sr(a)latin.gmo sv.gmo tr.gmo uk.gmo zh_CN.gmo zh_TW.gmo make[4]: Entering directory `/home/talespin/archlinux/pacman/lib/libalpm/po' : --update --lang=ca ca.po libalpm.pot rm -f ca.gmo && : -c --statistics --verbose -o ca.gmo ca.po mv: cannot stat `t-ca.gmo': No such file or directory make[4]: *** [ca.gmo] Error 1 make[4]: Leaving directory `/home/talespin/archlinux/pacman/lib/libalpm/po' make[3]: *** [stamp-po] Error 2 make[3]: Leaving directory `/home/talespin/archlinux/pacman/lib/libalpm/po' make[2]: *** [all-recursive] Error 1 make[2]: Leaving directory `/home/talespin/archlinux/pacman/lib/libalpm' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/talespin/archlinux/pacman' make: *** [all] Error 2 I have cloned the source from git://projects.archlinux.org/pacman.git and tried compiling with the commit e6f72c61a098b52ea29e54b8eb4739a2ff81e6b0 and its parents(until commit ae25167bcd592186749b79ea31b10fb78ed9fb2d) with no success. Any ideas. Br, Talespin

5 10

[pacman-dev] [translation] pacman 4.0.3 string freeze
by Dan McGee 28 Mar '12

28 Mar '12

Only very minor updates, very few strings have changed. More info: http://www.archlinux.org/pacman/translation-help.html Transifex page: https://www.transifex.net/projects/p/archlinux-pacman/r/4-0-3/ Due by: 2012-04-02 Thanks! -Dan

1 0

[pacman-dev] [PATCH] Check minimum required gpgme version
by Allan McRae 28 Mar '12

28 Mar '12

We use interfaces first introduced in gpgme-1.3.0 so test we have at least that version. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index b44e8ba..0fe3287 100644 --- a/configure.ac +++ b/configure.ac @@ -179,7 +179,7 @@ require_gpgme=no AS_IF([test "x$with_gpgme" != "xno"], [AS_IF([test "x$with_gpgme" = "xyes"], [require_gpgme=yes]) - AM_PATH_GPGME([], + AM_PATH_GPGME([1.3.0], [LIBS_save="$LIBS" CPPFLAGS_save="$CPPFLAGS" CFLAGS_save="$CFLAGS" -- 1.7.9.5

1 0

[pacman-dev] [PATCH] signing: cope with gpg2's failure at life
by Dave Reisner 28 Mar '12

28 Mar '12

For key searches only, gpg2 will fail to lookup any and all keys that are not prefixed with 0x. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- lib/libalpm/signing.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index 3ec957d..ef9f080 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -234,6 +234,8 @@ static int key_search(alpm_handle_t *handle, const char *fpr, gpgme_keylist_mode_t mode; gpgme_key_t key; int ret = -1; + size_t fpr_len; + char *full_fpr; memset(&ctx, 0, sizeof(ctx)); err = gpgme_new(&ctx); @@ -248,14 +250,21 @@ static int key_search(alpm_handle_t *handle, const char *fpr, _alpm_log(handle, ALPM_LOG_DEBUG, "looking up key %s remotely\n", fpr); - err = gpgme_get_key(ctx, fpr, &key, 0); + /* gpg2 goes full retard here. For key searches ONLY, we need to prefix the + * key fingerprint with 0x, or the lookup will fail. */ + fpr_len = strlen(fpr); + MALLOC(full_fpr, fpr_len + 3, RET_ERR(handle, ALPM_ERR_MEMORY, -1)); + sprintf(full_fpr, "0x%s", fpr); + + err = gpgme_get_key(ctx, full_fpr, &key, 0); + if(gpg_err_code(err) == GPG_ERR_EOF) { _alpm_log(handle, ALPM_LOG_DEBUG, "key lookup failed, unknown key\n"); /* Try an alternate lookup using the 8 character fingerprint value, since * busted-ass keyservers can't support lookups using subkeys with the full * value as of now. This is why 2012 is not the year of PGP encryption. */ - if(strlen(fpr) > 8) { - const char *short_fpr = fpr + strlen(fpr) - 8; + if(fpr_len - 2 > 8) { + const char *short_fpr = memcpy(&full_fpr[fpr_len - 8], "0x", 2); _alpm_log(handle, ALPM_LOG_DEBUG, "looking up key %s remotely\n", short_fpr); err = gpgme_get_key(ctx, short_fpr, &key, 0); @@ -268,6 +277,8 @@ static int key_search(alpm_handle_t *handle, const char *fpr, } } + free(full_fpr); + if(gpg_err_code(err) != GPG_ERR_NO_ERROR) { goto error; } -- 1.7.9.5

1 0

[pacman-dev] [PATCH] this patch is to add zsh completion to pacman-key
by Daniel.wallace＠gatech.edu 26 Mar '12

26 Mar '12

From: Daniel Wallace <daniel.wallace12(a)gmail.com> Sorry about my first email, hopefully this one will be in a better format and without the entire first paragraph in the subject line. The first thing I did was instead of running _pacman "$@" it now runs _pacman_comp "$@". This function uses a switch statement on $service so if pacman is the command it runs _pacman "$@" like it does right now, but if the command is pacman-key, _pacman_key "$@" is run instead. This function auto completes all the pacman-key flags and also provides the description next to the arguments. It also allows for completion of dirs for arguments like --gpgdir, and autocompletes .sig files for --verify, and gpg files for --import. This patch will also allow for keyids and emails to be auto completed from pacman-key --list-keys and other commands which use keyids. the _keys() function is the one that does this, it #uses the first \< to the end and remove first \> to the end to get all the emails #the using the first \/ to end then remove first '\ ' to the end this gets all the keyids (pub and sub) the ##uid* removes all the .jpeg pictures with ids, and ##pubbring.gpg removes the beginning line that lists where stuff is stored, ##\-\-* removes the line of dashes at the beginning, but doesn't remove email addresses with dashes in them. Signed-off-by: Daniel Wallace <daniel.wallace12(a)gmail.com> --- contrib/zsh_completion.in | 56 ++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 53 insertions(+), 3 deletions(-) diff --git a/contrib/zsh_completion.in b/contrib/zsh_completion.in index b30e960..608fa0c 100644 --- a/contrib/zsh_completion.in +++ b/contrib/zsh_completion.in @@ -1,4 +1,4 @@ -#compdef pacman pacman.static=pacman +#compdef pacman pacman.static=pacman pacman-key # copy this file to /usr/share/zsh/site-functions/_pacman @@ -332,5 +332,55 @@ _pacman() { esac } -# run the main dispatcher -_pacman "$@" +_pacman_key() { + _arguments \ + '(- :)'{-h,--help}"[show help]" \ + '(-a --add)*'{-a,--add}"[Add the specified keys (empty for stdin)]: :_files" \ + '(-d --delete)*'{-d,--delete}"[Remove the Specified keyids]: :_keys" \ + '(-e --export)*'{-e,--export}"[Export the specified or all keyids]: :_keys" \ + '(-f --finger)*'{-f,--finger}"[List fingreprint for specidied or all keyids]: :_keys" \ + '(-l --list-keys)*'{-l,--list-keys}"[List the specified or all keys]: :_keys" \ + '(-r --recv-keys)*'{-r,--recv-keys}"[Fetch the specified keyids]: :_keys" \ + '(-u --updatedb)*'{-u,--updatedb}"[Update the trustdb of pacman]" \ + '(-v --verify)*'{-v,--verify}"[Verify the file specified by the signature]: :_files -g '*.sig'" \ + '(-V --version)*'{-V,--version}"[Show program version]" \ + '(--config)*'{*,--config}"[Use an alternate config file]: :_files" \ + '(--edit-key)*'{*,--edit-key}"[Present a menu for key management task on keyids]: :_keys" \ + '(--gpgdir)*'{*,--gpgdir}"[Set an alternate directory for GnuPG (instead of '/etc/pacman.d/gnupg')]: :_files -/" \ + '(--import)*'{*,--import}"[Imports pubring.gpg from dir(s)]: :_files -g '*.gpg'" \ + '(--import-trustdb)*'{*,--import-tb}"[Imports ownertrust values from trustdb.gpg in dir(s)]: :_files -g '*.gpg'" \ + '(--init)*'{*,--init}"[Ensure the keyring is properly initialized]" \ + '(--keyserver)*'{*,--keyserver}"[Specify a keyserver to use if necessary]" \ + '(--list-sigs)*'{*,--list-sigs}"[List keys and their signatures]: :_keys" \ + '(--lsign-key)*'{*,--lsign-key}"[Locally sign the specified keyid]: :_keys" \ + '(--populate)*'{*,--populate}"[Reload the default keys from the (given) keyrings in '/usr/share/pacman/keyrings']: :_path_files -W /usr/share/pacman/keyrings" \ + '(--refresh-keys)*'{*,--refresh-keys}"[Update specified or all keys from a keyserver]: :_keys" +} + +_keys() { + local keys + #uses the first \< to the end and remove first \> to the end to get all the emails + #the using the first \/ to end then remove first '\ ' to the end this gets all the keyids (pub and sub) + #the ##uid* removes all the .jpeg pictures with ids, and ##pubbring.gpg removes the beginning line that lists + #where stuff is stored, ##\-\-* removes the line of dashes at the beginning, but doesn't remove email addresses + #with dashes in them + keys=( ${${${${${${${${${(f)"$(pacman-key --list-keys 2>/dev/null)"}}##*\<}%%\>}##*\/}%%\ *}##uid*}##pubring.gpg}##\-\-*} ) + _describe -t modules 'keys in keyring' keys && return 0 +} + +_pacman_comp() { + case "$service" in + pacman) + _pacman "$@";; + pacman-key) + _pacman_key "$@";; + *) + _message "Error";; + esac +} + +_pacman_comp "$@" + + + +# vim: ft=zsh sw=2 ts=2 et -- 1.7.9.4

1 0

[pacman-dev] [PATCH 164/164] pacman-key The first thing I did was instead of running _pacman "$@" it now runs _pacman_comp "$@". This function uses a switch statement on $service so if pacman is the command it runs _pacman "$@" like it does right now, but if the command is pacman-key, _pacman_key "$@" is run instead. This function auto completes all the pacman-key flags and also provides the description next to the arguments. It also allows for completion of dirs for arguments like --gpgdir, and autocomplet
by Daniel Wallace 26 Mar '12

26 Mar '12

This patch is for zsh_completions for pacman-key The first thing I did was instead of running _pacman "$@" it now runs _pacman_comp "$@". This function uses a switch statement on $service so if pacman is the command it runs _pacman "$@" like it does right now, but if the command is pacman-key, _pacman_key "$@" is run instead. This function auto completes all the pacman-key flags and also provides the description next to the arguments. It also allows for completion of dirs for arguments like --gpgdir, and autocompletes .sig files for --verify, and gpg files for --import. This patch will also allow for keyids and emails to be auto completed from pacman-key --list-keys and other commands which use keyids. the _keys() function is the one that does this, it #uses the first \< to the end and remove first \> to the end to get all the emails #the using the first \/ to end then remove first '\ ' to the end this gets all the keyids (pub and sub) the ##uid* removes all the .jpeg pictures with ids, and ##pubbring.gpg removes the beginning line that lists where stuff is stored, ##\-\-* removes the line of dashes at the beginning, but doesn't remove email addresses with dashes in them. Signed-off-by: Daniel Wallace <daniel.wallace12(a)gmail.com> --- contrib/zsh_completion.in | 56 ++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 53 insertions(+), 3 deletions(-) diff --git a/contrib/zsh_completion.in b/contrib/zsh_completion.in index b30e960..608fa0c 100644 --- a/contrib/zsh_completion.in +++ b/contrib/zsh_completion.in @@ -1,4 +1,4 @@ -#compdef pacman pacman.static=pacman +#compdef pacman pacman.static=pacman pacman-key # copy this file to /usr/share/zsh/site-functions/_pacman @@ -332,5 +332,55 @@ _pacman() { esac } -# run the main dispatcher -_pacman "$@" +_pacman_key() { + _arguments \ + '(- :)'{-h,--help}"[show help]" \ + '(-a --add)*'{-a,--add}"[Add the specified keys (empty for stdin)]: :_files" \ + '(-d --delete)*'{-d,--delete}"[Remove the Specified keyids]: :_keys" \ + '(-e --export)*'{-e,--export}"[Export the specified or all keyids]: :_keys" \ + '(-f --finger)*'{-f,--finger}"[List fingreprint for specidied or all keyids]: :_keys" \ + '(-l --list-keys)*'{-l,--list-keys}"[List the specified or all keys]: :_keys" \ + '(-r --recv-keys)*'{-r,--recv-keys}"[Fetch the specified keyids]: :_keys" \ + '(-u --updatedb)*'{-u,--updatedb}"[Update the trustdb of pacman]" \ + '(-v --verify)*'{-v,--verify}"[Verify the file specified by the signature]: :_files -g '*.sig'" \ + '(-V --version)*'{-V,--version}"[Show program version]" \ + '(--config)*'{*,--config}"[Use an alternate config file]: :_files" \ + '(--edit-key)*'{*,--edit-key}"[Present a menu for key management task on keyids]: :_keys" \ + '(--gpgdir)*'{*,--gpgdir}"[Set an alternate directory for GnuPG (instead of '/etc/pacman.d/gnupg')]: :_files -/" \ + '(--import)*'{*,--import}"[Imports pubring.gpg from dir(s)]: :_files -g '*.gpg'" \ + '(--import-trustdb)*'{*,--import-tb}"[Imports ownertrust values from trustdb.gpg in dir(s)]: :_files -g '*.gpg'" \ + '(--init)*'{*,--init}"[Ensure the keyring is properly initialized]" \ + '(--keyserver)*'{*,--keyserver}"[Specify a keyserver to use if necessary]" \ + '(--list-sigs)*'{*,--list-sigs}"[List keys and their signatures]: :_keys" \ + '(--lsign-key)*'{*,--lsign-key}"[Locally sign the specified keyid]: :_keys" \ + '(--populate)*'{*,--populate}"[Reload the default keys from the (given) keyrings in '/usr/share/pacman/keyrings']: :_path_files -W /usr/share/pacman/keyrings" \ + '(--refresh-keys)*'{*,--refresh-keys}"[Update specified or all keys from a keyserver]: :_keys" +} + +_keys() { + local keys + #uses the first \< to the end and remove first \> to the end to get all the emails + #the using the first \/ to end then remove first '\ ' to the end this gets all the keyids (pub and sub) + #the ##uid* removes all the .jpeg pictures with ids, and ##pubbring.gpg removes the beginning line that lists + #where stuff is stored, ##\-\-* removes the line of dashes at the beginning, but doesn't remove email addresses + #with dashes in them + keys=( ${${${${${${${${${(f)"$(pacman-key --list-keys 2>/dev/null)"}}##*\<}%%\>}##*\/}%%\ *}##uid*}##pubring.gpg}##\-\-*} ) + _describe -t modules 'keys in keyring' keys && return 0 +} + +_pacman_comp() { + case "$service" in + pacman) + _pacman "$@";; + pacman-key) + _pacman_key "$@";; + *) + _message "Error";; + esac +} + +_pacman_comp "$@" + + + +# vim: ft=zsh sw=2 ts=2 et -- 1.7.9.4

1 0

[pacman-dev] [PATCH] checkupdates: add new contrib script
by keenerd 25 Mar '12

25 Mar '12

checkupdates provides a way of checking for new updates without touching the local sync database. Based heavily on a script by Dave Reisner. --- contrib/.gitignore | 1 + contrib/Makefile.am | 3 +++ contrib/checkupdates.in | 44 ++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 48 insertions(+), 0 deletions(-) create mode 100644 contrib/checkupdates.in diff --git a/contrib/.gitignore b/contrib/.gitignore index 19b81e0..36c0918 100644 --- a/contrib/.gitignore +++ b/contrib/.gitignore @@ -1,5 +1,6 @@ bacman bash_completion +checkupdates paccache pacdiff paclist diff --git a/contrib/Makefile.am b/contrib/Makefile.am index eca39e7..1ead042 100644 --- a/contrib/Makefile.am +++ b/contrib/Makefile.am @@ -6,6 +6,7 @@ bin_SCRIPTS = \ OURSCRIPTS = \ bacman \ + checkupdates \ paccache \ pacdiff \ paclist \ @@ -22,6 +23,7 @@ EXTRA_DIST = \ PKGBUILD.vim \ bacman.in \ bash_completion.in \ + checkupdates.in \ paccache.in \ paclog-pkglist.in \ pacdiff.in \ @@ -76,6 +78,7 @@ uninstall-local: bacman: $(srcdir)/bacman.in bash_completion: $(srcdir)/bash_completion.in +checkupdates: $(srcdir)/checkupdates.in paccache: $(srcdir)/paccache.in pacdiff: $(srcdir)/pacdiff.in paclist: $(srcdir)/paclist.in diff --git a/contrib/checkupdates.in b/contrib/checkupdates.in new file mode 100644 index 0000000..e16378f --- /dev/null +++ b/contrib/checkupdates.in @@ -0,0 +1,44 @@ +#!/bin/bash +# +# checkupdates: Safely print a list of pending updates. +# +# Copyright (c) 2011 Kyle Keen <keenerd(a)gmail.com> +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. +# + +if (( $# > 0 )); then + echo "checkupdate: Safely print a list of pending updates." + echo "Use: checkupdates" + echo "Export the 'CHECKUPDATE_DB' variable to change the path of the temporary database." + exit 0 +fi + +if [[ -z $CHECKUPDATE_DB ]]; then + CHECKUPDATE_DB="${TMPDIR:-/tmp}/checkup-db-${USER}/" +fi + +trap 'rm -f $CHECKUPDATE_DB/db.lck' INT TERM EXIT + +eval $(awk '/DBPath/ {print $1$2$3}' /etc/pacman.conf) +DBPath="${DBPath:-@localstatedir@/lib/pacman/}" + +mkdir -p "$CHECKUPDATE_DB" +ln -s "${DBPath}/local" "$CHECKUPDATE_DB" &> /dev/null +fakeroot pacman -Sqy --dbpath "$CHECKUPDATE_DB" &> /dev/null +pacman -Qqu --dbpath "$CHECKUPDATE_DB" 2> /dev/null + +exit 0 + +# vim: set ts=2 sw=2 noet: -- 1.7.9

3 2

[pacman-dev] [PATCH] Make selecting scriptlet shell work
by Allan McRae 23 Mar '12

23 Mar '12

The initial patch to implement this achieved nothing apart from adding a configure option. This patch makes that configure option do what it advertises. Bug-found-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> --- lib/libalpm/trans.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index fe89e40..4309c07 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -343,7 +343,7 @@ int _alpm_runscriptlet(alpm_handle_t *handle, const char *filepath, _alpm_log(handle, ALPM_LOG_DEBUG, "executing \"%s\"\n", cmdline); - retval = _alpm_run_chroot(handle, "/bin/sh", argv); + retval = _alpm_run_chroot(handle, SCRIPTLET_SHELL, argv); cleanup: if(scriptfn && unlink(scriptfn)) { -- 1.7.9.4

2 2

[pacman-dev] [PATCH] libalpm: add pkg-config file
by Dave Reisner 20 Mar '12

20 Mar '12

Borrowed heavily from libarchive. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- Throw a bone to the few projects who link against us... configure.ac | 1 + lib/libalpm/Makefile.am | 3 +++ lib/libalpm/libalpm.pc.in | 11 +++++++++++ 3 files changed, 15 insertions(+) create mode 100644 lib/libalpm/libalpm.pc.in diff --git a/configure.ac b/configure.ac index 12a6e65..9093a90 100644 --- a/configure.ac +++ b/configure.ac @@ -382,6 +382,7 @@ AC_DEFINE_UNQUOTED([SCRIPTLET_SHELL], "$SCRIPTLET_SHELL", [The shell used to run AC_CONFIG_FILES([ lib/libalpm/Makefile lib/libalpm/po/Makefile.in +lib/libalpm/libalpm.pc src/pacman/Makefile src/pacman/po/Makefile.in src/util/Makefile diff --git a/lib/libalpm/Makefile.am b/lib/libalpm/Makefile.am index 61dcb87..17d9a58 100644 --- a/lib/libalpm/Makefile.am +++ b/lib/libalpm/Makefile.am @@ -23,6 +23,9 @@ if ENABLE_GNU89_INLINE_CC AM_CFLAGS += -fgnu89-inline endif +pkgconfigdir = $(libdir)/pkgconfig +pkgconfig_DATA = libalpm.pc + libalpm_la_SOURCES = \ add.h add.c \ alpm.h alpm.c \ diff --git a/lib/libalpm/libalpm.pc.in b/lib/libalpm/libalpm.pc.in new file mode 100644 index 0000000..12bea28 --- /dev/null +++ b/lib/libalpm/libalpm.pc.in @@ -0,0 +1,11 @@ +prefix=@prefix@ +exec_prefix=@exec_prefix@ +libdir=@libdir@ +includedir=@includedir@ + +Name: libalpm +Description: Arch Linux package management library +Version: @VERSION@ +Cflags: -I${includedir} +Libs: -L${libdir} -lalpm +Libs.private: @LIBS@ -- 1.7.9.4

2 1

[pacman-dev] Debug symbol packages bikeshedding
by Allan McRae 20 Mar '12

20 Mar '12

I have been looking at the patches for automatic creation of packages with debug symbols. I think they are good starts, but before I polish them we need to decide on a few things. My preferences are highlighted with a (*) 1) the option in makepkg.conf and PKGBUILD options=() array - debug (*) - debugpkg - splitdbg - splitdebug 2) the naming of the package with debug symbols - foo-dbg - foo-debug (*) - foo-debuginfo - foo-debug-symbols (Note -dbg is Debian style, -debuginfo is RPM style, and -debug potentially conflicts with a few packages in the AUR but none in the Arch repos...) 3) what information the .PKGINFO file has: pkgdesc="Debugging symbols for $pkgname" Must be included: - pkgname - pkgver/pkgrel/epoch/arch as in parent package - depends=(parent=$epoch:$pkgver-$pkgrel) - license Should be included? - url - replaces - conflicts Do not include: - groups - provides - backup - install - changelog Allan

2 1

[pacman-dev] The great VCS packages overhaul of 2015
by Allan McRae 18 Mar '12

18 Mar '12

It is well know that the VCS package support in makepkg is subpar... The number of bugs about this is into double figures. There have also been some patches submitted attempting to be able to use the source=() array in PKGBUILDs to specify a VCS repo and remove a lot of the repetitive crap involved for repo checkout/updating in the current PKGBUILDs. To get the ball rolling again, I think we should pick one VCS system and flesh out what we need and what the prototype PKGBUILD would look like. Then we can move on to the other VCS systems and finally implement it. I guess by the time all the bikeshedding is finished, this will all be done by 2015 and hence the subject line. :) I am going to start with git. Current: pkgver = the date the package was built _gitroot = the url of the git repo _gitname = the name of the directory we check out the repo into... According to the man page, _gitname is supposed to be a branch or tag but that is all lies. What would we like to have for a flexible git package implementation? - url of the repo - be able to specify the branch/tag/commit to use (and appropriate combinations) - a decent pkgver 1) URL: There were previous patches to the mailing list that never really got finished, but I think we were fairly happy with this syntax: source=(git://projects.archlinux.org/pacman.git) source=(git@@http://projects.archlinux.org/git/pacman.git) Does it make any sense to allow the "::" syntax here? i.e. source=(git@@dirname::http://projects.archlinux.org/git/pacman.git) where dirname is the name of the directory it checked out into? I am thinking we should probably do the checkout into $vcsdir=$startdir/vcs, so this would only be needed if #1 - we supported multiple VCS checkouts in one PKGBUILD there were two that wanted to used the same name... #2 - another source file conflicted #2 is readily dealt with and I am not sure we should allow #1 (see below) so I would vote to skip it until a genuine need is shown. 2) Specifying commit to work with: I think that this is the difficult bit... the syntax with the source array is already convoluted enough, so I do not think they should be added there. So that suggests we go with assigning them to variables like _git_branch, _git_commit, _git_tag... etc. But what if we have two git sources? For example, say pacman allowed building against an internal copy of libarchive if a folder named libarchive was found in its root directory. So: source=(git://projects.archlinux.org/pacman.git git://github.com/libarchive/libarchive) build() { cd $srcdir/pacman ln -s ../libarchive ./autogen.sh ... It might seem a somewhat convoluted example, but (e.g.) gcc does allow in source tree building of many of its dependencies. The question is should we consider this outside the realms of the reasonable and state one VCS repo should be one package. I'd say 99.999% of VCS PKGBUILDs (at a lower bound) would never use two VCS sources... and the ones that do need this could do manual checking out of the non-main source within the PKGBUILD anyway. 3) pkgver Use output of "git describe" (with a s/-/_/) and fall back to "git rev-list HEAD | wc -l" (with a trailing commit id added) if there are no tags in the repo. PROTOTYPE: pkgname=pacman-git pkgver=AUTO ... source=(git://projects.archlinux.org/users/allan/pacman.git) _git_branch="working" _git_commit="8c857343" build() { cd $srcdir/pacman ./autogen.sh ... } What makepkg does: 1) goes into $vcsdir, checks for the pacman directory - if not present, do the git checkout - if present, enter and do a "git pull" unless --holdver is specified 2) enters $srcdir, and does the appropriate clone of the repo in $vscdir to be at the required branch/tag/commit 3) starts build() etc... Oh wow... you are still reading this? Well then, you are now qualified to comment on the proposal. Please stick to just concerns with the git usage at the moment, unless you see something monumental that we will not be able to support using other VCS systems using this approach. Allan

6 24

[pacman-dev] [PATCH 1/3] Pass a file descriptor to getcols and flush_term_input
by Dan McGee 16 Mar '12

16 Mar '12

This makes these methods a bit more flexible. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- src/pacman/callback.c | 4 ++-- src/pacman/util.c | 31 ++++++++++++++++--------------- src/pacman/util.h | 2 +- 3 files changed, 19 insertions(+), 18 deletions(-) diff --git a/src/pacman/callback.c b/src/pacman/callback.c index 9fd9cc4..61f517c 100644 --- a/src/pacman/callback.c +++ b/src/pacman/callback.c @@ -384,7 +384,7 @@ void cb_progress(alpm_progress_t event, const char *pkgname, int percent, int len, wclen, wcwid, padwid; wchar_t *wcstr; - const unsigned short cols = getcols(); + const unsigned short cols = getcols(fileno(stdout)); if(config->noprogressbar || cols == 0) { return; @@ -544,7 +544,7 @@ void cb_dl_progress(const char *filename, off_t file_xfered, off_t file_total) const char *rate_label, *xfered_label; int file_percent = 0, total_percent = 0; - const unsigned short cols = getcols(); + const unsigned short cols = getcols(fileno(stdout)); if(config->noprogressbar || cols == 0 || file_total == -1) { if(file_xfered == 0) { diff --git a/src/pacman/util.c b/src/pacman/util.c index a5242ae..15cd6fd 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -130,10 +130,10 @@ int check_syncdbs(size_t need_repos, int check_valid) } /* discard unhandled input on the terminal's input buffer */ -static int flush_term_input(void) { +static int flush_term_input(int fd) { #ifdef HAVE_TCFLUSH - if(isatty(fileno(stdin))) { - return tcflush(fileno(stdin), TCIFLUSH); + if(isatty(fd)) { + return tcflush(fd, TCIFLUSH); } #endif @@ -142,24 +142,24 @@ static int flush_term_input(void) { } /* gets the current screen column width */ -unsigned short getcols(void) +unsigned short getcols(int fd) { const unsigned short default_tty = 80; const unsigned short default_notty = 0; unsigned short termwidth = 0; - if(!isatty(fileno(stdout))) { + if(!isatty(fd)) { return default_notty; } -#ifdef TIOCGSIZE +#if defined(TIOCGSIZE) struct ttysize win; - if(ioctl(1, TIOCGSIZE, &win) == 0) { + if(ioctl(fd, TIOCGSIZE, &win) == 0) { termwidth = win.ts_cols; } #elif defined(TIOCGWINSZ) struct winsize win; - if(ioctl(1, TIOCGWINSZ, &win) == 0) { + if(ioctl(fd, TIOCGWINSZ, &win) == 0) { termwidth = win.ws_col; } #endif @@ -259,7 +259,7 @@ void indentprint(const char *str, size_t indent) wchar_t *wcstr; const wchar_t *p; int len, cidx; - const unsigned short cols = getcols(); + const unsigned short cols = getcols(fileno(stdout)); if(!str) { return; @@ -622,7 +622,7 @@ int table_display(const char *title, const alpm_list_t *header, totalwidth = table_calc_widths(header, rows, padding, totalcols, &widths, &has_data); /* return -1 if terminal is not wide enough */ - if(totalwidth > getcols()) { + if(totalwidth > getcols(fileno(stdout))) { pm_printf(ALPM_LOG_WARNING, _("insufficient columns available for table display\n")); return -1; @@ -660,7 +660,7 @@ void list_display(const char *title, const alpm_list_t *list) if(!list) { printf("%s\n", _("None")); } else { - const unsigned short maxcols = getcols(); + const unsigned short maxcols = getcols(fileno(stdout)); size_t cols = len; const char *str = list->data; fputs(str, stdout); @@ -1377,7 +1377,7 @@ int multiselect_question(char *array, int count) break; } - flush_term_input(); + flush_term_input(fileno(stdin)); if(fgets(response, response_len, stdin)) { const size_t response_incr = 64; @@ -1440,7 +1440,7 @@ int select_question(int count) break; } - flush_term_input(); + flush_term_input(fileno(stdin)); if(fgets(response, sizeof(response), stdin)) { size_t len = strtrim(response); @@ -1463,6 +1463,7 @@ static int question(short preset, char *fmt, va_list args) { char response[32]; FILE *stream; + int fd_in = fileno(stdin); if(config->noconfirm) { stream = stdout; @@ -1489,7 +1490,7 @@ static int question(short preset, char *fmt, va_list args) } fflush(stream); - flush_term_input(); + flush_term_input(fd_in); if(fgets(response, sizeof(response), stdin)) { size_t len = strtrim(response); @@ -1499,7 +1500,7 @@ static int question(short preset, char *fmt, va_list args) /* if stdin is piped, response does not get printed out, and as a result * a \n is missing, resulting in broken output (FS#27909) */ - if(!isatty(fileno(stdin))) { + if(!isatty(fd_in)) { fprintf(stream, "%s\n", response); } diff --git a/src/pacman/util.h b/src/pacman/util.h index 0df46e8..fc97707 100644 --- a/src/pacman/util.h +++ b/src/pacman/util.h @@ -47,7 +47,7 @@ int trans_init(alpm_transflag_t flags, int check_valid); int trans_release(void); int needs_root(void); int check_syncdbs(size_t need_repos, int check_valid); -unsigned short getcols(void); +unsigned short getcols(int fd); int rmrf(const char *path); const char *mbasename(const char *path); char *mdirname(const char *path); -- 1.7.9.4

1 2

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.0.2-185-ge6f72c6
by dan＠archlinux.org 16 Mar '12

16 Mar '12

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 e6f72c61a098b52ea29e54b8eb4739a2ff81e6b0 (commit) via f988aa6b32503f5d4003b1402089df74adf8b485 (commit) via 573bd80f147ce90f21ef51fb8ca6f71e89cc3a74 (commit) via 44de3183ff42dc4e5972a9d563ebc7125c3eb4dd (commit) via 03465ad6b6719e0afb20ca8829581190a2b7658e (commit) via d521f30f071e2bd2c39c05f39df34be867dfe572 (commit) via 9e1a20022d8186eba0005e20b8b6db16cf7dec64 (commit) via 56f821b72a6b56cdcbc727515e43c16affd39f08 (commit) via 930e93b8925874a3c6d3460d2c0cebcda5fc0d09 (commit) via 89f4dd88d784632fed134ae88284074511cf4567 (commit) via 83e42dcccef08233134a0a5da7e3da70638e6c97 (commit) via f41037358af167424403d30a3b972b1fd733c755 (commit) via 952ee5e6cb8bb1849c1bceedae38744935c82fca (commit) via 47d0df6c101455c989887551bd0f0149a0f00de7 (commit) via 5bda38196fbd6ad389fd829924736b007732e2f1 (commit) via ff58e5cb2d6aca0e1c8ed0e77770f3c5d2442c46 (commit) from 8de0631edbd1fd7681f6a924f5e166621328c13f (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 e6f72c61a098b52ea29e54b8eb4739a2ff81e6b0 Merge: 573bd80 f988aa6 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Mar 16 12:08:10 2012 -0500 Merge branch 'maint' Conflicts: lib/libalpm/sync.c commit 573bd80f147ce90f21ef51fb8ca6f71e89cc3a74 Author: Allan McRae <allan(a)archlinux.org> Date: Fri Mar 9 17:27:43 2012 +1000 makepkg: abort when failing to create BUILDDIR Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 44de3183ff42dc4e5972a9d563ebc7125c3eb4dd Author: Allan McRae <allan(a)archlinux.org> Date: Fri Mar 9 17:15:28 2012 +1000 makepkg: prevent issues with files starting with a hyphen Most places in makepkg deal with full file paths, but a few use the file name only. Protect from potential issues when a file name starts with a hyphen. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 03465ad6b6719e0afb20ca8829581190a2b7658e Author: Allan McRae <allan(a)archlinux.org> Date: Fri Mar 9 16:01:28 2012 +1000 makepkg: append pkgbase to BUILDDIR if specified This means multiple packages can be build using the same BUILDDIR without their source files all ending up extracted in the same directory. Fixes FS#28417. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d521f30f071e2bd2c39c05f39df34be867dfe572 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Mar 13 06:33:56 2012 -0500 pactest: make OPTDEPEND rule look at non-description only Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 9e1a20022d8186eba0005e20b8b6db16cf7dec64 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Mar 14 22:19:13 2012 -0400 Always display download size if it applies Currently, we try to do a bunch of funkyness constraining download size to print only when doing a -S/--sync operation. However, it is possible we try to download packages on a -U/--upgrade operation, and we currently won't show any itemized download sizes. Fix this ommission by always including the download size stuff in the built table rows; this column will be completely omitted anyway if there are no values due to prior work in commit 33bb7dbd35. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 56f821b72a6b56cdcbc727515e43c16affd39f08 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Mar 14 22:18:37 2012 -0400 Handle printing NULL correctly in table display Treat this value as the empty string. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 930e93b8925874a3c6d3460d2c0cebcda5fc0d09 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Mar 9 14:24:48 2012 -0600 Prefer fputs/fputc over fprintf when writing local database Using fputs should be faster as no format string parsing is required. It also prevents silly errors related to unescaped '%' signs, and removes the need to double them up in a lot of places. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 89f4dd88d784632fed134ae88284074511cf4567 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Mar 9 14:10:04 2012 -0600 Extract a write_deps function from local database writing This reduces a lot of code duplication in the write function, which cleans it up a bit. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 83e42dcccef08233134a0a5da7e3da70638e6c97 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Mar 9 13:40:55 2012 -0600 Remove unnecessary path check from local database This removes a call to _alpm_local_db_pkgpath() as well as an access() call when reading the local database. This appears to be code from 2006 that has stuck around. We don't need it because: 1) We never use this path except to check it via access(); however, we are already in a readdir() loop so it exists, or at least did at the time of the call. 2) The fopen() and other calls will fail on accessing the database files anyway, and we need to check those for errors. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5bda38196fbd6ad389fd829924736b007732e2f1 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Mar 12 21:47:29 2012 -0500 Minor format-string related cleanups We had one stubbed out so we didn't require a translation update, and the other is more a code style issue. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/index.txt | 17 +++++-- lib/libalpm/be_local.c | 106 ++++++++++++++++++------------------------------ lib/libalpm/dload.c | 5 ++ lib/libalpm/sync.c | 10 +++- scripts/makepkg.sh.in | 25 ++++++++---- src/pacman/sync.c | 17 ++++++++ src/pacman/util.c | 34 +++++++-------- test/pacman/pmrule.py | 7 ++- 8 files changed, 118 insertions(+), 103 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v4.0.2-25-gf988aa6
by dan＠archlinux.org 16 Mar '12

16 Mar '12

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, maint has been updated via f988aa6b32503f5d4003b1402089df74adf8b485 (commit) via f41037358af167424403d30a3b972b1fd733c755 (commit) via 952ee5e6cb8bb1849c1bceedae38744935c82fca (commit) via 47d0df6c101455c989887551bd0f0149a0f00de7 (commit) via ff58e5cb2d6aca0e1c8ed0e77770f3c5d2442c46 (commit) from 1a8c3e52d70bfa21ba108aa77179adf90401949d (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 f988aa6b32503f5d4003b1402089df74adf8b485 Author: Allan McRae <allan(a)archlinux.org> Date: Fri Mar 9 17:32:11 2012 +1000 makepkg: quote removed filename as it can have spaces Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f41037358af167424403d30a3b972b1fd733c755 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Wed Mar 14 23:06:24 2012 -0400 lib/sync: avoid unwanted unlink after download fail In case we have a mirror failure, unlink_on_fail would remain set, causing an interrupt in a successive download attempt to be wrongly unlinked. This also fixes a memory leak in the url member, as we would allocate over the previous, unfreed URL. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> commit 952ee5e6cb8bb1849c1bceedae38744935c82fca Author: Dave Reisner <dreisner(a)archlinux.org> Date: Wed Mar 14 23:04:43 2012 -0400 dload: reset payload filename members before download To avoid conflicts on reusing a payload after a failed download, ensure that we reset the filename hints in the payload struct prior to the download operation. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 47d0df6c101455c989887551bd0f0149a0f00de7 Author: Allan McRae <allan(a)archlinux.org> Date: Fri Mar 9 23:16:40 2012 +1000 Skip special files when cleaning package cache Ignore *.sig, *.db*, and *.src.tar* when cleaning the package cache. Fixes FS#25166. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit ff58e5cb2d6aca0e1c8ed0e77770f3c5d2442c46 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Mar 12 21:16:14 2012 -0500 Add 3.0.x versions to doc/index.txt These got lost when I switched to the new Asciidoc tables syntax in commit 1ebe5dc1979e. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/index.txt | 17 ++++++++++++----- lib/libalpm/dload.c | 5 +++++ lib/libalpm/sync.c | 2 ++ scripts/makepkg.sh.in | 2 +- src/pacman/sync.c | 17 +++++++++++++++++ 5 files changed, 37 insertions(+), 6 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH 1/2] dload: reset payload filename members before download
by Dave Reisner 15 Mar '12

15 Mar '12

To avoid conflicts on reusing a payload after a failed download, ensure that we reset the filename hints in the payload struct prior to the download operation. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- lib/libalpm/dload.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 0598806..b3ac024 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -390,6 +390,11 @@ static int curl_download_internal(struct dload_payload *payload, CURL *curl = get_libcurl_handle(handle); handle->pm_errno = 0; + /* make sure these are NULL */ + FREE(payload->tempfile_name); + FREE(payload->destfile_name); + FREE(payload->content_disp_name); + payload->tempfile_openmode = "wb"; if(!payload->remote_name) { STRDUP(payload->remote_name, get_filename(payload->fileurl), -- 1.7.9.4

1 1

[pacman-dev] [PATCH 1/2] Handle printing NULL correctly in table display
by Dan McGee 15 Mar '12

15 Mar '12

Treat this value as the empty string. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- src/pacman/util.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/pacman/util.c b/src/pacman/util.c index 7be3dc5..a96aa62 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -500,6 +500,9 @@ static void table_print_line(const alpm_list_t *line, short col_padding, } value = curcell->data; + if(!value) { + value = ""; + } /* silly printf requires padding size to be an int */ cell_padding = (int)widths[i] - (int)string_length(value); if(cell_padding < 0) { -- 1.7.9.4

1 1

[pacman-dev] [PATCH 1/2] Revert "makepkg: calculate exact total file size"
by Dave Reisner 15 Mar '12

15 Mar '12

btrfs's cow snapshots seem to do very strange things, causing du to not count data still in filesystem buffers. Repackaging the same build of handbrake within a chroot on brynhild, I witnessed (via bash xtrace) du reporting a different $pkgdir size every time the write_pkginfo() function ran. Unfortunately, replacing du with stat has its own slew of problems, mostly due to hard links (e.g. git, with 106 hardlinks to the same file). Working around this is neither fun, nor practical. As it turns out, all we needed here all along was a simple call to sync to flush writes to disk before calling du. This reverts commit b264fb9e9ddcc31dc8782390309421965e507383. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- configure.ac | 1 + scripts/Makefile.am | 1 + scripts/makepkg.sh.in | 7 ++++++- 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 12a6e65..52f8ad4 100644 --- a/configure.ac +++ b/configure.ac @@ -282,6 +282,7 @@ esac AM_CONDITIONAL([CYGWIN], test "x$host_os_cygwin" = "xyes") AM_CONDITIONAL([DARWIN], test "x$host_os_darwin" = "xyes") +AC_PATH_PROGS([DUPATH], [du], [du], [/usr/bin$PATH_SEPARATOR/bin] ) AC_SUBST(SIZECMD) AC_SUBST(SEDINPLACE) AC_SUBST(STRIP_BINARIES) diff --git a/scripts/Makefile.am b/scripts/Makefile.am index 727de25..328fbff 100644 --- a/scripts/Makefile.am +++ b/scripts/Makefile.am @@ -53,6 +53,7 @@ edit = sed \ -e 's|@BUILDSCRIPT[@]|$(BUILDSCRIPT)|g' \ -e 's|@SIZECMD[@]|$(SIZECMD)|g' \ -e 's|@SEDINPLACE[@]|$(SEDINPLACE)|g' \ + -e 's|@DUPATH[@]|$(DUPATH)|g' \ -e 's|@SCRIPTNAME[@]|$@|g' \ -e 's|@configure_input[@]|Generated from $@.sh.in; do not edit by hand.|g' diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 89cd118..c2941b2 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1156,7 +1156,8 @@ write_pkginfo() { else local packager="Unknown Packager" fi - local size="$(find . -print0 | xargs -0 @SIZECMD@ | awk '{ sum += $1 } END { print sum }')" + local size="$(@DUPATH@ -sk)" + size="$(( ${size%%[^0-9]*} * 1024 ))" msg2 "$(gettext "Generating %s file...")" ".PKGINFO" echo "# Generated by makepkg $myver" @@ -1264,6 +1265,10 @@ create_package() { PKGARCH=$CARCH fi + # flush buffers to disk. This avoids problems with du calculating the wrong + # installed package size, particularly on btrfs CoW snapshots. + sync + write_pkginfo $nameofpkg > .PKGINFO local comp_files=('.PKGINFO') -- 1.7.9.4

2 4

[pacman-dev] [PATCH 0/6] VCS versioning modifications
by dgbaley27＠0x01b.net 13 Mar '12

13 Mar '12

From: Matthew Monaco <matthew.monaco(a)0x01b.net> I've been bothered by makepkg automatically adjusting pkgver if a package doesn't have the correct -git/svn/etc suffix. The first 4 patches might be useful regardless. They allow for multiple sets of _vcs variables in a PKGBUILD and use the package's name to clear up any ambiguity. The 5th patch changes the current semantics of makepkg. The 6th patch provides a new option to force the old semantics. An alternative might be to sanitize the --forcever option so that it may be used from the command line as well as internally. Matthew Monaco (6): makepkg: remove unused variable makepkg: devel_check(): determine vcs makepkg: devel_check(): consolidate common code makepkg: devel_check(): set newpkgver (cleanup) makepkg: devel_check(): only work for -vcs pkgnames makepkg: add --devver option for SCM packages doc/PKGBUILD.5.txt | 3 +- doc/makepkg.8.txt | 4 ++ scripts/makepkg.sh.in | 126 +++++++++++++++++++++++++++---------------------- 3 files changed, 75 insertions(+), 58 deletions(-) -- 1.7.9.3

5 22

[pacman-dev] [PATCH v3 0/5] development package cleanups
by Matthew Monaco 13 Mar '12

13 Mar '12

From: Matthew Monaco <matthew.monaco(a)0x01b.net> Some cleanup in the devel_check() function. The semantics aren't really changed here. Guard against the potential to specify multiple groups of development directives. Matthew Monaco (5): makepkg: remove unused variable makepkg: devel_check(): determine vcs makepkg: devel_check(): consolidate common code makepkg: devel_check(): set newpkgver (cleanup) makepkg: devel_check(): cleanup for hg version scripts/makepkg.sh.in | 117 +++++++++++++++++++++++++------------------------ 1 file changed, 59 insertions(+), 58 deletions(-) -- 1.7.9.3

2 6

[pacman-dev] [PATCH 0/6] fix some safety issues in makepkg
by Allan McRae 12 Mar '12

12 Mar '12

With the upcoming makepkg-lobotomy (removing the error trap apart from in the actual packaging steps), we need to add some more error checking and be as safe as possible when potentially dealing with strange variable contents. Some of these are a bit unnecessary (e.g. using printf to output variables that almost certainly cannot start with a "-n"), but there is no harm in applying them globally. Allan McRae (6): makepkg: the rhs in string comparisons should be quoted makepkg: use printf rather than echo to output variable makepkg: prevent issues with files starting with a hyphen makepkg: abort when failing to create BUILDDIR makepkg: quote removed filename as it can have spaces makepkg: safely change directories scripts/makepkg.sh.in | 147 ++++++++++++++++++++++++++----------------------- 1 file changed, 79 insertions(+), 68 deletions(-) -- 1.7.9.3

4 19

[pacman-dev] [PATCH 1/2] makepkg: fix extraction of soname in find_libdepends
by Florian Pritz 10 Mar '12

10 Mar '12

libperl.so results in soname="libperl.so.so" which is wrong. This returns the correct string: "libperl.so" Fix-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Florian Pritz <bluewind(a)xinu.at> --- This should be applied to both master and maint. scripts/makepkg.sh.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 6e7a556..9301730 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1071,7 +1071,7 @@ find_libdepends() { for sofile in $(LC_ALL=C readelf -d "$filename" 2>/dev/null | sed -nr 's/.*Shared library: \[(.*)\].*/\1/p') do # extract the library name: libfoo.so - soname="${sofile%%\.so\.*}.so" + soname="${sofile%.so?(+(.+([0-9])))}".so # extract the major version: 1 soversion="${sofile##*\.so\.}" if in_array "${soname}" ${depends[@]}; then -- 1.7.9.2

2 3

[pacman-dev] [PATCH v2] Skip special files when cleaning package cache
by Allan McRae 10 Mar '12

10 Mar '12

Ignore *.sig, *.db*, and *.src.tar* when cleaning the package cache. Fixes FS#25166. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- Why didn't I know about fnmatch before... so much easier than the regex stuff I was doing before! src/pacman/sync.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/src/pacman/sync.c b/src/pacman/sync.c index e69ca58..1df2a6f 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -26,6 +26,7 @@ #include <errno.h> #include <dirent.h> #include <sys/stat.h> +#include <fnmatch.h> #include <alpm.h> #include <alpm_list.h> @@ -222,6 +223,23 @@ static int sync_cleancache(int level) if(strcmp(ent->d_name, ".") == 0 || strcmp(ent->d_name, "..") == 0) { continue; } + + /* skip signature files - they are removed with their package file */ + if(fnmatch("*.sig", ent->d_name, 0) == 0) { + continue; + } + + /* skip package database within the cache directory */ + if(fnmatch("*.db*", ent->d_name, 0) == 0) { + continue; + } + + /* skip source packages withing the cache directory */ + if(fnmatch("*.src.tar*", ent->d_name, 0) == 0) { + continue; + } + + /* build the full filepath */ snprintf(path, PATH_MAX, "%s%s", cachedir, ent->d_name); -- 1.7.9.3

1 0