[pacman-dev] [GIT] pacman branch, master now at v3.0.0-378-gb250195
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 b2501950c7fca0b771fc79054d9592ea79753749 (commit) via b15a5194d1a8485a2769560e49e6ff03e1862533 (commit) via 53fc745aedc0a6d24abbc8bce6ca0b30c2179e5f (commit) via 678983d2623d7ed700a70634089eef1c9f0b9b21 (commit) via 9cceb3d9c4d4b0975781a4d48eabfdd29026453e (commit) via 39871375051856f9248d651005ab62e2a309d6ea (commit) via 461bc9e6ce8afee23b6402b4af65aa29b7268c35 (commit) via 824b7fd27b490e599025b38e629e53921df5883d (commit) from b3a1619457fa6424570c90c0eaacbbf39fd9662c (commit) - Log ----------------------------------------------------------------- commit b2501950c7fca0b771fc79054d9592ea79753749 Author: Dan McGee <dan@archlinux.org> Date: Thu Sep 6 19:52:55 2007 -0500 Fix sync db location in pactest so new location is acceptable Signed-off-by: Dan McGee <dan@archlinux.org> commit b15a5194d1a8485a2769560e49e6ff03e1862533 Author: Chantry Xavier <shiningxc@gmail.com> Date: Thu Aug 30 00:39:57 2007 +0200 libalpm/trans.c : remove the lock even on interrupted transactions. Signed-off-by: Chantry Xavier <shiningxc@gmail.com> commit 53fc745aedc0a6d24abbc8bce6ca0b30c2179e5f Author: Chantry Xavier <shiningxc@gmail.com> Date: Thu Aug 23 15:20:34 2007 +0200 Add a few pactests collected over time. remove047 : Remove a package required by other packages conflict004 : a package conflicts with itself http://www.archlinux.org/pipermail/pacman-dev/2007-August/009077.html sync400 : Install package with dep that conflicts with older version of package sync401 : Ensure we choose provider already in target list http://www.archlinux.org/pipermail/pacman-dev/2007-July/009041.html sync402/sync403 (failing) : Choice between two providers http://www.archlinux.org/pipermail/pacman-dev/2007-July/008787.html Signed-off-by: Chantry Xavier <shiningxc@gmail.com> commit 678983d2623d7ed700a70634089eef1c9f0b9b21 Author: Andrew Fyfe <andrew@neptune-one.net> Date: Wed Aug 29 10:49:24 2007 +0100 scripts/*.sh.in: Clean up and fix a few bugs repo-add, repo-remove: 'bsdtar -c * | ...' doesn't work (you need '-f -'). Code clean up eliminated this bug. Removed the multiple checksum support, pacman now only supports MD5, so there's no need for the database to contain multiple checksums. Quote all variables containing file/dir names to prevent paths containing spaces from causing problems. Add msg, warning and error functions. General code clean up. pacman-optimize: Use a sub-directory in /tmp for working files to make it easier to clean up at the end. Add quotes round $@ in die and die_r, otherwise printf can't display the message correctly. makepkg: Disable colour output if stderr is not a tty. Signed-off-by: Andrew Fyfe <andrew@neptune-one.net> commit 9cceb3d9c4d4b0975781a4d48eabfdd29026453e Author: Dan McGee <dan@archlinux.org> Date: Thu Sep 6 19:03:38 2007 -0500 Add a few missing includes, caught while compiling on cygwin Signed-off-by: Dan McGee <dan@archlinux.org> commit 39871375051856f9248d651005ab62e2a309d6ea Author: Chantry Xavier <shiningxc@gmail.com> Date: Wed Sep 5 13:15:22 2007 +0200 contrib : update zsh and bash completion to reflect the new sync dbs location. I wonder how many tools / scripts deal directly with the sync databases under /var/lib/pacman/ , I doubt these are the only ones. Signed-off-by: Chantry Xavier <shiningxc@gmail.com> commit 461bc9e6ce8afee23b6402b4af65aa29b7268c35 Author: Dan McGee <dan@archlinux.org> Date: Tue Sep 4 07:12:57 2007 -0500 libalpm: remove outdated files from POTFILES.in Fixes 'make dist' target as sha1.c and md5driver.c no longer exist. Signed-off-by: Dan McGee <dan@archlinux.org> commit 824b7fd27b490e599025b38e629e53921df5883d Author: Travis Willard <travis@archlinux.org> Date: Mon Sep 3 21:28:17 2007 -0400 Fix compile-time error with de.po, runtime error with db.c One string in de.po differed pretty strongly with its translated version. It may still be totally wrong as far as translations go, but it compiles now. Get translater to check. Also, ensure the proper dbpath gets set in the db when it's created. Signed-off-by: Travis Willard <travis@archlinux.org> Signed-off-by: Dan McGee <dan@archlinux.org> ----------------------------------------------------------------------- Diffstat: contrib/bash_completion | 4 +- contrib/vimproject | 1 - contrib/zsh_completion | 4 +- lib/libalpm/db.c | 6 +- lib/libalpm/po/POTFILES.in | 2 - lib/libalpm/po/de.po | 2 +- lib/libalpm/trans.c | 5 +- lib/libalpm/util.h | 1 + pactest/pmdb.py | 5 +- pactest/tests/{conflict001.py => conflict004.py} | 8 +- pactest/tests/remove047.py | 25 ++ pactest/tests/sync400.py | 25 ++ pactest/tests/sync401.py | 21 ++ pactest/tests/sync402.py | 21 ++ pactest/tests/sync403.py | 21 ++ scripts/makepkg.sh.in | 10 +- scripts/pacman-optimize.sh.in | 49 ++-- scripts/repo-add.sh.in | 283 ++++++++++------------ scripts/repo-remove.sh.in | 129 ++++++---- src/pacman/util.c | 1 + 20 files changed, 371 insertions(+), 252 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ b3a1619457fa6424570c90c0eaacbbf39fd9662c \ b2501950c7fca0b771fc79054d9592ea79753749
sync400 : Install package with dep that conflicts with older version of package
This should prove the PS of http://www.archlinux.org/pipermail/pacman-dev/2007-August/009078.html Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/
sync400 : Install package with dep that conflicts with older version
of
package This should prove the PS of http://www.archlinux.org/pipermail/pacman-dev/2007-August/009078.html Bye, ngaba This may not prove that, because the old version of pkg2 is installed; but as I see, in these case the reason field is not copied.
I'm suggest to work out a %REASON% policy, because I don't like the current method: -A/U copies the old reason field, if the old package is installed... OK, but the package may become orphan, so we can say, the reason is the _original_ reason; if the package is not installed, we get "explicitly installed" reason; fine. -however, -S seems buggy: it owerwrites the old reason iff the new reason is depend; so [1] if you have a pkg with depend reason, -S pkg will NOT overwrite the reason to explicit (here -S is an ~upgrade). [2] And if you have an explicitly installed pkg2, and "-S pkg1" also upgrades pkg2 (since pkg1 needs a newer version), the reason of pkg2 will be overwritten to depend (I don't like this.). Replace makes things even worse... So we should define and document the rules, then implement it;-) My suggestion: -A/U OK, user can use --asdeps, if he want to change the reason (--asexplicit is also needed IMHO) -S: the reason field should be kept iff the oldpkg is exists -change reason of localpkg switch can also be useful: this would let user change the depend reason to explicit (see [1]) Bye, ngaba A bit off: So I prefer "original reason" way, which can be changed by the user later. Only explicit packages are important to the user, all others are needed to satisfy deps of explicit packages. So I prefer using the important word instead of reason (HoldPkgs are the most important packages.) So I'd prefer important = 0 <=> depend && !HoldPkg important = 1 <=> explicit && !HoldPkg important = 2 <=> HoldPkg; so I think HoldPkg should be stored in local db; and in sync db ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/
On Fri, Sep 07, 2007 at 06:27:34PM +0200, Nagy Gabor wrote:
I'm suggest to work out a %REASON% policy, because I don't like the current method: -A/U copies the old reason field, if the old package is installed... OK, but the package may become orphan, so we can say, the reason is the _original_ reason; if the package is not installed, we get "explicitly installed" reason; fine. -however, -S seems buggy: it owerwrites the old reason iff the new reason is depend; so [1] if you have a pkg with depend reason, -S pkg will NOT overwrite the reason to explicit (here -S is an ~upgrade). [2] And if you have an explicitly installed pkg2, and "-S pkg1" also upgrades pkg2 (since pkg1 needs a newer version), the reason of pkg2 will be overwritten to depend (I don't like this.). Replace makes things even worse... So we should define and document the rules, then implement it;-)
My suggestion: -A/U OK, user can use --asdeps, if he want to change the reason (--asexplicit is also needed IMHO) -S: the reason field should be kept iff the oldpkg is exists -change reason of localpkg switch can also be useful: this would let user change the depend reason to explicit (see [1])
I just noticed [1] yesterday, as someone on IRC wanted to mark a few packages as explictly installed (the dependencies of the xorg meta package, which was removed). So indeed, the only way to mark these as explicitly installed using pacman was to remove them, then install them again.. It might be a good idea to have an --asexplicit flag for this case.
A bit off: So I prefer "original reason" way, which can be changed by the user later. Only explicit packages are important to the user, all others are needed to satisfy deps of explicit packages. So I prefer using the important word instead of reason (HoldPkgs are the most important packages.) So I'd prefer important = 0 <=> depend && !HoldPkg important = 1 <=> explicit && !HoldPkg important = 2 <=> HoldPkg; so I think HoldPkg should be stored in local db; and in sync db
Now I am afraid I lost you, I am not sure how this would be useful.
A bit off: So I prefer "original reason" way, which can be changed by the user later. Only explicit packages are important to the user, all others are needed to satisfy deps of explicit packages. So I prefer using the important word instead of reason (HoldPkgs are the most important packages.) So I'd prefer important = 0 <=> depend && !HoldPkg important = 1 <=> explicit && !HoldPkg important = 2 <=> HoldPkg; so I think HoldPkg should be stored in local db; and in sync db
Now I am afraid I lost you, I am not sure how this would be useful.
Well, this is not so important. 1. I tried to explain why I prefer the way described above (with no success :-P), I wanted to explain my "concept". So explicitly installed package means (for me) that user need this package, because he installed by hand (he mentioned in the target list); dependency package means, that pacman installed that "behind" the user, so it is needed to satisfy the dependencies of explicitly installed packages only, so they are not so important (they can be removed, as no longer needed by other packages). We need to decide that a package is explicit or not when it first appears in local db, and we keep this property during upgrades (I derived this rule from the current behaviour of -U; someone may prefer the following: when we mention the package in the target list in case of -S, -U then that means that user really need that package, so we can change the reason to explicit.) 2. I don't like the current HoldPkg implementation, this should be totally handled by libalpm IMHO. HoldPkg means (for me) that the package is really important; IMHO this should be recorded to localdb and to syncdb (e.g. current/glibc should be flagged as HoldPkg). So we may add HoldPkg property to reason (important) field to unify, but if you want to keep the current reason field, you may add a new HOLDPKG entry... (If user could change the reason field then he could also "set" HoldPkg "flag"; orphan remove ignores HoldPkg and explicitly installed packages, so it removes the unneeded (reason/important == 0) ones only; if a package is holdpkg then its reason==explicit/depend is not important...) Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/
-A/U OK, user can use --asdeps, ... By the way, please keep bash-completion up-to-date. Bye, ngaba
---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/
participants (3)
-
Dan McGee
-
Nagy Gabor
-
Xavier