July 2009 - Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] makepkg: add pkgbase to .PKGINFO and database for split packages
by Allan McRae 22 Jul '09

22 Jul '09

With split packages, the pkgbase variable provides a useful way to find out which packages were build from the same PKGBUILD. Add it to the packages .PKGINFO file and the repo database only when package splitting is used. Original-patch-by: Pierre Schmitz <pierre(a)archlinux.de> [Allan: restrict to including only with spilt packages and include after pkgname] Signed-off-by: Allan McRae <allan(a)archlinux.org> --- @Pierre: Was the a reason to put pkgbase above pkgname in the .PKGINFO and repo db files? It is secondary information so I have moved it to be included after. scripts/makepkg.sh.in | 1 + scripts/repo-add.sh.in | 1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 50cf54d..7e492f8 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -890,6 +890,7 @@ write_pkginfo() { fi echo "# $(LC_ALL=C date -u)" >>.PKGINFO echo "pkgname = $1" >>.PKGINFO + [ "$SPLITPKG" -eq 1 ] && echo "pkgbase = $pkgbase" >>.PKGINFO echo "pkgver = $pkgver-$pkgrel" >>.PKGINFO echo "pkgdesc = $pkgdesc" >>.PKGINFO echo "url = $url" >>.PKGINFO diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in index 5454fb0..d3566f5 100644 --- a/scripts/repo-add.sh.in +++ b/scripts/repo-add.sh.in @@ -242,6 +242,7 @@ db_write_entry() msg2 "$(gettext "Creating 'desc' db entry...")" echo -e "%FILENAME%\n$(basename "$1")\n" >>desc echo -e "%NAME%\n$pkgname\n" >>desc + [ -n "$pkgbase" ] && echo -e "%BASE%\n$pkgbase\n" >>desc echo -e "%VERSION%\n$pkgver\n" >>desc [ -n "$pkgdesc" ] && echo -e "%DESC%\n$pkgdesc\n" >>desc write_list_entry "GROUPS" "$_groups" "desc" -- 1.6.3.3

2 2

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

22 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 cf669eda9c6b8e0b1d38bcf32fa08c0637a9e030 (commit) via 1c4596b4be196e35c9982c53645d8ab1ada03988 (commit) from 344ced22c347b347b55818bb93579ffa2eb9c70b (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 cf669eda9c6b8e0b1d38bcf32fa08c0637a9e030 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Tue Jul 21 15:50:08 2009 +0200 Fix a minor memleak Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1c4596b4be196e35c9982c53645d8ab1ada03988 Author: Giovanni Scafora <giovanni(a)archlinux.org> Date: Tue Jul 21 20:24:06 2009 -0500 Update Italian Translation Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/po/it.po | 144 ++++++++++------------ lib/libalpm/remove.c | 1 + po/it.po | 337 ++++++++++++++++++++++++++++---------------------- 3 files changed, 257 insertions(+), 225 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [Translations] Update Italian translation
by Giovanni Scafora 22 Jul '09

22 Jul '09

Hi guys, here's an update Italian translation for pacman and libalpm. Thanks. -- Arch Linux Developer http://www.archlinux.org http://www.archlinux.it

2 2

[pacman-dev] need root
by Xavier 21 Jul '09

21 Jul '09

does this look ok?

1 2

[pacman-dev] [PATCH] Fix a minor memleak
by Nagy Gabor 21 Jul '09

21 Jul '09

>From 5979ea01d2f4217c1b199c675a7a3d4c728da9fa Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Tue, 21 Jul 2009 15:50:08 +0200 Subject: [PATCH] Fix a minor memleak Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- lib/libalpm/remove.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c index e370909..aa6a5cc 100644 --- a/lib/libalpm/remove.c +++ b/lib/libalpm/remove.c @@ -277,6 +277,7 @@ static void unlink_file(pmpkg_t *info, char *filename, pmtrans_t *trans) if(pkghash) { if(trans->flags & PM_TRANS_FLAG_NOSAVE) { _alpm_log(PM_LOG_DEBUG, "transaction is set to NOSAVE, not backing up '%s'\n", file); + FREE(pkghash); } else { char *filehash = alpm_compute_md5sum(file); int cmp = strcmp(filehash,pkghash); -- 1.6.0.3

1 0

Re: [pacman-dev] architecture warn/check
by Dan McGee 21 Jul '09

21 Jul '09

On Mon, Jul 20, 2009 at 3:38 PM, Steven Blatchford<sblatchford(a)gmail.com> wrote: > Hi Dan, > > I'm sure this has been brought up in the pacman ML but I couldn't find > it quickly. Do you think it would be useful to check the architecture > of the machine (eg the output of 'uname -m') against the binary pacman > is downloading? Twice I've sync'd the file /etc/pacman.d/mirrorlist via > unison to my slicehost server from my i686 network. The latest bash4.0 > upgrade hurt... like there were tears... and henceforth it's now known > in my house as "Grumpy Sunday". > > I have no trouble creating a wrapper script, I just thought I'd toss it > out there. > > Lastly, if you suggest I go the wrapper script method, besides trying to > parse the mirrorlist file, is there a nice way to get the architecture > of a file from pacman before it downloads it? /installs it? Would you mind sending this to the pacman-dev ML or filing a bug report instead next time? Unfortunately it will just get buried in my personal email inbox. I'm copying the list on this response. With that said, I think we could perhaps take some precautions for such things, such as adding a pacman.conf option to verify the architecture. Something such as: RootDir = / DBPath = /var/lib/pacman Architecture = x86_64 Where the accepted options would be something like: Architecture = { i686, x86_64, ppc, etc... } or "auto", which would make a uname system call, check the machine[] field, and use that instead of a value being hardcoded? What does the rest of the list think? This wouldn't be too hard, and of course a package coded with architecture "any" would get a free pass. -Dan

4 10

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

20 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 344ced22c347b347b55818bb93579ffa2eb9c70b (commit) via 60b6cde637338657e72312aef1ff1d04314df745 (commit) via 5d15bb68f7b6b3e25be286ddf7a615553a173b8f (commit) via 45f90de0eb9c33eee0deb63bae9aabe5988b8733 (commit) via bfd6817112b34b552e9139bdc582d048bcba38b9 (commit) via a3ecbec6b584b3595c041850e4c8bf242f7ce74a (commit) from 8ebc07744ac58ed1e42af3a52d998587fca85f8b (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 344ced22c347b347b55818bb93579ffa2eb9c70b Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Mon Jul 20 17:23:19 2009 +0200 Fix one bug with .paccheck leftover file This happens for example if you install a new package, and one of its backup config file is already on the file system. If the local file was different, it was saved to .pacorig which is fine. However if the local file and pkg file were the same, the pkg file (temporarily extracted as .paccheck) was left on the system. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 60b6cde637338657e72312aef1ff1d04314df745 Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Mon Jul 20 14:52:16 2009 +0200 Fix 2 minor memleaks Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5d15bb68f7b6b3e25be286ddf7a615553a173b8f Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Sun Jul 19 19:49:59 2009 +0200 Do not create .pacsave with -R, if the file is unchanged This fixes FS#15546 Also fix the interface of unlink_file which was really stupid.. (alpm_list_t used with only one element) Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 45f90de0eb9c33eee0deb63bae9aabe5988b8733 Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Sun Jul 19 16:54:12 2009 +0200 Fix klibc conflict case. A package can now replace symdir->dir by dir without fileconflicts. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit bfd6817112b34b552e9139bdc582d048bcba38b9 Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Fri Jul 17 15:48:57 2009 +0200 Fix fileconflict004 When one package wants to replace a directory by a file, we check that all files in that directory were owned by that package. Additionally pacman can be more verbose when the extraction of the symlink (or file) fails. The patch to add.c looks more complex than it is, I just moved and reindented code to handle cases 10 and 11 together. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a3ecbec6b584b3595c041850e4c8bf242f7ce74a Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Fri Jul 17 19:33:13 2009 +0200 pactest : safety check with MODE check that the file exists first, otherwise pactest just breaks. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/add.c | 55 ++++++++-------- lib/libalpm/conflict.c | 70 ++++++++++++++++++++ lib/libalpm/handle.c | 1 + lib/libalpm/remove.c | 47 +++++++------ lib/libalpm/sync.c | 1 + pactest/pmrule.py | 7 ++- pactest/tests/fileconflict004.py | 2 - pactest/tests/fileconflict005.py | 1 - .../{fileconflict005.py => fileconflict006.py} | 7 +- pactest/tests/fileconflict007.py | 17 +++++ pactest/tests/remove010.py | 4 +- 11 files changed, 152 insertions(+), 60 deletions(-) copy pactest/tests/{fileconflict005.py => fileconflict006.py} (81%) create mode 100644 pactest/tests/fileconflict007.py hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] Fileconflict error...
by Nagy Gabor 20 Jul '09

20 Jul '09

Hi! I've tried -Su again... I've just run into a very "hard to fix" error. [ngaba@Arch ~]$ LANG=C sudo pacman -Su --ignore mplayer ... xulrunner: /usr/include/xulrunner-1.9 exists in filesystem xulrunner: /usr/lib/xulrunner-1.9 exists in filesystem xulrunner: /usr/lib/xulrunner-devel-1.9 exists in filesystem xulrunner: /usr/share/idl/xulrunner-1.9 exists in filesystem ... These are directories... But in the package we have symlinks. Bye

7 43

[pacman-dev] 0001-Fix-2-minor-memleaks.patch
by Xavier 20 Jul '09

20 Jul '09

1 0

[pacman-dev] strange valgrind result
by Xavier 19 Jul '09

19 Jul '09

When running valgrind with pactest, I always get the following leak : > ./pactest/pactest.py -p ./src/pacman/.libs/lt-pacman -t pactest/tests/remove011.py --valgrind ==3131== ==3131== 20 bytes in 1 blocks are still reachable in loss record 1 of 1 ==3131== at 0x402314C: calloc (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so) ==3131== by 0x439E0E5: _dlerror_run (in /lib/libdl-2.10.1.so) ==3131== by 0x439DD2B: dlsym (in /lib/libdl-2.10.1.so) ==3131== by 0x402EC78: load_library_symbols (in /usr/lib/libfakeroot/libfakeroot-0.so) ==3131== by 0x402F676: tmp___xstat64 (in /usr/lib/libfakeroot/libfakeroot-0.so) ==3131== by 0x402D082: __xstat64 (in /usr/lib/libfakeroot/libfakeroot-0.so) ==3131== by 0x4047D73: alpm_option_set_root (stat.h:455) ==3131== by 0x804E57F: main (pacman.c:864) It happens on both maint and master. I dont remember seeing this error last time I tried valgrind with pactest. Ok it was a long time ago, but I don't think this code have changed. if I comment out the stat in set_root : if(stat(dbpath, &st) == -1 || !S_ISDIR(st.st_mode)) { Then I get the same error for set_dbpath, and etc... Also when I run directly valgrind on pacman (without pactest), it works fine. So might be only a new fakeroot related problem. Anyway, it's not important, but if anyone has any clue, just tell :)

1 0