March 2008 - Pacman-dev - lists.archlinux.org

[pacman-dev] bug during package upgrade?
by Roman Kyrylych 24 Mar '08

24 Mar '08

Hi! Here's Tobias' pacman.log. Notice that klibc-kbd is not installed at all, though the newest mkinitcpio depends on it. The line [2008-03-21 19:57] installed klibc-kbd (1.15.20080312-4) at the bottom is from manual installation. ---------- Forwarded message ---------- From: Tobias Powalowski <t.powa(a)gmx.de> Date: 22 бер. 2008 00:14 Subject: big upgrade To: Roman Kyrylych <roman.kyrylych(a)gmail.com> -- Roman Kyrylych (Роман Кирилич)

4 9

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.1.3-13-g1086950
by Dan McGee 23 Mar '08

23 Mar '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 1086950c82ab649ffc36a00cd8d343da376252fe (commit) via 7995a25d0ed75968df72fcf65d231508417ec060 (commit) from 7d451b6e6bc3926589a77e6a2d191e4f158d3980 (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 ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: lib/libalpm/add.c | 1 + lib/libalpm/backup.c | 3 +++ src/pacman/util.c | 2 +- 3 files changed, 5 insertions(+), 1 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.1.3-95-g5f1ccdb
by Dan McGee 23 Mar '08

23 Mar '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 5f1ccdbc27e3b4b92f9b65379db17a4d6523975d (commit) via d1ea16dfd0d497aff72d7a315890971e7f070786 (commit) via 6104f2e1fb872c32d3e6ed1823729592fc6245b9 (commit) via 1086950c82ab649ffc36a00cd8d343da376252fe (commit) via 7995a25d0ed75968df72fcf65d231508417ec060 (commit) via bf8670036907b3ede18d37b7a3f0f7e14542a5ac (commit) via 90a48c771d727e3bd231eeb1611dfcbecb88490c (commit) via f7f43dbb48add85c3c3e42696c4d3bae4591a7d6 (commit) via 3d10d460dff4c1321b40fa509c4ff9be93db0ca3 (commit) via 5af076f09f67bb3abd80a28fa3f33437f08b18bc (commit) from 2f8fb80ee631ef866c3bd8cb5fab09a7360ca8d2 (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 5f1ccdbc27e3b4b92f9b65379db17a4d6523975d Merge: d1ea16dfd0d497aff72d7a315890971e7f070786 1086950c82ab649ffc36a00cd8d343da376252fe Author: Dan McGee <dan(a)archlinux.org> Date: Sun Mar 23 16:58:53 2008 -0500 Merge branch 'maint' commit d1ea16dfd0d497aff72d7a315890971e7f070786 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Tue Mar 11 16:40:27 2008 +0100 Avoid duplicated target names. This patch should avoid duplicated target names in the backend. 1. sync_loadtarget will return with PM_ERR_TRANS_DUP_TARGET when trying to add a duplicated target 2. sysupgrade never pulls duplicated targets 3. resolvedeps won't pull duplicated targets anymore A pulled list was introduced in sync_prepare to improve the pmsyncpkg_t<->pmpkg_t list conversion by making it more direct. Also replace sync1005 and sync1006 by the sync1008 pactest, which is similar but more interesting (the provisions are dependencies instead of explicit targets). sync1005 didn't work as expected anyway. It was expecting that pacman failed, and pacman indeed failed, but not for the good reason. It didn't fail during the preparation step because of conflicting targets, but during the commit step, because of a md5 error... And sync1006 didn't pass and was not really worth fixing. We have already enough failing pactests more important than these two. sync1008 pass with this patch. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 6104f2e1fb872c32d3e6ed1823729592fc6245b9 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Mar 23 15:35:22 2008 -0500 memleak fix: ensure we free result of get_destfile() In the file:// download case, we didn't free the return from get_destfile() after we were done with it. Fix it. (Found with xfercommand001.py) Signed-off-by: Dan McGee <dan(a)archlinux.org> commit bf8670036907b3ede18d37b7a3f0f7e14542a5ac Author: Dan McGee <dan(a)archlinux.org> Date: Sat Jan 12 01:13:52 2008 -0600 Switch pmgrp_t to dynamic allocation, general group cleanup Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 90a48c771d727e3bd231eeb1611dfcbecb88490c Author: Dan McGee <dan(a)archlinux.org> Date: Sun Mar 23 14:25:38 2008 -0500 Add -q (quiet) option to valgrind call in pactest This gets rid of a lot of the unnecessary verbosity in the --valgrind output when running all the tests. It should also make diff-ing output between test runs a lot easier. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f7f43dbb48add85c3c3e42696c4d3bae4591a7d6 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Mar 23 14:08:31 2008 -0500 Update database mtime after it has been extracted This will reduce the need for running an -Syy if the DB was only half-extracted, as the mtime won't get updated until the new database is completely in place. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3d10d460dff4c1321b40fa509c4ff9be93db0ca3 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sat Feb 16 16:01:13 2008 +0100 Add new CleanMethod option. As it was already mentioned several times, the new -Sc behavior in 3.1 is great, but only when the package cache is not shared. This option has two possible values : KeepInstalled and KeepCurrent With KeepCurrent, -Sc will clean packages that are no longer available in any sync db, rather than packages that are no longer in the local db. The resulting behavior should be better for shared cache. Ref : http://www.archlinux.org/pipermail/pacman-dev/2008-February/011140.html Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5af076f09f67bb3abd80a28fa3f33437f08b18bc Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sat Mar 15 21:10:54 2008 +0100 Kill the dependsonly option. From the man page : "This is pretty useless and we're not sure why it even exists." Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> ----------------------------------------------------------------------- Summary of changes: contrib/bash_completion | 1 - contrib/zsh_completion | 1 - doc/pacman.8.txt | 7 +- doc/pacman.conf.5.txt | 9 +++ lib/libalpm/add.c | 1 + lib/libalpm/alpm.h | 4 +- lib/libalpm/backup.c | 3 + lib/libalpm/cache.c | 50 ++++++-------- lib/libalpm/db.c | 20 +++-- lib/libalpm/deps.c | 6 +- lib/libalpm/dload.c | 11 ++-- lib/libalpm/group.c | 9 ++- lib/libalpm/group.h | 11 ++-- lib/libalpm/sync.c | 57 +++++++-------- pactest/pmtest.py | 2 +- pactest/tests/sync1001.py | 18 ----- pactest/tests/sync1002.py | 19 ----- pactest/tests/sync1006.py | 14 ---- pactest/tests/{sync1005.py => sync1008.py} | 11 ++- src/pacman/conf.h | 14 +++- src/pacman/pacman.c | 13 +++- src/pacman/query.c | 28 ++++---- src/pacman/sync.c | 106 ++++++++++++++++++++------- src/pacman/util.c | 2 +- 24 files changed, 221 insertions(+), 196 deletions(-) delete mode 100644 pactest/tests/sync1001.py delete mode 100644 pactest/tests/sync1002.py delete mode 100644 pactest/tests/sync1006.py rename pactest/tests/{sync1005.py => sync1008.py} (54%) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.1.3-11-g7d451b6
by Dan McGee 22 Mar '08

22 Mar '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 7d451b6e6bc3926589a77e6a2d191e4f158d3980 (commit) via d07001f3ab3000ab4bd7277549b58641b0d02f0a (commit) via 1f30845e41b6e315e4553fade891be0b46ac2470 (commit) from 6e4b020654481c3b9050c5125a567b0b32cadbc6 (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 ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: contrib/PKGBUILD.vim | 4 +- etc/pacman.conf.in | 14 +++++++++ po/zh_CN.po | 80 +++++++++++++++++++++++++------------------------- 3 files changed, 56 insertions(+), 42 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.1.3-85-g2f8fb80
by Dan McGee 22 Mar '08

22 Mar '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 2f8fb80ee631ef866c3bd8cb5fab09a7360ca8d2 (commit) via 7d451b6e6bc3926589a77e6a2d191e4f158d3980 (commit) via d07001f3ab3000ab4bd7277549b58641b0d02f0a (commit) via e7a2232934e2658c3c7827d2ce706c1a8f5301b9 (commit) via 73ab153c44a1581b4bcf1e64eb6042b0631c2e66 (commit) via b3e6cf652c9e989badaf5499abb1d64c1a110927 (commit) via dae3f9deefdb86f726a68dc89a7391e9df7517df (commit) via 1f30845e41b6e315e4553fade891be0b46ac2470 (commit) via 6e4b020654481c3b9050c5125a567b0b32cadbc6 (commit) via 6820be9ba1e760e66df1d4cc1a73c04b55bdb00a (commit) via 804ab37ea62602b04ae7b65c3e7fddda12bbae5d (commit) from 35135c0a0cbac592e72296c0ca64e9def0308942 (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 2f8fb80ee631ef866c3bd8cb5fab09a7360ca8d2 Merge: e7a2232934e2658c3c7827d2ce706c1a8f5301b9 7d451b6e6bc3926589a77e6a2d191e4f158d3980 Author: Dan McGee <dan(a)archlinux.org> Date: Sat Mar 22 11:11:22 2008 -0500 Merge branch 'maint' commit e7a2232934e2658c3c7827d2ce706c1a8f5301b9 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sun Mar 16 22:36:31 2008 +0100 Kill PM_TRANS_TYPE_ADD. This was totally useless. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 73ab153c44a1581b4bcf1e64eb6042b0631c2e66 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sun Mar 16 22:23:47 2008 +0100 Kill some obsolete references to -A option. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit b3e6cf652c9e989badaf5499abb1d64c1a110927 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Thu Mar 13 16:36:44 2008 +0100 Drop case insensitive comparisons in the config parsing. These case insensitive comparisons didn't work in some locales, like tr_TR where upper(i) != I. So a second case sensitive comparison had to be made for each directive. Only keeping case sensitive comparisons make the code cleaner and treat all locales equally. Ref: http://www.archlinux.org/pipermail/pacman-dev/2008-March/011445.html Also fix pactests to use the correct case. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit dae3f9deefdb86f726a68dc89a7391e9df7517df Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Thu Mar 13 00:25:38 2008 +0100 Add zipman makepkg option. All other steps in tidy_install function were already controlled by an option in makepkg.conf, so this patch adds an option for the man page compression step too. This will allow to keep man pages uncompressed, which is required for some special meta man page, like the zshall one (see FS#4580). Ref: http://www.archlinux.org/pipermail/pacman-dev/2008-March/011472.html Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> ----------------------------------------------------------------------- Summary of changes: contrib/PKGBUILD.vim | 12 ++-- contrib/bash_completion | 8 +- contrib/zsh_completion | 11 +--- doc/PKGBUILD.5.txt | 3 + doc/makepkg.conf.5.txt | 7 +- doc/pacman.conf.5.txt | 9 ++- etc/makepkg.conf.in | 5 +- etc/pacman.conf.in | 14 +++ lib/libalpm/add.c | 14 +--- lib/libalpm/alpm.h | 4 +- lib/libalpm/deps.c | 13 ++-- lib/libalpm/deps.h | 2 +- lib/libalpm/error.c | 2 - lib/libalpm/remove.c | 2 +- lib/libalpm/sync.c | 2 +- lib/libalpm/trans.c | 3 - pactest/tests/remove030.py | 2 +- pactest/tests/sync021.py | 2 +- pactest/tests/sync120.py | 2 +- pactest/tests/sync133.py | 2 +- pactest/tests/sync138.py | 2 +- pactest/tests/upgrade010.py | 2 +- pactest/tests/upgrade070.py | 2 +- pactest/tests/xfercommand001.py | 2 +- pactest/util.py | 2 +- po/zh_CN.po | 172 ++++++++++++++++++++------------------- scripts/makepkg.sh.in | 64 ++++++++------- src/pacman/pacman.c | 52 +++++------- 28 files changed, 206 insertions(+), 211 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] uncompressed man pages
by Xavier 13 Mar '08

13 Mar '08

Hello, the tidy_install makepkg function is composed of 5 parts : 1) Removing info/doc controlled by docs option 2) compressing man pages 3) stripping debugging symbols controlled by strip 4) removing libtool controlled by libtool 5) removing empty directories controlled by emptydirs So as you can see, all steps can be enabled / disabled with an option except step 2) It turns out it would be useful to control that too. For example, the zsh package has a special zshall man page that includes all others, but this only works with uncompressed man page. Working around this automatic makepkg compression is really awful and ugly : http://bugs.archlinux.org/task/4580 So would it be acceptable to add a "compress" option or something?

5 6

[pacman-dev] Case sensitive options in pacman.conf
by Xavier 13 Mar '08

13 Mar '08

Currently, every options in pacman.conf apparently has a fixed case : RootDir = / DBPath = /var/lib/pacman/ CacheDir = /var/cache/pacman/pkg/ LogFile = /var/log/pacman.log HoldPkg = pacman glibc But in fact, pacman does an insensitive comparisons when parsing the file. } else if(!strcmp(key, "CACHEDIR")) { with key = strtoupper("CacheDir") So you can put cAcHeDiR if you like.. Then we found out this insensitive comparison didn't work in some locales, like tr_TR one, where for example, upper(i) != I To work around this, we also added a sensitive comparison : } else if(strcmp(origkey, "CacheDir") == 0 || strcmp(key, "CACHEDIR") == 0) { http://projects.archlinux.org/git/?p=pacman.git;a=commitdiff;h=1cd7567ff8af… So in fact, in tr_TR locale, only case sensitive options will really work (eg cAcHeDiR won't work). Why do we make a special case for this locale? I think that having to do every single comparisons twice during the config file parsing is really ugly, and more error prone when adding new options. So I would vote for only allowing case sensitive options in pacman.conf, to treat all locales equally, and cleaning the code :) If that's not acceptable, what about adding a new function, to do something like : } else if(compare(origkey, "CacheDir", "CACHEDIR") == 0) { That still isn't ideal but would at least reduce the duplication a bit.

7 9

[pacman-dev] PATHNAME in sync/*/desc
by Mark Constable 12 Mar '08

12 Mar '08

I'm just checking if this could be a reasonable idea and if so I'd look into providing a patch. Would it be feasible to add a %PATHNAME% variable to the sync desc file ? For instance in this case, fetching "any" arch packages would not need to symlinked into the same directory to be found... %FILENAME% hotstuff-0.9.2-1-any.pkg.tar.bz2 %PATHNAME% ../any/ It may provide repo maintainers with some flexibility (hard to test without it being possible) and perhaps %PATHNAME% could also be a URL and that might also be interesting, but the simple relative path prefix would be useful. Any thoughts? --markc

6 10

[pacman-dev] [PATCH] Don't ask for install confirmation if no dependencies
by mmiikkee13＠gmail.com 12 Mar '08

12 Mar '08

This is a patch to make Pacman only ask for confirmation if anything different from what the user requested (i.e. dependencies) is going to be installed. Since the user took the time to type out "pacman -Sy foo", they obviously did want foo installed, and it really wouldn't make sense to ask them this again unless something else will be installed.

8 9

[pacman-dev] [PATCH] Treat -Sc like an option
by Nagy Gabor 12 Mar '08

12 Mar '08

From 2ffe8c06f1fdadfa2f9ccd38a706568f261f3f6c Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Sat, 8 Mar 2008 17:58:15 +0100 Subject: [PATCH] Treat -Sc like an option From now on 'pacman -Sc target' will first clean the cache, then install target... Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- src/pacman/sync.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 130d356..1355cd8 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -773,7 +773,9 @@ int pacman_sync(alpm_list_t *targets) ret++; } - return(ret); + if(ret) { + return(ret); + } } /* ensure we have at least one valid sync db set up */ -- 1.5.3.8

5 9