- Pacman-dev - lists.archlinux.org

[pacman-dev] A prosition for an enhancement - addition of category info to .PKGINFO
by Bozhidar Batsov 13 Jul '07

13 Jul '07

Wouldn't it be wise if the category information was added to each package's .PKGINFO file? That way clients could search for or in package categories and the capabilities of GUI front-ends could enhanced a great deal. As a matter of fact why was this possibility ever considered and if so why was the idea dropped?

7 11

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-296-gc0a7d9d
by Dan McGee 13 Jul '07

13 Jul '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 c0a7d9d82d03d9831f53dbb30357ddb9f10e29ec (commit) via d402583c43a9d409531824573a6021a1965d8ee6 (commit) via fd3a1a92c8ffe5230e40a027838442752a843da9 (commit) via 20f73d6299c08fa79c2c8edd3d1955e208e701cb (commit) via 653fb8fe036363dfb550785c45adcf26f6fcbfbc (commit) via 147a32b32de4ebcbb51c963f2573d9218d9a2eb9 (commit) from cb164c3130f15b22dbf16991c28ba609cd2f79de (commit) - Log ----------------------------------------------------------------- commit c0a7d9d82d03d9831f53dbb30357ddb9f10e29ec Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Thu Jul 12 20:45:02 2007 +0100 Some doxygen comments. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> commit d402583c43a9d409531824573a6021a1965d8ee6 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jul 12 16:00:35 2007 -0400 Remove some more conditional include stuff Signed-off-by: Dan McGee <dan(a)archlinux.org> commit fd3a1a92c8ffe5230e40a027838442752a843da9 Author: Andrew Fyfe <andrew(a)neptune-one.net> Date: Thu Jul 12 20:12:08 2007 +0100 Fix previous makepkg patch, and some pacman output. makepkg: We still need the source files in $srcdir because PKGBUILDS need access to noextract() files and other file not handled by by extract_source(). (eg config files) query.c: Fix some output formating. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> commit 20f73d6299c08fa79c2c8edd3d1955e208e701cb Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jul 12 15:20:43 2007 -0400 Remove versioncmp.{c,h}, clean up selective #ifdefs Remove versioncmp.c by moving all functions to locations that make sense. Move replacement functions (for building without glibc) into util.c where they belong, and do proper checks for them instead of using __sun__, etc. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 653fb8fe036363dfb550785c45adcf26f6fcbfbc Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 11 23:36:13 2007 -0400 Remove 'buildtype' from libalpm and pacman Remove unused buildtype field from pmpkg_t struct and anything associated with it, as it is unused at the moment. If we need to readd it, it is an easy revert of this commit. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 147a32b32de4ebcbb51c963f2573d9218d9a2eb9 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 11 10:59:27 2007 -0400 Add {PKGBUILD,install}.proto to contrib/ directory Also update README and Makefile.am in the contrib/ directory. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: configure.ac | 2 +- contrib/Makefile.am | 6 +- contrib/PKGBUILD.proto | 35 +++++ contrib/README | 7 + contrib/install.proto | 40 ++++++ contrib/vimproject | 2 - lib/libalpm/Makefile.am | 3 +- lib/libalpm/add.c | 1 - lib/libalpm/be_files.c | 12 -- lib/libalpm/deps.c | 48 +++++++- lib/libalpm/package.c | 112 ++++++++++++++--- lib/libalpm/package.h | 4 +- lib/libalpm/po/POTFILES.in | 1 - lib/libalpm/remove.c | 1 - lib/libalpm/sync.c | 1 - lib/libalpm/trans.c | 8 +- lib/libalpm/util.c | 131 +++++++++++++++---- lib/libalpm/util.h | 8 +- lib/libalpm/versioncmp.c | 294 -------------------------------------------- lib/libalpm/versioncmp.h | 33 ----- scripts/makepkg.sh.in | 2 + src/pacman/package.c | 4 +- src/pacman/query.c | 6 +- 23 files changed, 347 insertions(+), 414 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ cb164c3130f15b22dbf16991c28ba609cd2f79de \ c0a7d9d82d03d9831f53dbb30357ddb9f10e29ec

1 0

[pacman-dev] [PATCH] Some doxygen comments.
by Andrew Fyfe 12 Jul '07

12 Jul '07

Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> --- lib/libalpm/deps.c | 2 +- lib/libalpm/trans.c | 5 +++++ src/pacman/deptest.c | 4 +++- 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/lib/libalpm/deps.c b/lib/libalpm/deps.c index 591e5a8..3560fdd 100644 --- a/lib/libalpm/deps.c +++ b/lib/libalpm/deps.c @@ -222,7 +222,7 @@ alpm_list_t *_alpm_sortbydeps(alpm_list_t *targets, pmtranstype_t mode) * @param db pointer to the local package database * @param op transaction type * @param packages an alpm_list_t* of packages to be checked - * @return an alpm_list_t* of missing_t pointers. + * @return an alpm_list_t* of pmpkg_t* of missing_t pointers. */ alpm_list_t *_alpm_checkdeps(pmdb_t *db, pmtranstype_t op, alpm_list_t *packages) diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index 2078a5d..75f2a1d 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -298,6 +298,11 @@ int _alpm_trans_sysupgrade(pmtrans_t *trans) return(_alpm_sync_sysupgrade(trans, handle->db_local, handle->dbs_sync)); } +/** Add a target to the transaction. + * @param trans the current transaction + * @param target the name of the target to add + * @return 0 on success, -1 on error (pm_errno is set accordingly) + */ int _alpm_trans_addtarget(pmtrans_t *trans, char *target) { ALPM_LOG_FUNC; diff --git a/src/pacman/deptest.c b/src/pacman/deptest.c index 0f149b2..3b6b7ac 100644 --- a/src/pacman/deptest.c +++ b/src/pacman/deptest.c @@ -35,7 +35,9 @@ extern config_t *config; -/* TODO: This should use _alpm_checkdeps() */ +/** + * @todo This should use _alpm_checkdeps() + */ int pacman_deptest(alpm_list_t *targets) { int retval = 0; -- 1.5.2.3

2 4

[pacman-dev] Summary of pacman development
by Dan McGee 12 Jul '07

12 Jul '07

This is a quick summary for all of us on the list to see if we are on the same page. I just wanted to get out what we've gotten accomplished in the last month or so, and where we need to go from here to maybe get a pacman 3.1 release pushed out the door. We have had some 272 commits since the version 3.0 tag, so we have gotten some work done, but I don't think any of us are very clear on where the 3.1 release should be. Things we've done: * A lot of makepkg cleanup courtesy of Andrew Fyfe. (http://projects.archlinux.org/git/?p=pacman.git&a=search&st=commit&s=makepkg) * Brought xdelta support into official makepkg- we still need to decide on a strategy for using this if we want to. * Switched to bsdtar usage in makepkg- we no longer depend on the multitude of unzip programs as bsdtar can handle most of them now. * Added several pactests to isolate small issues, most from Nagy Gabor. * Implemented topological support for dependency sorting, courtesy of Nagy Gabor. * Several other small fixes all over the place, and a good amount of code refactoring. * Gettext will now work for our shell scripts- we now have a whole lot more strings that need translating. :) * Bugs closed: http://bugs.archlinux.org/index.php?events%5B%5D=2&event_number=20&project=… Things to do: * Pacman and directory symlinks: http://bugs.archlinux.org/task/7484 http://archlinux.org/pipermail/pacman-dev/2007-June/008638.html This is quite the fun little bug. A lot of it deals with our transition both to libarchive, and then to the 2.X branch of libarchive. Andrew has a proposed patch for this problem * Remove CHOST from makepkg.conf? ( http://archlinux.org/pipermail/pacman-dev/2007-July/008662.html) * Fix some bugs related to dependencies: FS 7495, 7416 * FS 7406- NREP umask issue. We should ensure that libalpm and pacman are setting sane umasks for their operations, and we don't get things that have the wrong umask such as db entries. * FS 7193- pacman -D is missing. We should reimplement this as a suboperation of -U such as --asdep or something. It shouldn't be too hard- look at the commit we removed this operation in and revert the parts of it that are necessary, and just use different command line flags. * FS 7137/7133- looks like DB issues * FS 7271- makepkg caching netfiles- should be a super quick fix for someone like Andrew (hint, hint) :) Just look at Roman's last comment * Asciidoc- this is coming along great, Andrew and I have a separate branch for now we are maintaining it in. This should make writing man pages much more pain-free. A few formatting issues probably need working out, but this will also allow us to make much prettier online HTML-formatted documentation. * Current pactests known to fail: requiredby004, sync022, upgrade051. If anyone else on the ML has things to add to this list, please get it out here. Eventually we can lay out a timetable for pacman 3.1 release and start doing some serious testing and regression testing. -Dan

5 12

[pacman-dev] [PATCH] Fix previous makepkg patch, and some pacman output.
by Andrew Fyfe 12 Jul '07

12 Jul '07

makepkg: We still need the source files in $srcdir because PKGBUILDS need access to noextract() files and other file not handled by by extract_source(). (eg config files) query.c: Fix some output formating. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> --- scripts/makepkg.sh.in | 2 ++ src/pacman/query.c | 6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 137d58e..d340252 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -447,9 +447,11 @@ download_sources() { local file=$(strip_url "$netfile") if [ -f "$startdir/$file" ]; then msg2 "$(gettext "Found %s in build dir")" "$file" + cp -s --remove-destination "$startdir/$file" "$srcdir/" continue elif [ -f "$SRCDEST/$file" ]; then msg2 "$(gettext "Using cached copy of %s")" "$file" + cp -s --remove-destination "$SRCDEST/$file" "$srcdir/" continue fi diff --git a/src/pacman/query.c b/src/pacman/query.c index cfd53df..f6c6b5d 100644 --- a/src/pacman/query.c +++ b/src/pacman/query.c @@ -251,7 +251,7 @@ static int query_test(void) int ret = 0; alpm_list_t *testlist; - printf(_("Checking database for consistency...")); + printf(_("Checking database for consistency... ")); testlist = alpm_db_test(db_local); if(testlist == NULL) { printf(_("check complete.\n")); @@ -271,7 +271,7 @@ static int query_test(void) static int query_upgrades(void) { - printf(_("Checking for package upgrades...")); + printf(_("Checking for package upgrades... \n")); alpm_list_t *syncpkgs; if((syncpkgs = alpm_db_get_upgrades()) != NULL) { @@ -279,7 +279,7 @@ static int query_upgrades(void) return(0); } - printf(_("no upgrades found")); + printf(_("no upgrades found.\n")); return(1); } -- 1.5.2.3

2 1

[pacman-dev] cosmetics: move alpm_depcmp to deps.c
by ngaba＠petra.hos.u-szeged.hu 12 Jul '07

12 Jul '07

Hi! I told everything in the subject, I think alpm_depcmp belongs to deps.c Bye, ngaba ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program.

3 2

[pacman-dev] pmpkg_t->buildtype?
by Andrew Fyfe 12 Jul '07

12 Jul '07

Anybody know what pmpkg_t->buildtype is used for? Andrew

3 2

[pacman-dev] Can the update of a package induce the update of its depends?
by ngaba＠petra.hos.u-szeged.hu 11 Jul '07

11 Jul '07

Hi! When I was thinking on the symlink puzzle, I ran into the question above. Let me explain: Suppose, that an updated version of audacious moves from /usr to /opt; and user does a "pacman -S audacious". In this case the installed audacious-plugins will be broken, because it thinks that audacious is installed to /usr. The dir change of audacious wasn't planned so audacious-plugins simply depends on audacious (with no version number), so audacious-plugins won't be updated. To avoid this, the package maintainer can do some ugly tricks only (conflicts, install_script/warn etc.). So it would be useful, if the maintainer somehow could record to the repo that there was a big change between audacious 2.0-1 and 2.0-2; so if user updates from a <=2.0-1 version to a >=2.0-2 version this will induce the update of audacious' depends (we can assume, that the updated version of audacious-plugins works fine with the updated audacious.). Or we can define a special ("major") version number, whose change indicates a "recursive" update. Bye, ngaba ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program.

3 4

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-290-gcb164c3
by Dan McGee 11 Jul '07

11 Jul '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 cb164c3130f15b22dbf16991c28ba609cd2f79de (commit) via 37736a56f9c5e4819a8f132a51bc0784c4b288ec (commit) via 39b654965580bfb7dc0ab72b8c901fbf7729a568 (commit) via 1646fcc5bdb71dc4d27204de88da952d95d5902c (commit) via 64e45a36f868ca0386393d6b6cf05ffff09b5439 (commit) via d12c4f4b29458ff04671bf7b9686735ad988c775 (commit) via ea1fef69add040b9e5b1de6e9238eda4576ccd3f (commit) from 9a9928f1b88f4fa08826aa83ff16e6ef38bd4e6c (commit) - Log ----------------------------------------------------------------- commit cb164c3130f15b22dbf16991c28ba609cd2f79de Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Thu Jun 21 12:46:37 2007 +0200 conflict.c: start cleaning conflict check functions The three chk_ functions overlap for packages both in targets and in the database. This caused the same conflict to be found in both direction (A conflicts with B, and B conflicts with A). This patch avoids this duplication. which shouldn't be needed, but other changes might be required for that to work correctly. This also has the unexpected side effect to hide the failure of sync022 pactest, for FS #7415. That's maybe not a good thing though.. Note from Dan: sync022 does succeed, but a sync023 pactest added to check regressions also seems to pass. This may be a valid fix to this 'problem' sync022 was meant to find. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 37736a56f9c5e4819a8f132a51bc0784c4b288ec Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 11 00:45:15 2007 -0400 Add new pactest that is backwards of sync022 sync022 was added here: 39b654965580bfb7dc0ab72b8c901fbf7729a568 This pactest reverses the installed package to see if it is correctly picked, in order to test some further changes to this depcheck code that currently makes sync022 fail. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 39b654965580bfb7dc0ab72b8c901fbf7729a568 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jul 11 00:31:27 2007 -0400 Update translation files on libalpm side after gettext change Because of this commit: ea1fef69add040b9e5b1de6e9238eda4576ccd3f we lost a lot of gettext-ized messages on the libalpm side. Remove them in order to clean out these files a bit. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1646fcc5bdb71dc4d27204de88da952d95d5902c Author: Sergej Pupykin <ps(a)lx-ltd.ru> Date: Tue Jul 10 23:36:15 2007 -0400 Use mbstowcs instead of strlen where applicable Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 64e45a36f868ca0386393d6b6cf05ffff09b5439 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jul 10 14:58:45 2007 -0400 Create the pacman DB and cachedir directories upon install Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d12c4f4b29458ff04671bf7b9686735ad988c775 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jul 10 14:31:39 2007 -0400 Fix issue with <dbpath>/db.lck being truncated snprintf takes a length including the '\0' character, this wasn't accounted for originally. Fix it. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ea1fef69add040b9e5b1de6e9238eda4576ccd3f Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jul 10 14:24:58 2007 -0400 Remove gettext calls from all PM_LOG_DEBUG messages There is no real reason to burden our translators with these messages, as anyone helping to debug these will probably want them in English. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: Makefile.am | 7 + lib/libalpm/add.c | 83 +- lib/libalpm/alpm.c | 3 +- lib/libalpm/be_files.c | 22 +- lib/libalpm/cache.c | 18 +- lib/libalpm/conflict.c | 41 +- lib/libalpm/db.c | 41 +- lib/libalpm/deps.c | 47 +- lib/libalpm/error.h | 2 +- lib/libalpm/handle.c | 9 +- lib/libalpm/md5driver.c | 2 +- lib/libalpm/package.c | 10 +- lib/libalpm/po/de.po | 1259 ++++---------------------- lib/libalpm/po/en_GB.po | 1252 ++++---------------------- lib/libalpm/po/es.po | 1267 ++++----------------------- lib/libalpm/po/fr.po | 1415 ++++-------------------------- lib/libalpm/po/hu.po | 1257 ++++---------------------- lib/libalpm/po/it.po | 1314 ++++------------------------ lib/libalpm/po/libalpm.pot | 1201 ++++---------------------- lib/libalpm/po/pl_PL.po | 1263 ++++----------------------- lib/libalpm/po/pt_BR.po | 1235 ++++---------------------- lib/libalpm/po/ru_RU.po | 1238 ++++---------------------- lib/libalpm/remove.c | 39 +- lib/libalpm/server.c | 19 +- lib/libalpm/sha1.c | 2 +- lib/libalpm/sync.c | 82 +- lib/libalpm/trans.c | 16 +- lib/libalpm/util.c | 2 +- lib/libalpm/versioncmp.c | 4 +- pactest/tests/sync022.py | 2 +- pactest/tests/{sync022.py => sync023.py} | 12 +- src/pacman/pacman.c | 44 +- src/pacman/util.c | 6 +- 33 files changed, 2076 insertions(+), 11138 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 9a9928f1b88f4fa08826aa83ff16e6ef38bd4e6c \ cb164c3130f15b22dbf16991c28ba609cd2f79de

1 0

[pacman-dev] installing a package as user (permission problems)
by Xavier 11 Jul '07

11 Jul '07

Should pacman allow installing a package as user ? It looks quite hard to check for all permissions before doing anything. And if permissions are checked on the fly, and at one point it fails, it looks hard to revert what has been done. For example, suppose an user wants to install foo-1 package, and has write access to everything but $dbpath/local/foo-1/ pacman will be able to install the package, extracting all files on filesystem, but it won't be able to add the package in the database. So the package will be installed, but not recorded in the db. A permission problem can also appear earlier, eg during the extraction of files. Maybe all files from the package can be extracted except one, which needs to be installed in a directory without write access. That looks quite complicated to handle. Oh, I just remembered Dan's permissions branch : http://code.toofishes.net/gitweb.cgi?p=pacman.git;a=commitdiff;h=b06c89e3e5… "Remove geteuid calls and things related to uid checking in anticipation of reimplementing a more robust permissions checking system. This will allow libalpm to be used in situations where we do not always have root on a system (cygwin, dreamhost, etc.)." More robust permissions checking system, that would be cool. :) To Dan : what you had in mind would solve the problems above ?

2 1