- Pacman-dev - lists.archlinux.org

[pacman-dev] [GIT] The official pacman repository annotated tag, v3.0.5 now at v3.0.4-5-g0cfb412
by Dan McGee 29 Oct '07

29 Oct '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 annotated tag, v3.0.5, to 77fa42dd001ecf309af7f72faeccd101f0d7c9f4 (tag) replaces v3.0.4-4 tagged by Dan McGee <dan(a)archlinux.org> on 2007-10-28 23:50 -0400 - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- hooks/update --- Git Source Code Management System hooks/update refs/tags/v3.0.5 \ 0000000000000000000000000000000000000000 \ 77fa42dd001ecf309af7f72faeccd101f0d7c9f4

1 0

[pacman-dev] [GIT] The official pacman repository annotated tag, v3.0.6 now at v3.0.5-7-ga06d91f
by Dan McGee 29 Oct '07

29 Oct '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 annotated tag, v3.0.6, to 723b37f6fac724405d9fe5abf75b46d0e7a0adda (tag) replaces v3.0.5-6 tagged by Dan McGee <dan(a)archlinux.org> on 2007-10-28 23:51 -0400 - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- hooks/update --- Git Source Code Management System hooks/update refs/tags/v3.0.6 \ 0000000000000000000000000000000000000000 \ 723b37f6fac724405d9fe5abf75b46d0e7a0adda

1 0

[pacman-dev] [GIT] The official pacman repository branch, master now at v3.0.0-492-g671224b
by Dan McGee 29 Oct '07

29 Oct '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 671224b64c9478644125deec1496a27e8f579da9 (commit) via 0dc34f496dca3d916eae85a0357049b8b9d711bb (commit) via 3e3e7a97a629cc86a53044766039c66d85cf67cb (commit) via 8a9c83dd4bffff575a21207248e7acaae5a0d6f9 (commit) via 3e1b72f4f2b3f8766f9ad3493ad736d81263c9c1 (commit) via a8731ff2f7be564b745a4447df9c2157c5ee0777 (commit) via b38e046621e173a1194eb354e6b8d2525fa60242 (commit) via 17f99a0f0042efeb4c5822a6446918f1ac852e32 (commit) via d903fc607ee2aa2527202f1e54a44be325eabe48 (commit) via b609cb0a5dfe595318975dfe4c6c37a168b7d979 (commit) from 0cff7c6bdf5461c3286edc940aabb88d63345381 (commit) - Log ----------------------------------------------------------------- commit 671224b64c9478644125deec1496a27e8f579da9 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Oct 28 22:24:11 2007 -0500 pactest: generate the filelist the correct way Actually use python commands to generate the filelist instead of os.system calls that don't necessarily work everywhere. Noticed when running "make check" on FreeBSD where the tar program is actually bsdtar. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 0dc34f496dca3d916eae85a0357049b8b9d711bb Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sun Oct 28 13:37:49 2007 +0100 extend type001 pactest with a symlink. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 3e3e7a97a629cc86a53044766039c66d85cf67cb Author: Dan McGee <dan(a)archlinux.org> Date: Sun Oct 28 21:18:39 2007 -0500 makepkg: add explicit dir to find calls Some of the find calls did not have a path, which isn't valid in some versions of the find utility. Add a . for compatibility. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8a9c83dd4bffff575a21207248e7acaae5a0d6f9 Author: Scott Horowitz <stonecrest(a)gmail.com> Date: Fri Oct 26 20:32:09 2007 -0600 Integrate versionpkg into makepkg If PKGBUILDs use the correct svn/cvs/etc variables, determine the latest revision number and force this as the pkgver for building. Signed-off-by: Scott Horowitz <stonecrest(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3e1b72f4f2b3f8766f9ad3493ad736d81263c9c1 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Oct 26 20:40:20 2007 -0500 Clean up LDADD usage We had a lot of unnecessary overstatements of libraries to include on linking, and autoconf/automake takes care of this for us. This also helps some compilation issues on other platforms. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a8731ff2f7be564b745a4447df9c2157c5ee0777 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Oct 26 20:31:25 2007 -0500 Fix mcheck detection and usage Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b38e046621e173a1194eb354e6b8d2525fa60242 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Oct 26 19:09:21 2007 -0500 Explicitly cast void* to char* during printf call If we don't have an explicit cast, make fails during -Wall -Werror. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 17f99a0f0042efeb4c5822a6446918f1ac852e32 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Fri Oct 26 16:37:47 2007 +0200 dependency error message fix in libalpm/remove.c Normally you must never see that error message. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d903fc607ee2aa2527202f1e54a44be325eabe48 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Sat Oct 27 01:14:35 2007 +0200 Dependency error/log messages fix The old code used only the depend.name in messages, which might have not been informative. The new code uses the whole dependency string in %DEPENDS% format. (Dan: slight English clarification in one of the messages) Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b609cb0a5dfe595318975dfe4c6c37a168b7d979 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Oct 26 18:36:08 2007 -0500 makepkg: readd -p option that was lost in getopt conversion The -p option was accidentally dropped in commit 54b71f0427e87e6d525423df06f8a06f2b71c518. Readd it. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: configure.ac | 12 +++-- lib/libalpm/Makefile.am | 1 - lib/libalpm/deps.c | 18 +++++--- lib/libalpm/remove.c | 2 +- lib/libalpm/sync.c | 4 +- pactest/pmpkg.py | 22 ++++++++--- pactest/tests/type001.py | 14 ++++-- scripts/makepkg.sh.in | 97 ++++++++++++++++++++++++++++++++++++++++++--- src/pacman/Makefile.am | 6 +-- src/pacman/pacman.c | 4 +- src/util/testdb.c | 4 +- 11 files changed, 144 insertions(+), 40 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 0cff7c6bdf5461c3286edc940aabb88d63345381 \ 671224b64c9478644125deec1496a27e8f579da9

1 0

Re: [pacman-dev] [tur-users] A Note about Groups
by Jeff 'codemac' Mickey 28 Oct '07

28 Oct '07

* On Sunday, October 28 2007, Alessio 'mOLOk' Bolognino wrote: >On Mon 2007-10-22 11:21 , Aaron Griffin wrote: >> This is just a note for all of you guys. >> Please DO NOT stick community packages in official groups. This is >> ESPECIALLY important for the base and base-devel groups. >> [...] > >So we can't use, say "vim-plugins' ? That's exactly what I was thinking... I think the stance should be DO NOT put community packages in official groups unless we developers know about it. There are some very "crucial" groups that should not be messed with as the selinux packages make proof of, but why can't packages in [community] be part of something like vim-pulgins? I think this is actually a design flaw in how pacman handles repositories. I think groups should not span repositories. But maybe this is a discussion for the pacman-dev list. Cc'd! // jeff -- .: [ + carpe diem totus tuus + ] :.

1 0

[pacman-dev] dbs should be locked between _alpm_db_open and _alpm_db_close
by Nagy Gabor 27 Oct '07

27 Oct '07

Hi! The problem bellow doesn't appear with the current dummy front-end, but if you want to design a flexible alpm lib, you have to deal with it. The main problem, that the whole pkgcache concept _assumes_ that the db won't be modified by other processes. However, pacman locks db iff he modifies it, so he cannot trust his assumption. Let me explain. Imagine a nice pacman GUI which starts with alpm_initialize, opens the databases and let user do multiple operations. pacman_gui can run all the day, without unregistering the databases, alpm_release etc. etc. The main problem, that if pacman_gui (== libalpm) let an other pacman instance to modify the local db, he should always reload the pkgcache, because it may be outdated! So the following situation can happen: pacman_gui loaded the pkgcache to list the available local packages to the user, and now he is waiting. During this an other pacman instance removed lot of packages (he could lock anf unlock the db, since pacman_gui didn't lock anything yet) and exited. pacman_gui didn't detect anything from this. Now the user ask pacman_gui to install some packages: now he locks the dbs, but he trust its outdated pkgcache: for example its depchecks will use this outdated information... So _alpm_db_open should lock database, and _alpm_db_close should unlock it. The problem here, that now a non-root user cannot write /var/lib/pacman/local/db.lck, but we can solve this somehow. (Note, that I'm suggesting a lock-per-db method here.) Another (not-so-strict but slower) possible solution a .lastupdate-like method to detect if the database was modified or not after pkgcache load, and reload if needed. Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

2 2

[pacman-dev] testdb
by Nagy Gabor 27 Oct '07

27 Oct '07

Hi! First of all, many thanks to Xavier for this cool stuff. Some suggestions/feature requests: -you don't free() anything in testpkg.c; this is not a big problem, but rigorous c-programmers frees everything in even helloworld.c;-) -conflict checking also would be nice and easy to implement -a small speedup suggestion (however, speed is not important here): line 176: data = alpm_checkdeps(db, PM_TRANS_TYPE_ADD, alpm_db_getpkgcache(db)); here you could use a fake_empty_db as a first parameter: -O(n^2) operation -my biggest feature request: an option to fix the problems instead of just listing them (at least rewrite wrong requiredby fields, and a suggestion "pacman -S --asdeps [missingdeps]") I will share my results with you, this is a nice database-breaker-bug history: -testdb found many duplicated requiredby fields; I had to fix them with db-fix.sh + re-pacman + pacman -U (I have no net at home at the moment; see my biggest feature request ;-) -testdb found 4 other wrong requiredby fields (however, extra requiredby field doesn't cause an error now: see remove048.py): this is the category of the REMOVEUPGRADE transaction: requiredby004.py -and an other old bug (fixed): sysvinit needs awk, gawk provides awk; and sysvinit wasn't listed in requiredby field of gawk. Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

5 18

[pacman-dev] pacman doesn't realize about letter package versioning when doing upgrades
by Mateusz Jedrasik 27 Oct '07

27 Oct '07

Simply put: pacman -Syu <snip> warning: openssh: local (4.7p1-3) is newer than core (4.7p1-2) warning: openssl: local (0.9.8g-1) is newer than core (0.9.8f-1) <snip> :: openssh requires openssl>=0.9.8g :-) It simply does not realize that 0.9.8f-1 is newer than 0.9.8g-1. Cheers! //m. -- Mateusz Jędrasik <m.jedrasik(a)gmail.com> tel. +48(79)022-9393, +48(51)69-444-90 http://imachine.szklo.eu.org

3 5

[pacman-dev] small bug in libalpm/remove.c
by Nagy Gabor 27 Oct '07

27 Oct '07

Hi! Look at line 122-123: _alpm_log(PM_LOG_ERROR, _("could not find %s in database -- skipping\n"), miss->depend.name); Probably you want to say miss->target here. You can treat this mail as a patch, since it is very easy to fix it (now I'm working on removing pmdepend_t and I don't want to create a new branch;-) Bye, ngaba

4 9

[pacman-dev] [PATCH] Dependency error/log messages fix
by Nagy Gabor 26 Oct '07

26 Oct '07

From 4ee4c412b7ce0a688a81b1f856ef6462c8ee269d Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Sat, 27 Oct 2007 01:14:35 +0200 Subject: [PATCH] Dependency error/log messages fix The old code used only the depend.name in messages, which might have not been informative. The new code uses the whole dependency string in %DEPENDS% format. --- lib/libalpm/deps.c | 18 +++++++++++------- lib/libalpm/sync.c | 4 +++- src/util/testdb.c | 4 +++- 3 files changed, 17 insertions(+), 9 deletions(-) diff --git a/lib/libalpm/deps.c b/lib/libalpm/deps.c index 7f4fb0b..4beaf70 100644 --- a/lib/libalpm/deps.c +++ b/lib/libalpm/deps.c @@ -306,7 +306,7 @@ alpm_list_t *_alpm_checkdeps(pmdb_t *db, pmtranstype_t op, if(alpm_depcmp(pkg, depend)) { _alpm_log(PM_LOG_DEBUG, "checkdeps: dependency '%s' has moved from '%s' to '%s'\n", - depend->name, alpm_pkg_get_name(oldpkg), alpm_pkg_get_name(pkg)); + k->data, alpm_pkg_get_name(oldpkg), alpm_pkg_get_name(pkg)); satisfied = 1; break; } @@ -322,7 +322,7 @@ alpm_list_t *_alpm_checkdeps(pmdb_t *db, pmtranstype_t op, /* we ignore packages that will be updated because we know * that the updated ones don't satisfy depend */ _alpm_log(PM_LOG_DEBUG, "checkdeps: dependency '%s' satisfied by installed package '%s'\n", - depend->name, alpm_pkg_get_name(pkg)); + k->data, alpm_pkg_get_name(pkg)); satisfied = 1; break; } @@ -380,7 +380,7 @@ alpm_list_t *_alpm_checkdeps(pmdb_t *db, pmtranstype_t op, /* else if still not found... */ if(!found) { _alpm_log(PM_LOG_DEBUG, "missing dependency '%s' for package '%s'\n", - depend->name, alpm_pkg_get_name(tp)); + j->data, alpm_pkg_get_name(tp)); miss = _alpm_depmiss_new(alpm_pkg_get_name(tp), PM_DEP_TYPE_DEPEND, depend->mod, depend->name, depend->version); if(!_alpm_depmiss_isin(miss, baddeps)) { @@ -426,7 +426,7 @@ alpm_list_t *_alpm_checkdeps(pmdb_t *db, pmtranstype_t op, pmpkg_t *pkg = l->data; if(alpm_depcmp(pkg, depend) && !_alpm_pkg_find(alpm_pkg_get_name(pkg), packages)) { _alpm_log(PM_LOG_DEBUG, "checkdeps: dependency '%s' satisfied by installed package '%s'\n", - depend->name, alpm_pkg_get_name(pkg)); + k->data, alpm_pkg_get_name(pkg)); satisfied = 1; break; } @@ -686,8 +686,10 @@ int _alpm_resolvedeps(pmdb_t *local, alpm_list_t *dbs_sync, pmpkg_t *syncpkg, for(j = *list; j && !found; j = j->next) { pmpkg_t *sp = j->data; if(alpm_depcmp(sp, missdep)) { + char *missdepstring = alpm_dep_get_string(missdep); _alpm_log(PM_LOG_DEBUG, "%s satisfies dependency %s -- skipping\n", - alpm_pkg_get_name(sp), missdep->name); + alpm_pkg_get_name(sp), missdepstring); + free(missdepstring); found = 1; } } @@ -713,8 +715,10 @@ int _alpm_resolvedeps(pmdb_t *local, alpm_list_t *dbs_sync, pmpkg_t *syncpkg, } if(!found) { - _alpm_log(PM_LOG_ERROR, _("cannot resolve dependencies for \"%s\" (\"%s\" is not in the package set)\n"), - miss->target, missdep->name); + char *missdepstring = alpm_dep_get_string(missdep); + _alpm_log(PM_LOG_ERROR, _("cannot resolve \"%s\" dependency of \"%s\"\n"), + missdepstring, miss->target); + free(missdepstring); if(data) { if((miss = malloc(sizeof(pmdepmissing_t))) == NULL) { _alpm_log(PM_LOG_ERROR, _("malloc failure: could not allocate %d bytes\n"), sizeof(pmdepmissing_t)); diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 3e12ffb..f4ed6dd 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -674,8 +674,10 @@ int _alpm_sync_prepare(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync pmsyncpkg_t *sp = l->data; pmpkg_t *sppkg = sp->pkg; if(alpm_depcmp(sppkg, &(miss->depend))) { + char *missdepstring = alpm_dep_get_string(&(miss->depend)); _alpm_log(PM_LOG_DEBUG, "sync: dependency '%s' satisfied by package '%s'\n", - miss->depend.name, alpm_pkg_get_name(sppkg)); + missdepstring, alpm_pkg_get_name(sppkg)); + free(missdepstring); satisfied = 1; } } diff --git a/src/util/testdb.c b/src/util/testdb.c index 1331f2c..a06456a 100644 --- a/src/util/testdb.c +++ b/src/util/testdb.c @@ -178,8 +178,10 @@ int main(int argc, char **argv) for(i = data; i; i = alpm_list_next(i)) { pmdepmissing_t *miss = alpm_list_getdata(i); pmdepend_t *dep = alpm_miss_get_dep(miss); + char *depstring = alpm_dep_get_string(dep); printf("missing dependency for %s : %s\n", alpm_miss_get_target(miss), - alpm_dep_get_name(dep)); + depstring); + free(depstring); } /* check requiredby */ -- 1.5.3.4

2 1

[pacman-dev] alpm_list_remove is buggy
by Nagy Gabor 26 Oct '07

26 Oct '07

Hi! That function seems totally odd. /* TODO I modified this to remove ALL matching items. Do we need a remove_first? */ <- this is partly true: *data contains only the last found "i->data", and the function doesn't return with an alpm_list. MEMLEAK!! Bye, ngaba PS: I don't create a patch, because I dunno want do you want: remove the matching _list_ or remove the first matching _item_'s data. My vote: 2nd. (Because then I don't have to resubmit my -Ru patch ;-)

2 2