- Pacman-dev - lists.archlinux.org

Re: [pacman-dev] Bugs in pacman and yaourt bash completion
by Dan McGee 09 Aug '08

09 Aug '08

On Sat, Aug 9, 2008 at 4:38 PM, Jim Pryor <jim(a)jimpryor.net> wrote: > Hi, I've found a bug in the /etc/bash_completion.d/{pacman,yaourt} > files. > > The bug is in the function _available_groups, which is included in both > files. The second-to-last line of the function (line 70 in pacman, line > 71 in yaourt) looks like this: > > available_groups=$(...do sed '/%GROUPS%/...) > > That sed should have a '-n' flag. CCing to pacman-dev. Can you open a bug for this instead of a private email, or at least email the mailing list next time? Thanks.

1 0

Re: [pacman-dev] [arch-dev-public] info pages
by Allan McRae 09 Aug '08

09 Aug '08

Eric Belanger wrote: > On Wed, 6 Aug 2008, Allan McRae wrote: > >> Allan McRae wrote: >>> Hi all, >>> >>> Just a heads up with something I have remembered while packaging >>> with the new pacman release. Including docs is the default in >>> makepkg.conf now so if there is a large amount of documentation you >>> will want to add a !docs to the PKGBUILD options. Also, every >>> package that needs the --mandir configure flag will probably need >>> and --infodir one too. An annoying number of packages with info >>> pages install a /usr/share/info/dir file so that will need removed >>> at the end of the PKGBUILD. >>> >> >> I have discovered that at least one package has to have a >> /usr/share/info/dir file or the info command complains that it is >> missing. It seems that ideally all packages with info directories >> should add/remove entries in this file when installing/removing the >> package, although calling "info <something not in dir file>" seems to >> work, but I might be missing something.... The point is that adding >> info files properly involves an install file with "install-info" >> commands for each info page in the package. Isn't including docs fun! >> >> Allan >> > > The emacs package already does this. The emacs.install file could be > used as a template for this info stuff. > A template based in that is making its way into the prototypes found in the ABS packages which I am going to make a release of soon. CC: pacman-dev - We should also automatically compress the info files like we do the man pages. Allan

2 2

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.2.0-8-g57bd897
by dan＠archlinux.org 09 Aug '08

09 Aug '08

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 57bd8974c73571d827617f4df0a6917f13696493 (commit) via d8f8a126658a25fdad5e9fb00b1b0838a7e93c68 (commit) via 57393eb730641ca912a421b70897fcd289c44b21 (commit) via f201f107dbee25294aa7c7f1139bf521e6afa267 (commit) from 72c5a298a3ee0f18019010ef2eb43da654ec25f8 (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 57bd8974c73571d827617f4df0a6917f13696493 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Aug 9 15:54:29 2008 +1000 Fix creation of source package with local source files Fixes FS#11149. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit d8f8a126658a25fdad5e9fb00b1b0838a7e93c68 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Aug 8 22:18:00 2008 -0500 Fix segfault on x86_64 when using UseSyslog Due to differences in handling va_list between i686 and x86_64, this bug can only be seen on x86_64. va_list usage is not allowed but we had been getting away with it. See http://lists.opensuse.org/opensuse-programming/2008-02/msg00005.html for details and explanation. This fixes FS#11096. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 57393eb730641ca912a421b70897fcd289c44b21 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Aug 8 13:45:46 2008 -0500 Fix variable naming issues in _alpm_db_cmp When this function got a rewrite in commit f43805d875, argument and variable names got a bit mixed up when separating the casts from the strcmp operation. Fix the mixup which also fixes a possible segfault when this function is called. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f201f107dbee25294aa7c7f1139bf521e6afa267 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 7 13:11:20 2008 -0500 doc: allow asciidoc to format the note By doing the bolding manually, it doesn't look as cool in the HTML generated manpages. Let asciidoc do the work. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/pacman.conf.5.txt | 5 +++-- lib/libalpm/db.c | 4 ++-- lib/libalpm/db.h | 2 +- lib/libalpm/util.c | 11 +++++++++-- scripts/makepkg.sh.in | 2 +- 5 files changed, 16 insertions(+), 8 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.2.0-8-g57bd897
by dan＠archlinux.org 09 Aug '08

09 Aug '08

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 57bd8974c73571d827617f4df0a6917f13696493 (commit) via d8f8a126658a25fdad5e9fb00b1b0838a7e93c68 (commit) via 57393eb730641ca912a421b70897fcd289c44b21 (commit) via f201f107dbee25294aa7c7f1139bf521e6afa267 (commit) from 72c5a298a3ee0f18019010ef2eb43da654ec25f8 (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 57bd8974c73571d827617f4df0a6917f13696493 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Aug 9 15:54:29 2008 +1000 Fix creation of source package with local source files Fixes FS#11149. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit d8f8a126658a25fdad5e9fb00b1b0838a7e93c68 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Aug 8 22:18:00 2008 -0500 Fix segfault on x86_64 when using UseSyslog Due to differences in handling va_list between i686 and x86_64, this bug can only be seen on x86_64. va_list usage is not allowed but we had been getting away with it. See http://lists.opensuse.org/opensuse-programming/2008-02/msg00005.html for details and explanation. This fixes FS#11096. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 57393eb730641ca912a421b70897fcd289c44b21 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Aug 8 13:45:46 2008 -0500 Fix variable naming issues in _alpm_db_cmp When this function got a rewrite in commit f43805d875, argument and variable names got a bit mixed up when separating the casts from the strcmp operation. Fix the mixup which also fixes a possible segfault when this function is called. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f201f107dbee25294aa7c7f1139bf521e6afa267 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 7 13:11:20 2008 -0500 doc: allow asciidoc to format the note By doing the bolding manually, it doesn't look as cool in the HTML generated manpages. Let asciidoc do the work. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/pacman.conf.5.txt | 5 +++-- lib/libalpm/db.c | 4 ++-- lib/libalpm/db.h | 2 +- lib/libalpm/util.c | 11 +++++++++-- scripts/makepkg.sh.in | 2 +- 5 files changed, 16 insertions(+), 8 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] Fix creation of source package with local source files
by Allan McRae 09 Aug '08

09 Aug '08

Fixes FS#11149. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 3604d10..c5dbed9 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -957,7 +957,7 @@ create_srcpackage() { local file=$(strip_url "$netfile") if [ -f "$netfile" ]; then msg2 "$(gettext "Adding %s...")" "$netfile" - ln -s $netfile "${srclinks}/${pkgname}" + ln -s "${startdir}/$netfile" "${srclinks}/${pkgname}" elif [ "$SOURCEONLY" = "2" -a -f "$SRCDEST/$file" ]; then msg2 "$(gettext "Adding %s...")" "$file" ln -s "$SRCDEST/$file" "${srclinks}/${pkgname}/" -- 1.5.6.5

1 0

[pacman-dev] [BUG] alpm_list_remove need an initalized pointer for void **data
by solsTiCe d'Hiver 08 Aug '08

08 Aug '08

hi. i am playing with alpm and db4 to make a little program for me. just for fun. and in this new release of libalpm (along pacman 3.2.0) i have found that alpm_list_remove behave differently than before. so the last parameter (void **data) need to be initialized to avoid a seg fault. it happens in alpm_db_unregister if you unregister a sync db if you run this, you will see it happen #include <stdio.h> #include <stdlib.h> #include <alpm.h> int main(void) { pmdb_t *db; pmpkg_t *pkg; const char *s; alpm_initialize(); alpm_option_set_root("/"); alpm_option_set_dbpath("/var/lib/pacman"); alpm_option_add_cachedir("/var/cache/pacman/pkg"); alpm_option_set_logfile("/dev/stdout"); db = alpm_db_register_sync("extra"); puts("alpm_db_unregister(db)"); alpm_db_unregister(db); puts("alpm_release()"); alpm_release(); exit(EXIT_SUCCESS); } i wonder how it has not generated more bug in libalpm. so i check all uninitalised pointer in libalpm before alpm_list_remove and i found 3 occurences see the patch diff --git a/lib/libalpm/cache.c b/lib/libalpm/cache.c index 032cc97..b7681db 100644 --- a/lib/libalpm/cache.c +++ b/lib/libalpm/cache.c @@ -135,7 +135,7 @@ int _alpm_db_add_pkgincache(pmdb_t *db, pmpkg_t *pkg) int _alpm_db_remove_pkgfromcache(pmdb_t *db, pmpkg_t *pkg) { - void *vdata; + void *vdata=NULL; pmpkg_t *data; ALPM_LOG_FUNC; diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c index d9a3931..c0d322c 100644 --- a/lib/libalpm/db.c +++ b/lib/libalpm/db.c @@ -145,7 +145,7 @@ int SYMEXPORT alpm_db_unregister(pmdb_t *db) * databases by walking through the list returned by * alpm_option_get_syncdbs, because the db is removed from that list here. */ - void *data; + void *data=NULL; handle->dbs_sync = alpm_list_remove(handle->dbs_sync, db, _alpm_db_cmp, &data); if(data) { diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c index 864fafa..6ea16c0 100644 --- a/lib/libalpm/remove.c +++ b/lib/libalpm/remove.c @@ -118,7 +118,7 @@ static void remove_prepare_keep_needed(pmtrans_t *trans, pmdb_t *db, alpm_list_t *i; for(i = lp; i; i = i->next) { pmdepmissing_t *miss = (pmdepmissing_t *)i->data; - void *vpkg; + void *vpkg=NULL; pmpkg_t *pkg = _alpm_pkg_find(trans->packages, miss->causingpkg); if(pkg == NULL) { continue;

3 13

Re: [pacman-dev] [arch-general] rotate pacman log?
by Aaron Griffin 08 Aug '08

08 Aug '08

On Thu, Aug 7, 2008 at 4:05 PM, Dan McGee <dpmcgee(a)gmail.com> wrote: > On Thu, Aug 7, 2008 at 3:49 PM, Dale Blount <dale(a)archlinux.org> wrote: >> On Thu, 2008-08-07 at 15:42 -0500, Aaron Griffin wrote: >>> On Thu, Aug 7, 2008 at 3:38 PM, David Rosenstrauch <darose(a)darose.net> wrote: >>> > I noticed that the pacman log doesn't get rotated. Seems like it should, as >>> > it could get rather big over time. Any particular reason why the package >>> > doesn't provide a /etc/logrotate.d/ script by default? >>> >>> I don't see a problem with that. Probably a good idea too. If you want >>> to provide a file for pacman, I'm sure Dan would be happy to include >>> it in the arch package. >> >> My only request here is that it saves at least a years worth of updates >> by default. If something breaks on my system and I don't notice it for >> a few months, I can't tell if an update broke it or not. >> >> I have a 828Kb pacman.log from a 5 year old install. Granted I don't >> -Syu as often as I should, but it still seems manageable at many times >> that on modern hardware. > > I'm actually with Dale here. I find it nice to go all the way back to > the "beginning of time" with my install so I can see exactly what may > have pulled in a now unneeded dep, etc. I just used this on my Eee > yesterday to remove unnecessary packages originally pulled in by > OpenOffice (hsqldb). I would rather old logs never get deleted; but > even more I would rather the file never get touched. > > There is a separate concern I have wanted to address for a while, and > that is the mixing of what was previously a pristine pacman.log with > the scriptlet messages. It is a great idea, but in practice, it makes > this file not near as concise as it once was. In an ideal world: > 1) pacman.log would return to only being upgrade/install/remove messages. > 2) another log file would be added that contained the verbose stuff. > pacman_messages.log or something. > 3) pacman.log never rotates/deletes. > 4) pacman_messages.log rotates/deletes. > > What do people think of this? While we're proposing ideas.... what about this: * new scriptlet function "message()", that just outputs text. * add a -Q operation to call the message() function. Then there's really no need for scriptlet logging at all.

4 4

[pacman-dev] -h/ --help option
by Matthias Gorissen 07 Aug '08

07 Aug '08

A user in the .de-forum pointed out that -h/ --help have the same output, even though the second version should display more info. Any thoughts on that?

5 5

[pacman-dev] [translation] Problems with DE-version
by Matthias Gorissen 07 Aug '08

07 Aug '08

So version 3.2.0 went to [core], and upon the next -Syu, pacman greeted me with the following message: :: pacman hat eine neuere Version von sich selbst gefunden. :: Möchtest du die momentan Handlung abbrechen :: und die neue pacman Version installieren? [Y/n]y WTF??? This is anything but German! Style and taste aside, there are at least three serious errors: 1. Pacman uses a colloquial form to address its user - this is logically inconsistent. All the rest of the De-translations use a more formal one (just to avoid ambivalent meanings - it is not about formality/politeness). 2. "momentan" should be "momentane" - it sounds like an "ery annoying error" in English. 3. [Y/n] should be [J/n]. If you type "y", pacman gets updated with all the other packages - not in before. Of course, I don't want to blame anyone in particular here. There is no-one to blame but me - I should have held these files under much closer surveillance. I was quite sure, however, that I had purged these lines, and that the changes were merged. So this an "ery annyoing error" indeed. For the sake of interest, this error was originally introduced here: http://projects.archlinux.org/?p=pacman.git;a=commitdiff;h=2b0c89b06a5cf68f… I didn't find any indication that this went over the ML (where I would have seen/checked it). Maybe a private mail to Dan? - I dunno, but it should not have been accepted this way. I thought I had cleaned up the entire issue here (but maybe I was wrong): http://projects.archlinux.org/?p=pacman.git;a=commitdiff;h=4ebad47ae7dff0f0… This one was merged, so I don't see why the the older version keeps popping up. I have provided another patch here: http://archlinux.org/pipermail/pacman-dev/2008-July/012427.html This one may not have been merged, but hey - there are at least 20 people subscribed to this who can checkout git-repos retrospectively 10 times better than I can, aren't they? So I'll write a patch for 3.2.1 (maybe this evening, if I stay sober enough). But despite my own short-comings, I wonder about organization. Maybe we could figure out a way to avoid such mistakes in the future. Best regards, Matt

2 3

[pacman-dev] added --config option to makepkg
by hoar heor 07 Aug '08

07 Aug '08

I added a --config=value option so you can use makepkg --config=safe and it will source the makepkg.safe file instead of makepkg.conf >From 4669853aac72506201af4d6e9ec79f080177d42f Mon Sep 17 00:00:00 2001 From: Imanol Celaya <ilcra1989(a)gmail.com> Date: Thu, 7 Aug 2008 13:15:27 +0200 Subject: [PATCH] adds --config=value option to use makepkg.value instead of makepkg.conf, as makepkg.safe or makepkg.blender Signed-off-by: Imanol Celaya <ilcra1989(a)gmail.com> --- scripts/makepkg.sh.in | 32 +++++++++++++++++++++++++------- 1 files changed, 25 insertions(+), 7 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 3604d10..267eb81 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1084,6 +1084,7 @@ usage() { echo "$(gettext " -m, --nocolor Disable colorized output messages")" echo "$(gettext " -o, --nobuild Download and extract files only")" printf "$(gettext " -p <buildscript> Use an alternate build script (instead of '%s')")\n" "$BUILDSCRIPT" + echo "$(gettext " --config=value use /etc/makepkg.value file instead of makepkg.conf")" echo "$(gettext " -r, --rmdeps Remove installed dependencies after a successful build")" # fix flyspray feature request #2978 echo "$(gettext " -R, --repackage Repackage contents of pkg/ without building")" @@ -1126,14 +1127,31 @@ _PKGDEST=${PKGDEST} _SRCDEST=${SRCDEST} # Source makepkg.conf; fail if it is not found -if [ -r "$confdir/makepkg.conf" ]; then - source "$confdir/makepkg.conf" -else - error "$(gettext "%s not found.")" "$confdir/makepkg.conf" - plain "$(gettext "Aborting...")" - exit 1 # $E_CONFIG_ERROR +# if --config=value option found, use makepkg.value +CUSTOMCONFIG=0 +for ARG in $@; do +if [ ${ARG:0:9} = "--config=" ]; then + if [ -r "$confdir/makepkg.${ARG:9}" ]; then + plain "$(gettext "Using custom makepkg file: %s")" "$confdir/makepkg.${ARG:9}" + source "$confdir/makepkg.${ARG:9}" + CUSTOMCONFIG=1 + break + else + error "$(gettext "%s not found.")" "$confdir/makepkg.${ARG:9}" + plain "$(gettext "Aborting...")" + exit 1 # $E_CONFIG_ERROR +fi +if [ $CUSTOMCONFIG = 0 ]; then + if [ -r "$confdir/makepkg.conf" ]; then + source "$confdir/makepkg.conf" + else + error "$(gettext "%s not found.")" "$confdir/makepkg.conf" + plain "$(gettext "Aborting...")" + exit 1 # $E_CONFIG_ERROR + fi fi + # Source user-specific makepkg.conf overrides if [ -r ~/.makepkg.conf ]; then source ~/.makepkg.conf @@ -1166,7 +1184,7 @@ while true; do # Pacman Options --noconfirm) PACMAN_OPTS="$PACMAN_OPTS --noconfirm" ;; --noprogressbar) PACMAN_OPTS="$PACMAN_OPTS --noprogressbar" ;; - + # Makepkg Options --allsource) SOURCEONLY=2 ;; --asroot) ASROOT=1 ;; -- 1.5.6.5

3 7