- Pacman-dev - lists.archlinux.org

[pacman-dev] difficult symlink puzzle, part 2.
by Nagy Gabor 20 Aug '07

20 Aug '07

Hi! If I didn't misunderstood conflict.c then fileconflict check fails in the following case (pacman -A pkg1 pkg2): pkg1 contains: /dir1/file pkg2 contains: /dir2/file, if /dir1 and /dir2 exists on filesystem, and /dir2 is a symlink pointing to /dir1. Much more difficult problem (and fortunately much rarer), if pkg2 creates the /dir2 symlink, so it didn't exist before the transaction. Any tests/feedbacks appreciated... How is this handled in other package managers? Bye, ngaba PS: a possible semi-solution: pacman shouldn't stop/overwrite in case of these "hidden" fileconflict errors, it extract /dir2/file as /dir2/file.pkg2.pacnew and do something at the end of the transaction. (Speed is a big limiting factor here.)

4 3

[pacman-dev] "FATAL: kernel too old" error message during upgrade
by Roman Kyrylych 20 Aug '07

20 Aug '07

Hi! One of our local community members is getting this output when doing -Syu after installing 0.7.2: checking package integrity... done. cleaning up... done. (8/8) checking for file conflicts [#####################] 100% (1/8) installing kernel-headers [#####################] 100% (2/8) upgrading glibc [#####################] 100% FATAL: kernel too old (3/8) installing binutils [#####################] 100% (4/8) installing gcc [#####################] 100% (5/8) installing db [#####################] 100% (6/8) installing gdbm [#####################] 100% (7/8) upgrading openssl [#####################] 100% (8/8) installing python [#####################] 100% FATAL: kernel too old FATAL: kernel too old Any ideas what causes this? -- Roman Kyrylych (Роман Кирилич)

3 6

[pacman-dev] Missing sanity checks for path -> weird behaviors and segfaults
by Xavier 20 Aug '07

20 Aug '07

If Root isn't specified, paths like (null)var/ will be created. I think pacman should either default to / for Root, or just fail if it isn't set. Also, if no Cache Dirs is set, pacman will try to fallback to /tmp/ , but then it segfaults. (try pacman -Sw bar). Apparently it segfaults on this in libalpm/sync.c , sync_commit function, after downloading the file succesfully : 718 for(i = handle->dbs_sync; i; i = i->next) I found it weird that inside that loop, i was used a second time in an inner loop : 758 for(i = alpm_option_get_cachedirs(); i; i = alpm_list_next(i)) { I thought that was the problem, but it didn't fix the segfault. Second strange thing is that the files list seem to be freed in any cases : 795 FREELIST(files); but it's used at the end of that _alpm_sync_commit function : 1018 if(!validcache && !(trans->flags & PM_TRANS_FLAG_DOWNLOADONLY)) { 1019 /* delete packages */ 1020 for(i = files; i; i = i->next) { 1021 unlink(i->data); 1022 } 1023 } Finally, if the cache isn't valid (so it fallbacks to /tmp/) and DOWNLOADONLY is used, files won't be deleted, but will they be used next times? Because the fallback (lines 779-790) happen after the check of existence of the package in the cache (lines 740-744). I can't check what happens because of the segfault.

3 8

[pacman-dev] [patch] Conflict variables
by Nagy Gabor 18 Aug '07

18 Aug '07

Hi! After the professional cleanup of conflict.c (by Xavier, thx; anyway, do you want to add conflict checking to db_test?;-) we don't need to restrict conflicts to conflict-name only, we can use the depend syntax. So this patch doesn't change any important thing in the source code, mainly this is a testcase instead. Bye, ngaba

3 3

[pacman-dev] [patch] for resolvedeps + some notes
by Nagy Gabor 18 Aug '07

18 Aug '07

Hi! With this patch resolvedeps will search for other satisfiers, if we don't let it add a pkg from IgnorePkg. Note: this patch assumes my depcheck.diff patch applied. Bye PS: Resolvedeps is still dummy, because pulling a new package to the *list may break an earlier verified dependency of a pkg in the *list (because the new package will overwrite something in the localdb!). The best (but much slower) solution is to call depcheck with the *list parameter (plus syncpkg).

2 4

[pacman-dev] Pacman 3.0.5-3: strange behaviour
by Giovanni Scafora 18 Aug '07

18 Aug '07

Hi guys, I noticed that new pacman release doesn't ever show the message "is up to date" when I run pacman -Sy (or -Syu). Pacman thinks there are always new dbs. -- Giovanni Scafora Arch Linux Trusted User (voidnull) http://www.archlinux.org linuxmania(a)gmail.com

2 3

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-349-gf81e619
by Dan McGee 17 Aug '07

17 Aug '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via f81e619974ca47f6dba8c3002577447a71e0f1ec (commit) via fc65a9bcb1722ae9768518aa921c5f02cf282b96 (commit) via 16c91bedf844f5f7764aa78b801ca023810611c3 (commit) via 1cb6d80631359306d70bbe5fbcbbca1474594ada (commit) via ba67fdae63122b40d8fb4abf9c1c7695999f143d (commit) via ee977019e39fe33a508a60abe4a3ad7eff819b74 (commit) via 47cada81a0a8fdd4c77645233ed8406b829b552c (commit) via 8038190c7c4786e1c49494eea1b40cdddcbd5136 (commit) via 0f539832f45fa2c183cfe52c1881b3acaa54e3e6 (commit) via 064de149b7e4f6ea2891ae522102c4ae958f55e5 (commit) via f4dbb204f1ad99179c145558ababf1190f067083 (commit) via a65ad4efc1102449a087b464d3b8d4670a614b99 (commit) via 49c29e16b31ce35e81be210bc74a3834530bb9e0 (commit) from 47ee89ada9d13935510ef92541649901789c23c8 (commit) - Log ----------------------------------------------------------------- commit f81e619974ca47f6dba8c3002577447a71e0f1ec Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Fri Aug 17 14:07:56 2007 +0200 libalpm/sync.c : don't go back on STATE_PREPARED when committing fails. In my opinion, a commiting transaction that failed isn't equivalent to a prepared transaction. Some things could have been done in the meantime. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit fc65a9bcb1722ae9768518aa921c5f02cf282b96 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 16 16:19:06 2007 -0400 Fix some errors spit out by -Wextra Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 16c91bedf844f5f7764aa78b801ca023810611c3 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 16 15:03:00 2007 -0400 configure.ac: Clean up and document version number increments Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1cb6d80631359306d70bbe5fbcbbca1474594ada Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Wed Jul 25 22:49:32 2007 +0100 Clean up MD5 code. * Move alpm md5 functions to lib/libalpm/util.c * Remove unneeded includes for md5.h * Replace md5 implementation with one from http://www.xyssl.org Dan: clean up XySSL code by removing parts we don't use, and add a note saying what changed. Dan: fix alpm_get_md5sum, off by one error on the malloc call and other small things. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ba67fdae63122b40d8fb4abf9c1c7695999f143d Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Wed Jul 25 22:35:29 2007 +0100 Remove support for SHA1 from pacman. There's no need for a second hashing algorithm. MD5 serves the purpose of verifying that a package file hasn't been corrupted during download. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ee977019e39fe33a508a60abe4a3ad7eff819b74 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Mon Jul 9 17:02:29 2007 +0200 libalpm/db.c : add alpm_db_unregister_all. This basically moves the code from alpm_release, which was mostly about unregistering all databases, to a safer alpm_db_unregister_all. This allows to avoid modifying the dbs_sync list while iterating over it, and and also prevent alpm_release from looping infinitely when a database can't be unregistered. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 47cada81a0a8fdd4c77645233ed8406b829b552c Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 16 11:04:58 2007 -0400 pacman.c: Alphabetize listing of query options Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8038190c7c4786e1c49494eea1b40cdddcbd5136 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sat Jul 7 21:24:30 2007 +0200 libalpm/db.c : allow unregistering db for interrupted transaction. This prevents alpm_release to loop infinitely in case of an interrupted transaction, where the database wasn't unregistered. alpm_release should probably also be fixed, as it can still loop if db_unregister fails for another reason. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 0f539832f45fa2c183cfe52c1881b3acaa54e3e6 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sun Jul 15 00:44:06 2007 +0200 new query options : explicit (-Qe) and deps (-Qd). The t shortcut for --test was removed, the orphan option (previously -Qe) was renamed to -Qt, -Qe lists all packages installed explictly, and -Qd lists all packages installed as dependencies. Besides, t can be combined with either e or d. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 064de149b7e4f6ea2891ae522102c4ae958f55e5 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sat Jul 14 20:45:30 2007 +0200 libalpm/query.c : makes orphans and foreign options as filters. The --foreign and --orphans functions now behave as a filter for the other options. This cleans the code a bit, and will make easier the adding of new filter options, like explicit (show only explictly installed packages) or depends (show only packages installed as dependencies). Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit f4dbb204f1ad99179c145558ababf1190f067083 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Aug 15 20:16:46 2007 -0400 src/util: Clean up headers and a few fixes Remove some unnecessary headers in the two utilities as well as fix a possible non-null termination issue in vercmp. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a65ad4efc1102449a087b464d3b8d4670a614b99 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Aug 14 10:14:35 2007 -0400 package.h: implement origin_data union in pmpkg_t struct We were using a void *data element in pmpkg_t before, which is unsafe by its nature of being untyped. Reimplement data as origin_data being a union that can hold either a path to a package file or a pointer to a cache database, and make the other necesary updates in the code to reflect this. See package.h for details. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 49c29e16b31ce35e81be210bc74a3834530bb9e0 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Aug 14 09:33:55 2007 -0400 Code cleanup Remove the commented desc_localized stuff, we can find it later in version control. Also remove some unnecessary includes of the stat header and use -fstack-protector-all which is a bit more broad. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: configure.ac | 72 +++++- contrib/vimproject | 2 - lib/libalpm/Makefile.am | 2 - lib/libalpm/add.c | 53 ++--- lib/libalpm/alpm.c | 16 +-- lib/libalpm/alpm.h | 4 +- lib/libalpm/backup.c | 6 +- lib/libalpm/be_files.c | 49 +---- lib/libalpm/cache.c | 3 +- lib/libalpm/conflict.c | 3 - lib/libalpm/db.c | 64 +++++- lib/libalpm/db.h | 1 + lib/libalpm/md5.c | 561 ++++++++++++++++++++++++---------------------- lib/libalpm/md5.h | 122 +++++++---- lib/libalpm/md5driver.c | 93 -------- lib/libalpm/package.c | 153 ++++---------- lib/libalpm/package.h | 12 +- lib/libalpm/remove.c | 6 +- lib/libalpm/sha1.c | 431 ------------------------------------ lib/libalpm/sha1.h | 72 ------ lib/libalpm/sync.c | 25 +-- lib/libalpm/trans.c | 2 +- lib/libalpm/trans.h | 2 +- lib/libalpm/util.c | 45 ++++- src/pacman/callback.c | 3 +- src/pacman/conf.h | 2 + src/pacman/package.c | 21 +-- src/pacman/pacman.c | 36 ++-- src/pacman/query.c | 215 +++++++----------- src/pacman/sync.c | 1 - src/pacman/util.c | 2 +- src/util/Makefile.am | 2 +- src/util/testpkg.c | 8 +- src/util/vercmp.c | 23 +- 34 files changed, 742 insertions(+), 1370 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 47ee89ada9d13935510ef92541649901789c23c8 \ f81e619974ca47f6dba8c3002577447a71e0f1ec

2 1

[pacman-dev] CVS update of pacman-lib/lib/libalpm (add.c)
by Dan McGee 17 Aug '07

17 Aug '07

Date: Friday, August 17, 2007 @ 16:07:05 Author: dan Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: add.c (1.133 -> 1.134) Fix symlink overwrite issue in CVS -------+ add.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) Index: pacman-lib/lib/libalpm/add.c diff -u pacman-lib/lib/libalpm/add.c:1.133 pacman-lib/lib/libalpm/add.c:1.134 --- pacman-lib/lib/libalpm/add.c:1.133 Wed Jun 20 17:52:24 2007 +++ pacman-lib/lib/libalpm/add.c Fri Aug 17 16:07:05 2007 @@ -700,7 +700,9 @@ archive_entry_set_pathname(entry, filename); - if(archive_read_extract(archive, entry, ARCHIVE_EXTRACT_FLAGS) != ARCHIVE_OK) { + int ret = archive_read_extract(archive, entry, + ARCHIVE_EXTRACT_FLAGS | ARCHIVE_EXTRACT_NO_OVERWRITE); + if(ret != ARCHIVE_OK && ret != ARCHIVE_WARN) { _alpm_log(PM_LOG_ERROR, _("could not extract %s (%s)"), filename, strerror(errno)); alpm_logaction(_("error: could not extract %s (%s)"), filename, strerror(errno)); errors++;

1 0

[pacman-dev] Better handling of broken databases
by Xavier 16 Aug '07

16 Aug '07

An user had a broken database with duplicated entries, and pacman behaved quite weird with it : http://bbs.archlinux.org/viewtopic.php?id=36117 It's maybe possible to handle better such cases, but that's a low priority todo.

1 0

[pacman-dev] _alpm_checkconflicts may add duplicated entries to baddeps
by Nagy Gabor 14 Aug '07

14 Aug '07

Hi! As I see, if there is a conflict in package list, then we may add this twice to the conflict list (because find that conflict twice). Anyway, I don't like this assymmetric "conflict" storing. Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

3 4

[pacman-dev] resolvedeps<->resolveconflict discussion
by Nagy Gabor 14 Aug '07

14 Aug '07

Hi! Now I'd like to discuss some things I dislike in the current resolvedeps<->resolveconflict relation (they are not bugs just annoying things): 1. They are too separated from each other: I mean, we first resolve deps, but this may cause conflict, we resolve it by removing some (target) packages [we stop here now], but this may cause some broken deps, we should resolve them (by adding new targets), but this may cause a conflict again... Possible (semi)solution: resolvedeps should choose a non-conflicting package (yes, monster functions are the more effective again ;-) 2. Resolveconflicts may cause orphan install: If we remove a package from the sync target list, we don't touch its induced dependencies, so we may install some unneeded dependencies (which we pulled earlier to the target list during dep resolution). Possible solution: pmsyncpkg_t-like induced dependency-storing or we could use something similar to canremove. (On the other hand if it is needed to remove a dependency (from the target list) to resolve a conflict, we could do a cascade remove; but it's easier to say that an unresolveable conflict detected.) Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

1 0

[pacman-dev] [patch] resolveconflict clean-up
by Nagy Gabor 13 Aug '07

13 Aug '07

Hi! If you want to apply this patch, please follow my patch flow ;-) I think my alpm_sync_prepare is much cleaner than the old one. As you see, sync040.py and sync041.py now fails, because patched pacman autoresolves target vs target conflict now (I marked this as TODO: user should be asked to 'remove pkg1' or 'remove pkg2' or don't resolve this conflict.) Bye, ngaba Ps: Both the old and patched version of sync_prepare have a problem: If conflict resolution induces a replace, we can loose the PM_SYNC_TYPE_DEPEND information!!

3 4

[pacman-dev] [patch] small speed-up and fix in conflict.c
by Nagy Gabor 13 Aug '07

13 Aug '07

Hi! Patch attached. ngaba

2 1

[pacman-dev] [patch] conflict.c memleak fix
by Nagy Gabor 13 Aug '07

13 Aug '07

Hi! This is just a small memleak fix. Bye, ngaba

3 2

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-336-g47ee89a
by Dan McGee 13 Aug '07

13 Aug '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via 47ee89ada9d13935510ef92541649901789c23c8 (commit) via f2c7187e432a3abc732d53e0a9d95f44f9338535 (commit) via 3a0a4db12936199efd0ee1714b7219fab3d83392 (commit) via ab06221521cd38299743f33c3c5a0d81cbd3f589 (commit) via b96922679e4ed14aa687bf194b766d82a1d10577 (commit) via dc5e6d5d87ceee61c38a8ecc2a87e99d49887b83 (commit) via e0ffc60c5310fbc0a8030fee86bf6f3931ba2d6e (commit) via e4f56298be2b2a4da16b1bdb61b08276cdabea27 (commit) via ce10b5f1e2552f2e2f0117b57c7305652ffd4fd1 (commit) via 178c1d228d5b910908e16f9c3715e318d6df4882 (commit) from ea9a756eeaca7398c0860b55f8abe2932ad195bd (commit) - Log ----------------------------------------------------------------- commit 47ee89ada9d13935510ef92541649901789c23c8 Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Sat Aug 4 01:17:54 2007 +0100 scripts/makepkg.sh.in: Fix 2 bugs - Downloaded files weren't being copied to $srcdir - Counter wasn't being incremented for checksum checks Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> commit f2c7187e432a3abc732d53e0a9d95f44f9338535 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Thu Aug 9 09:36:20 2007 +0200 pacman/sync.c : return 0 on succesful search, 1 otherwise. Closes FS#7690. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 3a0a4db12936199efd0ee1714b7219fab3d83392 Author: Nagy Gabor <ngaba(a)petra.hos.u-szeged.hu> Date: Sun Aug 5 17:50:24 2007 +0200 libalpm/conflict.c : small speed-up and fix. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit ab06221521cd38299743f33c3c5a0d81cbd3f589 Author: Nagy Gabor <ngaba(a)petra.hos.u-szeged.hu> Date: Sat Aug 4 17:11:03 2007 +0200 libalpm/conflict.c : small memleak fix. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit b96922679e4ed14aa687bf194b766d82a1d10577 Author: Nagy Gabor <ngaba(a)petra.hos.u-szeged.hu> Date: Sat Aug 4 17:08:03 2007 +0200 Add two pactest for versioned conflicts. A side effect of the previous commit ( ea9a756eeaca7398c0860b55f8abe2932ad195bd ) is that it's now possible to use versioned conflicts. Add two new conflict pactests for showing it. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit dc5e6d5d87ceee61c38a8ecc2a87e99d49887b83 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jul 30 23:11:32 2007 -0400 Update mtune/mcpu/march comments in makepkg.conf Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e0ffc60c5310fbc0a8030fee86bf6f3931ba2d6e Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jul 26 20:21:38 2007 -0400 Fix re-pacman locale issues Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e4f56298be2b2a4da16b1bdb61b08276cdabea27 Author: Juan Pablo González Tognarelli <jotapesan(a)gmail.com> Date: Thu Jul 26 19:05:57 2007 -0400 Update Spanish translation Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ce10b5f1e2552f2e2f0117b57c7305652ffd4fd1 Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Wed Jul 25 22:15:02 2007 +0100 Fix gettext output in repo-{add,remove} Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 178c1d228d5b910908e16f9c3715e318d6df4882 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jul 23 16:58:04 2007 -0400 Cleanup of pacman.c and addition of default paths to frontend Instead of barfing when the root path and db path haven't been defined, have pacman set them to some sane defaults when they aren't specified on either the command line or the config file. Also do some cleaning of error output and Doxygen comments. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: contrib/re-pacman | 2 + etc/makepkg.conf.in | 4 +- lib/libalpm/conflict.c | 22 +++------- pactest/tests/conflict002.py | 13 ++++++ pactest/tests/conflict003.py | 13 ++++++ po/es.po | 23 +++++------ scripts/makepkg.sh.in | 3 + scripts/repo-add.sh.in | 29 ++++++------- scripts/repo-remove.sh.in | 21 +++++----- src/pacman/Makefile.am | 3 + src/pacman/pacman.c | 91 +++++++++++++++++++++++------------------ src/pacman/sync.c | 5 ++- 12 files changed, 133 insertions(+), 96 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ ea9a756eeaca7398c0860b55f8abe2932ad195bd \ 47ee89ada9d13935510ef92541649901789c23c8

1 0

[pacman-dev] [patch] for sync1003.py
by Nagy Gabor 06 Aug '07

06 Aug '07

Hi! Patch attached. NOTE: this affects resolvedeps too. TODO: I did a dirty hack in sync_prepare (now sync990.py fails), I will clean it up later. Bye, ngaba

4 10

Re: [pacman-dev] [patch] for sync1003.py
by ngaba＠bibl.u-szeged.hu 03 Aug '07

03 Aug '07

> TODO: I did a dirty hack in sync_prepare (now sync990.py fails), I will Well, that patch is broken: pkg3 should be installed (pkg1 needs it.) Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

1 0

[pacman-dev] sync_prepare clean-up III, TODO
by Nagy Gabor 30 Jul '07

30 Jul '07

Hi! Well, the remaining part is very messy and disappointing: The whole conflict checking should be refactored: as I assume this wants to do something similar to checkdeps(...TRANS_TYPE_UPDATE), but that is buggy: this should be fixed as deps.c (conflict001.py shows this.) However, as Xavier said earlier a new data stucture is needed for conflicts; and all conflict resolve stuff should be moved to conflict.c. The second pactest file shows checkdeps cannot get enough information from sync.c to figure out if there is any broken dependency: it should know BOTH the remove and upgrade list to decide properly. Well, both of this bugs needs hard work: a proper conflict resolution is a difficult (probably recursive) problem, it would be easier to let user do this work manually. Bye, ngaba PS: I don't like inefficient pmsyncpkg_t neither.

3 13

[pacman-dev] pacman_deptest function
by Xavier 28 Jul '07

28 Jul '07

I've benn looking at that function again, and here is what I saw in Andrew's commit : +/* TODO: This should use _alpm_checkdeps() */ int pacman_deptest(alpm_list_t *targets) As I tried to explain there : http://www.archlinux.org/pipermail/pacman-dev/2007-June/008654.html checkdeps isn't really what we want. Here, we just want to check that a list of dependencies from the PKGBUILD is already installed locally. checkdeps however checks if we can install a package, by checking its dependencies. And checkdeps gets this information from the sync or local databases, it can't really get the deps list from an external PKGBUILD. But there is at least one way to do that, it's creating a dummy package, with the same list of dependencies as in the PKGBUILD, and work with that in libalpm. I looked at the history of deptest(), and it's actually what Aurelien did at some point : http://projects.archlinux.org/git/?p=pacman.git;a=commit;h=0756af675d6b99a1… http://projects.archlinux.org/git/?p=pacman.git;a=commit;h=cda9c6671ced31a7… But it was rather messy, so Aaron cleaned that : http://projects.archlinux.org/git/?p=pacman.git;a=commit;h=4dd6c9222879198e… So well, I still have the same opinion that in my earlier answer to Andrew, what pacman_deptest does is just the mythic "search for satisfier" function that Nagy has been talked about for ages, and that is used everywhere in the code :)

2 2

[pacman-dev] Issue while testing largefile support
by Dan McGee 27 Jul '07

27 Jul '07

Interesting debug output while testing largefile support in pacman (http://bugs.archlinux.org/task/7578) [14:55:58] debug: config: finished parsing /usr/local/etc/pacman.conf [14:55:58] debug: registering database 'local' [14:55:58] debug: opening database 'local' [14:55:58] debug: opening database from path '/usr/local/var/lib/pacman/local/' loading package data... [14:55:58] debug: loading target 'largepackage-1-3-i686.pkg.tar.gz' [14:55:58] debug: reading 'largepackage' metadata done. checking dependencies... [14:56:58] debug: looking for unsatisfied dependencies [14:56:58] debug: loading package cache for repository 'local' [14:56:58] debug: cannot find package installed 'largepackage' [14:56:58] debug: looking for conflicts [14:56:58] debug: checkconflicts: db vs target 'largepackage' [14:56:59] debug: sorting by dependencies [14:56:59] debug: started sorting dependencies [14:56:59] debug: sorting dependencies finished done. checking for file conflicts... [14:56:59] debug: looking for file conflicts [14:56:59] debug: searching for filesystem conflicts: largepackage [14:56:59] debug: /opt/ is a directory, not a conflict [14:56:59] debug: /opt/largefile is a directory, not a conflict done. installing largepackage... [14:56:59] debug: adding package largepackage-1-3 [14:56:59] debug: extracting files [14:56:59] debug: decompression progress: 0.001907% (40960 / -2146827972) [14:56:59] debug: decompression progress: 0.001907% (40960 / -2146827972) [14:56:59] debug: decompression progress: 0.001907% (40960 / -2146827972) [14:56:59] debug: extracting /opt/ [14:56:59] debug: warning extracting opt/ (Already exists: File exists) [14:56:59] debug: decompression progress: 0.001907% (40960 / -2146827972) [14:56:59] debug: extracting /opt/largefile [14:59:25] debug: updating database [14:59:25] debug: adding database entry 'largepackage' [14:59:25] debug: writing largepackage-1-3 DESC information back to db [14:59:25] debug: writing largepackage-1-3 FILES information back to db [14:59:25] debug: writing largepackage-1-3 DEPENDS information back to db [14:59:25] debug: adding entry 'largepackage' in 'local' cache [14:59:25] debug: package has no dependencies, no other packages to update done. [14:59:25] debug: running "ldconfig -r /" Here is the package filelist: opt/ (directory) opt/largefile (a 2GB sized normal file) Notice what the above output says- that /opt/largefile is a directory, not a conflict. Clearly a problem here. This is pacman built off my working branch, so not including any recent patches from Andrew or Chantry with regards to the overwrite stuff. -Dan

4 9

[pacman-dev] Pointless public/private functions
by Andrew Fyfe 27 Jul '07

27 Jul '07

Throughout the libalpm and pacman code there's a lot of public functions that simply wrap a private function (maybe with some asserts). What's the point? Why not have the asserts in the private function and make the private function public reducing the number of function being called? Andrew

3 2

[pacman-dev] Parsing config file question
by Andrew Fyfe 27 Jul '07

27 Jul '07

Why does pacman do this... if(strcmp(key, "ShowSize") == 0 || strcmp(upperkey, "SHOWSIZE") == 0) Is there any reason why we can't do... if(strcmp(upperkey, "SHOWSIZE") == 0) Andrew PS Just found the ILoveCandy option :D

3 4

[pacman-dev] po file for es translation
by Juan Pablo González T. 26 Jul '07

26 Jul '07

Hi, an update to the spanish translation of pacman attached to this mail. __________________________________________________ Correo Yahoo! Espacio para todos tus mensajes, antivirus y antispam ¡gratis! Regístrate ya - http://correo.espanol.yahoo.com/

2 1

[pacman-dev] [PATCH] Fix gettext output in repo-{add,remove}
by Andrew Fyfe 26 Jul '07

26 Jul '07

Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> --- scripts/repo-add.sh.in | 29 ++++++++++++++--------------- scripts/repo-remove.sh.in | 21 ++++++++++----------- 2 files changed, 24 insertions(+), 26 deletions(-) diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in index 0992b9d..f8c5de2 100644 --- a/scripts/repo-add.sh.in +++ b/scripts/repo-add.sh.in @@ -28,7 +28,6 @@ myver='@PACKAGE_VERSION@' FORCE=0 REPO_DB_FILE="" -TMP_DIR="" # print usage instructions usage() { @@ -136,14 +135,14 @@ db_write_entry() # ensure $pkgname and $pkgver variables were found if [ -z "$pkgname" -o -z "$pkgver" ]; then - echo "$(gettext " error: invalid package file")" + printf "$(gettext " error: invalid package file")\n" return 1 fi # remove any other package in the DB with same name for existing in *; do if [ "${existing%-*-*}" = "$pkgname" ]; then - echo "$(gettext ":: removing existing package '%s'")" $existing + printf "$(gettext ":: removing existing package '%s'")\n" "$existing" rm -rf $existing fi done @@ -153,7 +152,7 @@ db_write_entry() cd "$pkgname-$pkgver" # create desc entry - echo "$(gettext ":: creating 'desc' db entry")" + printf "$(gettext ":: creating 'desc' db entry")\n" echo -e "%FILENAME%\n$(basename $1)\n" >>desc echo -e "%NAME%\n$pkgname\n" >>desc echo -e "%VERSION%\n$pkgver\n" >>desc @@ -171,7 +170,7 @@ db_write_entry() # compute checksums for chk in ${DB_CHECKSUMS[@]}; do name="$(checksum_name $chk)" - echo "$(gettext ":: computing %s checksums")" $name + printf "$(gettext ":: computing %s checksums")\n" "$name" if [ -n "$name" ]; then echo -e "%$name%\n$(get_checksum $chk $pkgfile)\n" >>desc fi @@ -240,7 +239,7 @@ fi if [ -r @sysconfdir@/makepkg.conf ]; then source @sysconfdir@/makepkg.conf else - echo "$(gettext "ERROR: /etc/makepkg.conf not found. Can not continue.")" >&2 + printf "$(gettext "ERROR: /etc/makepkg.conf not found. Can not continue.")\n" >&2 exit 1 # $E_CONFIG_ERROR fi @@ -251,7 +250,7 @@ fi # main routine if [ $# -gt 1 ]; then gstmpdir=$(mktemp -d /tmp/gensync.XXXXXXXXXX) || (\ - echo "$(gettext "cannot create temp directory for database building")"; \ + printf "$(gettext "cannot create temp directory for database building")\n"; \ exit 1) success=0 @@ -262,18 +261,18 @@ if [ $# -gt 1 ]; then elif [ -z "$REPO_DB_FILE" ]; then REPO_DB_FILE="$(readlink -f $arg)" if ! test_repo_db_file; then - echo "$(gettext "error: repository file '%s' is not a proper pacman db")" $REPO_DB_FILE + printf "$(gettext "error: repository file '%s' is not a proper pacman db")\n" "$REPO_DB_FILE" exit 1 elif [ -f "$REPO_DB_FILE" ]; then - echo "$(gettext ":: extracting database to a temporary location")" + printf "$(gettext ":: extracting database to a temporary location")\n" bsdtar -xf "$REPO_DB_FILE" -C "$gstmpdir" fi else if [ -f "$arg" ]; then if ! bsdtar -tf "$arg" .PKGINFO 2>&1 >/dev/null; then - echo "$(gettext "error: '%s' is not a package file, skipping")" $arg + printf "$(gettext "error: '%s' is not a package file, skipping")\n" "$arg" else - echo "$(gettext ":: adding package '%s'")" $arg + printf "$(gettext ":: adding package '%s'")\n" "$arg" this_dir="$(pwd)" if db_write_entry "$arg"; then @@ -282,14 +281,14 @@ if [ $# -gt 1 ]; then cd $this_dir fi else - echo "$(gettext "error: package '%s' not found")" $arg + printf "$(gettext "error: package '%s' not found")\n" "$arg" fi fi done # if all operations were a success, rezip database if [ "$success" = "1" ]; then - echo "$(gettext ":: creating updated database file %s")" ${REPO_DB_FILE} + printf "$(gettext ":: creating updated database file %s")\n" "$REPO_DB_FILE" cd $gstmpdir if [ -n "$(ls)" ]; then [ -f "${REPO_DB_FILE}.old" ] && rm "${REPO_DB_FILE}.old" @@ -297,12 +296,12 @@ if [ $# -gt 1 ]; then case "$DB_COMPRESSION" in gz) bsdtar -c * | gzip -9 >$REPO_DB_FILE ;; bz2) bsdtar -c * | bzip2 -9 >$REPO_DB_FILE ;; - *) echo "$(gettext "warning: no compression set")" + *) printf "$(gettext "warning: no compression set")\n" bsdtar -c * >$REPO_DB_FILE;; esac fi else - echo "$(gettext ":: no packages modified, nothing to do")" + printf "$(gettext ":: no packages modified, nothing to do")\n" fi fi diff --git a/scripts/repo-remove.sh.in b/scripts/repo-remove.sh.in index 7943b2b..49ff358 100644 --- a/scripts/repo-remove.sh.in +++ b/scripts/repo-remove.sh.in @@ -28,7 +28,6 @@ myver='@PACKAGE_VERSION@' FORCE=0 REPO_DB_FILE="" -TMP_DIR="" # print usage instructions usage() { @@ -66,7 +65,7 @@ db_remove_entry() # remove any other package in the DB with same name for existing in *; do if [ "${existing%-*-*}" = "$1" ]; then - echo "$(gettext ":: removing existing package '%s'")" $existing + printf "$(gettext ":: removing existing package '%s'")\n" "$existing" rm -rf $existing fi done @@ -96,7 +95,7 @@ fi if [ -r @sysconfdir@/makepkg.conf ]; then source @sysconfdir@/makepkg.conf else - echo "$(gettext "ERROR: /etc/makepkg.conf not found. Can not continue.")" >&2 + printf "$(gettext "ERROR: /etc/makepkg.conf not found. Can not continue.")\n" >&2 exit 1 # $E_CONFIG_ERROR fi @@ -107,7 +106,7 @@ fi # main routine if [ $# -gt 1 ]; then gstmpdir=$(mktemp -d /tmp/gensync.XXXXXXXXXX) || (\ - echo "$(gettext "cannot create temp directory for database building")"; \ + printf "$(gettext "cannot create temp directory for database building")\n"; \ exit 1) success=0 @@ -116,20 +115,20 @@ if [ $# -gt 1 ]; then if [ -z "$REPO_DB_FILE" ]; then REPO_DB_FILE="$(readlink -f $arg)" if ! test_repo_db_file; then - echo "$(gettext "error: repository file '%s' is not a proper pacman db")" $REPO_DB_FILE + printf "$(gettext "error: repository file '%s' is not a proper pacman db")\n" "$REPO_DB_FILE" exit 1 elif [ -f "$REPO_DB_FILE" ]; then - echo "$(gettext ":: extracting database to a temporary location")" + printf "$(gettext ":: extracting database to a temporary location")\n" bsdtar -xf "$REPO_DB_FILE" -C "$gstmpdir" fi else - echo "$(gettext ":: searching for package '%s'")" + printf "$(gettext ":: searching for package '%s'")\n" this_dir="$(pwd)" if db_remove_entry "$arg"; then success=1 else - echo "$(gettext "error: package matching '%s' not found")" $arg + printf "$(gettext "error: package matching '%s' not found")\n" "$arg" fi cd $this_dir fi @@ -137,7 +136,7 @@ if [ $# -gt 1 ]; then # if all operations were a success, rezip database if [ "$success" = "1" ]; then - echo "$(gettext ":: creating updated database file %s")" ${REPO_DB_FILE} + printf "$(gettext ":: creating updated database file %s")\n" "$REPO_DB_FILE" cd $gstmpdir if [ -n "$(ls)" ]; then [ -f "${REPO_DB_FILE}.old" ] && rm "${REPO_DB_FILE}.old" @@ -145,12 +144,12 @@ if [ $# -gt 1 ]; then case "$DB_COMPRESSION" in gz) bsdtar -c * | gzip -9 >$REPO_DB_FILE ;; bz2) bsdtar -c * | bzip2 -9 >$REPO_DB_FILE ;; - *) echo "$(gettext "warning: no compression set")" + *) printf "$(gettext "warning: no compression set")\n" bsdtar -c * >$REPO_DB_FILE;; esac fi else - echo "$(gettext ":: no packages modified, nothing to do")" + printf "$(gettext ":: no packages modified, nothing to do")\n" fi fi -- 1.5.2.4

2 1

[pacman-dev] [patch] sync_prepare clean-up I: 2in1 -Se bugfix (fwd)
by Nagy Gabor 24 Jul '07

24 Jul '07

Hi! The pactest shows 2 bugs: Pacman should add pkg3 to the transaction, because pkg2 depends on pkg3 (bug1), but at least checkdeps should show that a dependency is missing (bug2): here a wrong variable (list) is passed to checkdeps. The patch is designed for bugfix for bug1, but also fixes bug2. Bye, ngaba

2 4