- Pacman-dev - lists.archlinux.org

[pacman-dev] IgnorePkg handling
by Xavier 16 Nov '07

16 Nov '07

I just found another thing I overlooked in a patch I submitted a while ago : "Let IgnorePkg superceed the FORCE package flag" , commit 5de03497c3f. I didn't see a change Dan made one month before : "* --ignore was being ignored (haha) on sysupgrade when a package was listed as being a force upgrade. This adds a prompt to the user in this case asking what to do." , commit 4a0aac3862 . So the change I made in alpm_pkg_compare_versions made this fix from Dan in _alpm_sync_sysupgrade obsolete. This code can't be run anymore. So I don't know, maybe this commit could be reverted now?

3 5

[pacman-dev] Segmentation fault
by Nagy Gabor 15 Nov '07

15 Nov '07

Hi! Finally I could reproduce FS#8350. Some background: -this is probably not important, but my "union" sync repo is sitting on my pendrive [in the suggestive file:///p/1 path ;-], so my syncdb is always out of sync [pendrive's size-limitation] -tk is in my sync repo [/p/1], cdparanoia is not [neither of the packages is installed, both can be found in my syncdb] S_good.txt: # pacman -S cdparanoia tk --debug S_fail.txt # pacman -S cdparanoia tk --debug Hm. What's the difference?: In the first case tk is in my /var/cache, so pacman tries to download only cdparanoia. In the second case my cache is empty (after -Scc), so pacman tries to download both (however, pacman could download tk to my cache before segfault) Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

3 7

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.0.0-563-g8236be9
by Dan McGee 15 Nov '07

15 Nov '07

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 8236be9fd8f97ea8cb4999cf775768bdc276e53e (commit) via 96f8faa6664714943201d86393099dbf7464abc2 (commit) via 4a835f5f53f23d3564ceb4f53b84f4b62b0074fe (commit) via b6b3b0135edd7bf0fae43bfe522e41cfa5eb0d9b (commit) via 434ea5bf619cd27d99d4b443fe058bf46cc5d7b2 (commit) via cc15d29db22bbc0815c4fb1f50a0e7ba53500a39 (commit) via 2898ccb609da38cf4e7b62d83b88f56396515120 (commit) via 7b4573d851464af53d34820769c0914f08c5ffeb (commit) via dd0275b759752a4f1f561dc490823ca289abd717 (commit) via a55a07f5ddb3ae16d4e60de75aebc2d7106db206 (commit) via 84433c880055faeaa7cf48a4f0a4fe9a7cf5ca1d (commit) via ed37d78664d2d6d036715ee0e939bfeea4a6ede6 (commit) via 6b9859995378a3419e6191df036a8d707cbb93a8 (commit) via 8ec27835f40e3df1ce409bc3d913587c474a30c3 (commit) via b206af78e0e6d2ff3324f3b2dc333d1b4e54f5b9 (commit) via 3312de65e642a7b6f2d853ce870910bddddf559d (commit) via 5c58b3d500d0971747af9a0c978ff6cfac668882 (commit) via 5cd6ffda722c79cf4689e559f214bcc27561fa5c (commit) via 6f5ee2432ccdd0a3bef742938cdd7552bc6a5c32 (commit) from 7d51882dd0afdb87fe986a7d7c672cc0be93795b (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 8236be9fd8f97ea8cb4999cf775768bdc276e53e Author: Dan McGee <dan(a)archlinux.org> Date: Sun Nov 11 11:30:16 2007 -0600 Add a horrible little hack to get symlink001.py to pass again This really doesn't give us any regressions in behavior, so it is safe to do although quite ugly. Tell the conflict checking code to ignore symlinks to dirs so that they are not seen as conflicts. Hopefully this entire commit will get factored out soon enough. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 96f8faa6664714943201d86393099dbf7464abc2 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Sun Nov 11 10:52:51 2007 -0600 Add two requiredby pactests One currently should succeed (006), and 005 fails. requiredby005.py is originally from Nagy Gabor <ngaba(a)petra.hos.u-szeged.hu>. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 4a835f5f53f23d3564ceb4f53b84f4b62b0074fe Author: Dan McGee <dan(a)archlinux.org> Date: Sun Nov 11 10:47:28 2007 -0600 Ensure list tail pointer is updated when we remove tail node Commit 2ee90ddae23dd86c68223c0d6c49f0b92d62429d did a special check to see if we were removing the head node, but not the tail node. Add a special case for the tail node to ensure all relevant pointers get updated. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b6b3b0135edd7bf0fae43bfe522e41cfa5eb0d9b Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Sat Nov 10 18:11:40 2007 +0100 Incorrect usage of alpm_db_whatprovides in sync.c The old code thought that alpm_db_whatprovides returns with a list of strings (package names). Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 434ea5bf619cd27d99d4b443fe058bf46cc5d7b2 Author: Aaron Griffin <aaronmgriffin(a)gmail.com> Date: Fri Nov 9 02:45:22 2007 -0600 Typo fix (sepArately) Found by Giovanni Scafora <linuxmania(a)gmail.com> Signed-off-by: Aaron Griffin <aaronmgriffin(a)gmail.com> commit cc15d29db22bbc0815c4fb1f50a0e7ba53500a39 Author: Aaron Griffin <aaronmgriffin(a)gmail.com> Date: Fri Nov 9 02:16:08 2007 -0600 Missing quote in output Found by Giovanni Scafora <linuxmania(a)gmail.com> Signed-off-by: Aaron Griffin <aaronmgriffin(a)gmail.com> commit 2898ccb609da38cf4e7b62d83b88f56396515120 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Nov 11 09:37:59 2007 -0600 libalpm: fix lstat wrapper to actually use newpath Commit b55abdce7aebb142ce79da3aa3645afe7693a3c4 introduced an lstat wrapper function that never dereferences paths with a trailing slash, but still called lstat on path instead of newpath. Oops! Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 7b4573d851464af53d34820769c0914f08c5ffeb Author: Dan McGee <dan(a)archlinux.org> Date: Sun Nov 11 09:36:03 2007 -0600 Remove unused and broken alpm_list_remove_node function Signed-off-by: Dan McGee <dan(a)archlinux.org> commit dd0275b759752a4f1f561dc490823ca289abd717 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Nov 11 09:28:35 2007 -0600 Add a missing newline in sync confirmation output Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a55a07f5ddb3ae16d4e60de75aebc2d7106db206 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Nov 9 08:40:09 2007 -0600 Add a symlink-based pactest This passes with both the upcoming 3.1 devel tree and the 3.0.6 pacman code. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 84433c880055faeaa7cf48a4f0a4fe9a7cf5ca1d Author: Dan McGee <dan(a)archlinux.org> Date: Fri Nov 9 00:23:25 2007 -0600 Update bash completion Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ed37d78664d2d6d036715ee0e939bfeea4a6ede6 Author: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Fri Nov 9 00:01:45 2007 -0600 Update Hungarian translation Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 6b9859995378a3419e6191df036a8d707cbb93a8 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Nov 8 23:59:02 2007 -0600 pacman: remove leftover help string for -Rh Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8ec27835f40e3df1ce409bc3d913587c474a30c3 Author: Nathan Jones <nathanj(a)insightbb.com> Date: Fri Nov 9 19:54:19 2007 -0500 Implement TotalDownload option. Setting this option will change the download progress to show the amount downloaded, download rate, ETA, and download percent of the entire download list rather than per each individual file. The progress bar is still based on the completion of the current file regardless if the TotalDownload option is set. This closes FS#7205. Signed-off-by: Nathan Jones <nathanj(a)insightbb.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b206af78e0e6d2ff3324f3b2dc333d1b4e54f5b9 Author: Nathan Jones <nathanj(a)insightbb.com> Date: Fri Nov 9 19:54:18 2007 -0500 Add TotalDownload option. This will be used in the next commit. Signed-off-by: Nathan Jones <nathanj(a)insightbb.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3312de65e642a7b6f2d853ce870910bddddf559d Author: Nathan Jones <nathanj(a)insightbb.com> Date: Fri Nov 9 20:13:29 2007 -0500 Implement IgnoreGroup. This option acts as if IgnorePkg was set on each package in the group. This closes FS#1592. Signed-off-by: Nathan Jones <nathanj(a)insightbb.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5c58b3d500d0971747af9a0c978ff6cfac668882 Author: Nathan Jones <nathanj(a)insightbb.com> Date: Fri Nov 9 20:13:28 2007 -0500 Add IgnoreGroup and --ignoregroup option. This will be used in the next commit. Signed-off-by: Nathan Jones <nathanj(a)insightbb.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5cd6ffda722c79cf4689e559f214bcc27561fa5c Author: Giovanni Scafora <linuxmania(a)gmail.com> Date: Fri Nov 9 19:43:48 2007 +0100 makeworld: gettext support Signed-off-by: Giovanni Scafora <linuxmania(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 6f5ee2432ccdd0a3bef742938cdd7552bc6a5c32 Author: Roman Kyrylych <roman(a)archlinux.org> Date: Sun Nov 11 16:25:44 2007 +0200 makepkg: remove .pacsave files when uninstalling dependencies Signed-off-by: Roman Kyrylych <roman(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: contrib/bash_completion | 22 +- doc/pacman.conf.5.txt | 9 + lib/libalpm/alpm.h | 7 +- lib/libalpm/alpm_list.c | 41 +- lib/libalpm/alpm_list.h | 1 - lib/libalpm/conflict.c | 19 +- lib/libalpm/db.c | 5 +- lib/libalpm/deps.c | 2 +- lib/libalpm/handle.c | 21 + lib/libalpm/handle.h | 1 + lib/libalpm/package.c | 32 +- lib/libalpm/package.h | 1 + lib/libalpm/server.c | 47 +- lib/libalpm/server.h | 6 +- lib/libalpm/sync.c | 27 +- lib/libalpm/util.c | 2 +- pactest/tests/{remove041.py => requiredby005.py} | 14 +- .../tests/{requiredby004.py => requiredby006.py} | 7 +- pactest/tests/symlink001.py | 20 + po/hu.po | 1023 +++++++++++--------- scripts/makepkg.sh.in | 4 +- scripts/makeworld.sh.in | 53 +- src/pacman/callback.c | 68 +- src/pacman/callback.h | 3 +- src/pacman/conf.h | 3 + src/pacman/pacman.c | 19 +- src/pacman/sync.c | 4 +- src/pacman/util.c | 1 + 28 files changed, 871 insertions(+), 591 deletions(-) copy pactest/tests/{remove041.py => requiredby005.py} (51%) copy pactest/tests/{requiredby004.py => requiredby006.py} (69%) create mode 100644 pactest/tests/symlink001.py hooks/post-receive -- The official pacman repository

7 24

[pacman-dev] [PATCH] Add VerbostListInfo Option
by Justin Lampley 15 Nov '07

15 Nov '07

Attached is a patch to add the VerboseListInfo option. This option takes the place of ShowSize as well as ShowOldVersion, which was submitted as a patch but never committed. During searches, the size of the packages are still shown, just like when ShowSize was enabled. During upgrades, the size as well as the old package version are shown in a formatted output with each package having its own line as well as each element (version, size) having its own column. The way I decided to handle printing in this format was to create a new function, formatted_list_display. This function accepts the title of the list, a list of rows with columns separated by commas, and a list of formats for the rows with the formats for each column also separated by commas. Example, the rows passed to the function would look like: lib32-gcc-libs,New,4.2.2-2,1.05 MB audacious-player,1.3.2-4,1.4.0-1,1.07 MB and the format would be something like: %-19s,%-10s,%-10s,%7s The function splits the row up by column and prints each columns with the widths specified. Since the columns are separated by commas, you can't have any commas in the rows or the format, so we may want to discuss another way to handle the formatted output. Other than the possible comma problem, this function should be usable anywhere you want to display a table of information. Example Output: --- $ src/pacman/pacman.static -Qu Checking for package upgrades... warning: xpdf: forcing upgrade to version 3.02_pl2-1 Remove: lib32-gcc Total Removed Size: 0.00 MB Targets: lib32-gcc-libs New 4.2.2-2 1.05 MB audacious-player 1.3.2-4 1.4.0-1 1.07 MB audacious-plugins 1.3.5-5.1 1.4.0-2 1.68 MB eclipse-pydev 1.3.9-1 1.3.10-1 3.39 MB fakeroot 1.7.1-3 1.8.4-1 0.06 MB glibc 2.7-4 2.7-6 10.78 MB glproto 1.4.8-1 1.4.9-1 0.02 MB hal-info 0.20071011-1 0.20071030-1 0.03 MB imagemagick 6.3.6.6-1 6.3.6.10-1 2.30 MB lib32-glibc 2.6.1-2 2.7-5 3.64 MB libarchive 2.2.6-2 2.4.0-1 0.32 MB libgphoto2 2.3.1-3 2.4.0-2 1.36 MB libmcs 0.4.1-2 0.6.0-1 0.02 MB libsamplerate 0.1.2-3 0.1.2-4 0.20 MB lincity-ng 1.1.1-1 1.1.2-1 29.16 MB lm_sensors 2.10.5-1 2.10.5-2 0.32 MB tcl 8.4.16-1 8.4.16-2 1.63 MB tk 8.4.16-1 8.4.16-2 1.40 MB ttf-dejavu 2.20-1 2.21-1 3.79 MB xf86-video-ati 6.7.195-1 6.7.195-3 3.67 MB xorg-server 1.4-4 1.4-5 7.25 MB xpdf 3.02_pl1-1 3.02_pl2-1 0.49 MB Total Download Size: 73.64 MB Total Installed Size: 91.08 MB --- Justin Lampley

3 3

[pacman-dev] [PATCH 0/2] Remove REQUIREDBY usage
by Dan McGee 15 Nov '07

15 Nov '07

This is mainly an RFC on the whole idea. Nagy, you made me think enough about it that I just went ahead and whipped something up. I'd first like to say that I did this in the quickest way possible, so there is no guarantee I did everything right. I also did nothing to optimize for efficiency. In some cases, we could be calling the alpm_pkg_compute_requiredby() function way more than we should, because I basically replaced any call to alpm_pkg_get_requiredby() with a corresponding alpm_pkg_compute_requiredby() and free(). Note that we should definitely find a way to pactest or otherwise test the computed requiredby entries. Some quick observations: $ time pacman -Qt > /dev/null (pacman 3, latest GIT release) real 0m0.084s user 0m0.020s sys 0m0.043s $ time ./src/pacman/pacman -Qt > /dev/null (with compute_requiredby switch) real 0m1.893s user 0m1.800s sys 0m0.037s Obviously a slowdown, but is it all that bad in the big scheme of things when corrupted requiredby entries no longer hurt us? Also: $ pacman -Qt > orphans-old $ ./src/pacman/pacman -Qt > orphans-new $ diff orphans-old orphans-new 5a6 > agg 2.5-2 121a123 > libnet 1.1.2.1-1 144a147 > mysql 5.0.45-1 184a188 > python-eyed3 0.6.14-1 237a242 > ttf-bitstream-vera 1.10-5 306a312 > xmlsec 1.2.10-3 At first, I was worried. Then I see that the new code is much better! openoffice-base has at least two stale entries in the requiredby fields of agg and xmlsec, while libnet is stale from being an old depend of ettercap. So yeah, this new code helps us in a lot of places (everywhere but speed). -Dan

4 25

[pacman-dev] output issues
by Dan McGee 15 Nov '07

15 Nov '07

checking package integrity... done. (2/2) checking for file conflicts [---------------------] 100% warning: directory permissions differ on var/cache/pacman/ o o o o o ] 0% filesystem: 555 package: 755 (1/2) upgrading pacman-git [---------------------] 100% (2/2) upgrading xscreensaver [---------------------] 100% Who wants to step up to the plate and fix this stuff? I'm sure Xavier can point us to the past conversations we've had on the subject, and we really should fix it. I liked the suggestion of buffering all these messages until after the progress bar reaches 100%,. Of course, this could be a hard change to make. Output functions in general need a sprucing up and/or unification. I know there are still printf("error...") type things scattered around the front end, and most of these could be replaced with a call to one of the pm_vprintf functions. -Dan

3 2

[pacman-dev] [PATCH] 2 new failing pactests.
by Nagy Gabor 15 Nov '07

15 Nov '07

Hi! I've created patches for the two pactests that I've mentioned here nowadays. I assigned add043.py-fix to myself (see also: http://www.archlinux.org/pipermail/pacman-dev/2007-November/009951.html) For trans001.py see also: http://www.archlinux.org/pipermail/pacman-dev/2007-November/009963.html Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

2 3

[pacman-dev] [PATCH] Add ShowOldVersion Option
by Justin Lampley 14 Nov '07

14 Nov '07

First of all, I just wanted to thank all of you for your hard work on pacman, I think it is one of the best package management tools out there. I have been following the list for a bit with the intent of starting to help out with the development of pacman (Or at least to try to help out anyway) so I figured I might as well dive in with a feature that I have had in mind for a while. This patch adds the option of allowing the old package version that is being updated from to be displayed. It takes advantage of the data element in the sync package struct which is populated with the currently installed version of the package. This feature can be beneficial in many ways, but mostly it is a good indicator (at least for me) of whether I should check out the change log of the program. With minor version increases, it is most likely minor updates or bug fixes, but if it is a major version increase, many things might have been changed that are worth checking up on before going through with the install. The following is some output with the ShowOldVersion option enabled as well as the ShowSize option ---- $ sudo src/pacman/pacman.static -Qu Checking for package upgrades... warning: xpdf: forcing upgrade to version 3.02_pl2-1 Remove: lib32-gcc Total Removed Size: 0.00 MB Targets: lib32-gcc-libs-4.2.2-2 [New] [1.05 MB] audacious-player-1.4.0-1 [1.3.2-4] [1.07 MB] audacious-plugins-1.4.0-2 [1.3.5-5.1] [1.68 MB] eclipse-pydev-1.3.10-1 [1.3.9-1] [3.39 MB] fakeroot-1.8.4-1 [1.7.1-3] [0.06 MB] glibc-2.7-6 [2.7-4] [10.78 MB] glproto-1.4.9-1 [1.4.8-1] [0.02 MB] hal-info-0.20071030-1 [0.20071011-1] [0.03 MB] imagemagick-6.3.6.10-1 [6.3.6.6-1] [2.30 MB] lib32-glibc-2.7-5 [2.6.1-2] [3.64 MB] libarchive-2.4.0-1 [2.2.6-2] [0.32 MB] libgphoto2-2.4.0-2 [2.3.1-3] [1.36 MB] libmcs-0.6.0-1 [0.4.1-2] [0.02 MB] libsamplerate-0.1.2-4 [0.1.2-3] [0.20 MB] lincity-ng-1.1.2-1 [1.1.1-1] [29.16 MB] lm_sensors-2.10.5-2 [2.10.5-1] [0.32 MB] tcl-8.4.16-2 [8.4.16-1] [1.63 MB] tk-8.4.16-2 [8.4.16-1] [1.40 MB] ttf-dejavu-2.21-1 [2.20-1] [3.79 MB] xf86-video-ati-6.7.195-3 [6.7.195-1] [3.67 MB] xorg-server-1.4-5 [1.4-4] [7.25 MB] xpdf-3.02_pl2-1 [3.02_pl1-1] [0.49 MB] Total Download Size: 73.64 MB Total Installed Size: 91.08 MB ---- Let me know what you think. Thanks, Justin Lampley

8 12

[pacman-dev] [PATCH] alpm_list_add == alpm_list_add_last I: recursedeps clean-up
by Nagy Gabor 14 Nov '07

14 Nov '07

Hi! Patch attached. See the patch description for details. Bye, ngaba ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

3 5

[pacman-dev] alpm_list.c "summary"
by Nagy Gabor 14 Nov '07

14 Nov '07

Hi! The main issues are (hopefully) fixed now (at least in Dan's repo) But: I refer to point 1. in this mail again: http://www.archlinux.org/pipermail/pacman-dev/2007-November/009902.html (I'm pretty sure that this doesn't break anything now, just for safety) And some (_subjective_) "elegance" notes: 1. There are some always-true if() stuffs in list_remove: line 312, 320, 323 2. alpm_list_msort and alpm_list_mmerge works fine with no-tail lists (well, it is designed for such lists), so the ~time-costly set tail in line 240 can be stripped out from mmerge. But then we must set tail after alpm_list_msort is finished (since this is a recursive function, we may need an alpm_list_msort wrapper to the (old) _alpm_list_msort) <- this may not worth the effort, I simply don't "like" this. Bye ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

1 0