- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH 0/4] Package list find performance improvements
by Dan McGee 14 Dec '10

14 Dec '10

This series of patches makes finding a package in our linked list implementation a whole lot faster, if that search is using the standard _alpm_pkg_find, which nearly all are (after the first patch). It does this by adding a hash function to util.c which is nothing too complicated and named after a publicly available algorithm. When packages are created, we fill in this hash value as soon as the pkgname is read. Finally, the _alpm_pkg_find function is rewritten to take advantage of this field, avoiding repeated strcmp() calls and only falling back to that if a hash is not available and to verify the hash value was not some sort of collision. Performance figures and numbers are available in the last patch. This actually speeds up operations by nearly 33%, so this is not a total waste of time to consider. :) Review and questions/comments/concerns welcome! -Dan Dan McGee (4): Use _alpm_pkg_find in deps search Add hash_sdbm function When setting package name, set hash value as well Used hashed package name in _alpm_pkg_find lib/libalpm/be_package.c | 1 + lib/libalpm/deps.c | 4 ++-- lib/libalpm/package.c | 16 ++++++++++++++-- lib/libalpm/package.h | 1 + lib/libalpm/util.c | 22 ++++++++++++++++++++++ lib/libalpm/util.h | 1 + 6 files changed, 41 insertions(+), 4 deletions(-) -- 1.7.3.3

5 8

[pacman-dev] [PATCH] doc/PKGBUILD: document that functions run in -e mode
by Dan McGee 14 Dec '10

14 Dec '10

Caught this noted on the forums, but it is definitely worth a note in the manpage as well. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- For maint. doc/PKGBUILD.5.txt | 18 +++++++++++------- 1 files changed, 11 insertions(+), 7 deletions(-) diff --git a/doc/PKGBUILD.5.txt b/doc/PKGBUILD.5.txt index fdb868a..8bdb3c6 100644 --- a/doc/PKGBUILD.5.txt +++ b/doc/PKGBUILD.5.txt @@ -235,9 +235,11 @@ name. The syntax is: `source=('filename::url')`. build() Function ---------------- In addition to the above directives, the optional build() bash function usually -comprises the remainder of the PKGBUILD. This is directly sourced and executed by -makepkg, so anything that bash or the system has available is available for use -here. Be sure any exotic commands used are covered by `makedepends`. +comprises the remainder of the PKGBUILD. This is directly sourced and executed +by makepkg, so anything that bash or the system has available is available for +use here. The function is run in `bash -e` mode, meaning any command that exits +with a non-zero status will cause the function to exit. Be sure any exotic +commands used are covered by `makedepends`. All of the above variables such as `pkgname` and `pkgver` are available for use in the build function. In addition, makepkg defines three variables for your @@ -261,10 +263,12 @@ the build function. package() Function ------------------ -An optional package() function can be specified in addition to the build() function. -This function is run immediately after the build() function. When specified in -combination with the fakeroot BUILDENV option in linkman:makepkg.conf[5], fakeroot -usage will be limited to running the packaging stage. An existing build() function +An optional package() function can be specified in addition to the build() +function. This function is run immediately after the build() function. The +function is run in `bash -e` mode, meaning any command that exits with a +non-zero status will cause the function to exit. When specified in combination +with the fakeroot BUILDENV option in linkman:makepkg.conf[5], fakeroot usage +will be limited to running the packaging stage. An existing build() function will be run as the user calling makepkg. Package Splitting -- 1.7.3.3

1 0

[pacman-dev] Misleading info when epoch is used
by Nagy Gabor 14 Dec '10

14 Dec '10

$ sudo pacman -Su warning: supertuxkart: local (0.6.2a-2) is newer than community (0.7rc1-1) What? First I thought that our vercmp code is buggy, but vercmp binary worked as expected. Then I figured out that my local package has epoch=1, but the epoch is unset on the community package (so this seems to be a packager bug). So the above message is simply misleading (probably this is not the only one). It would be better to switch to a default version printing: "0.6.2a-2 [epoch=1]", or "1#0.6.2a-2" etc. In fact I don't like neither force nor epoch. Epoch is just a version prefix, why don't we let the packager to workaround this (KISS)? We can introduce a new separator (now we have one: '.'), for example '#', and let the packager define his favourite pkgversion (maybe epoch in mind), like "1#0.6.2a-2". Epoch just complicates code and leads to "wtf" imho... NG

7 13

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.4.1-191-g5f36523
by dan＠archlinux.org 13 Dec '10

13 Dec '10

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 5f36523af9de20a8c74714da4125bb553b0fae2c (commit) via ed367fe96d36fabe4127deab99df98cdb834145a (commit) via 1ff8118212cf566607c61e2f340dc3064c67c2ac (commit) from 13a2847aa14cb7643d50ca82b00180b440e401fe (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 5f36523af9de20a8c74714da4125bb553b0fae2c Author: Bruno Widmann <bruno.widmann(a)gmail.com> Date: Wed Oct 20 08:59:34 2010 +0200 Abort db_populate if dbpath is not set Rather than segfault. Fixes FS#21345. Signed-off-by: Bruno Widmann <bruno.widmann(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ed367fe96d36fabe4127deab99df98cdb834145a Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 22:42:22 2010 -0600 Update scripts/ .gitignore Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1ff8118212cf566607c61e2f340dc3064c67c2ac Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 22:41:02 2010 -0600 Create sync/ DB directory if it does not exist Rather than error out, this is easy enough. Looks quite similar to the code in be_local for creating the local directory. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/be_local.c | 3 +++ lib/libalpm/be_sync.c | 18 ++++++++++++++++++ scripts/.gitignore | 1 + 3 files changed, 22 insertions(+), 0 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] Local database structure change
by Dan McGee 13 Dec '10

13 Dec '10

Everyone, The commits Allan wrote and I just pushed change the local database format. depends no longer exists; everything is now in desc. Provided in the code is a 'pacman-db-upgrade' script to help you guys out if you upgrade, so be sure to run it if running pacman-git. -Dan On Sun, Dec 12, 2010 at 10:03 PM, Dan McGee <dan(a)archlinux.org> wrote: > 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". > > commit d98bacd4ec4289750e740b2dff8c4adaaa7a028a > Author: Allan McRae <allan(a)archlinux.org> > Date: Wed Dec 8 22:16:37 2010 +1000 > > Add script to update pacman database format > > The pacman-db-upgrade script was added to detect old pacman database > formats and upgrade them. > > Currently performs the merging of depends files into desc files in > the local database. > > Signed-off-by: Allan McRae <allan(a)archlinux.org> > Signed-off-by: Dan McGee <dan(a)archlinux.org> > > commit 21833d90e26635fdd2c6af247790a9bf374b4d80 > Author: Allan McRae <allan(a)archlinux.org> > Date: Sat Oct 30 15:35:43 2010 +1000 > > Merge desc and depends files in local db > > Whenever depends is needed from the local db, so is desc. The only > disadvantage to merging them is the additional time taken to read the > depends entries when they are not needed. As depends is in general > relatively small, the additional time taken to read it in will be > negligable. Also, merging these files will speed up local database > access due to less file seeks. > > Signed-off-by: Allan McRae <allan(a)archlinux.org> > Signed-off-by: Dan McGee <dan(a)archlinux.org>

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.4.1-188-g13a2847
by dan＠archlinux.org 13 Dec '10

13 Dec '10

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 13a2847aa14cb7643d50ca82b00180b440e401fe (commit) via d98bacd4ec4289750e740b2dff8c4adaaa7a028a (commit) via 8f3b485517a401ece348a1c5166d951ec4a8a28f (commit) via 21833d90e26635fdd2c6af247790a9bf374b4d80 (commit) via 209d0643e5fd38f8add70cfb3651012993d861f1 (commit) via b276a76dc9e02e1cdc227661ae4ffe97a24d9f4d (commit) from 3030542d10d4e87997a894a9eb63122cd2426a94 (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 13a2847aa14cb7643d50ca82b00180b440e401fe Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 21:40:24 2010 -0600 pacman-optimize: ensure database directory contains local/ And also default dbpath to the one we may find in pacman.conf. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d98bacd4ec4289750e740b2dff8c4adaaa7a028a Author: Allan McRae <allan(a)archlinux.org> Date: Wed Dec 8 22:16:37 2010 +1000 Add script to update pacman database format The pacman-db-upgrade script was added to detect old pacman database formats and upgrade them. Currently performs the merging of depends files into desc files in the local database. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8f3b485517a401ece348a1c5166d951ec4a8a28f Author: Allan McRae <allan(a)archlinux.org> Date: Sat Oct 30 16:01:30 2010 +1000 Update pactest suite for change in db structure Merging desc and depends files in sync and local db. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 21833d90e26635fdd2c6af247790a9bf374b4d80 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Oct 30 15:35:43 2010 +1000 Merge desc and depends files in local db Whenever depends is needed from the local db, so is desc. The only disadvantage to merging them is the additional time taken to read the depends entries when they are not needed. As depends is in general relatively small, the additional time taken to read it in will be negligable. Also, merging these files will speed up local database access due to less file seeks. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 209d0643e5fd38f8add70cfb3651012993d861f1 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Oct 31 22:43:55 2008 -0500 Attempt to stop installation when we encounter problems This should hopefully address some of the concerns raised in FS#11639 with regards to continuing after filling the disk up. Add some more checks and passing of error conditions between our functions. When a libarchive warning is encountered, check if it is due to lack of disk space and if so upgrade it to an error condition. A review of other libarchive warnings suggests that these are less critical and can remain as informative warning messages at this stage. Note the presence of errors after extraction of an entire package is complete. If so, we abort the transaction to be on the safe side and keep damage to a minimum. Signed-off-by: Dan McGee <dan(a)archlinux.org> [Allan: make ENOSPC warning into an error] Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b276a76dc9e02e1cdc227661ae4ffe97a24d9f4d Author: Dan McGee <dan(a)archlinux.org> Date: Sun Oct 12 21:08:39 2008 -0500 Turn libarchive warnings into libalpm warnings Rather than hiding these warnings, show them to the user as they happen. This will prevent things such as hiding full filesystem errors (ENOSPC) from the user as seen in FS#11639. Signed-off-by: Dan McGee <dan(a)archlinux.org> [Allan: adjust warning wording and add gettext calls] Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: lib/libalpm/add.c | 80 +++++++++++++--------- lib/libalpm/be_local.c | 144 +++++++++++++++------------------------ lib/libalpm/be_package.c | 2 +- lib/libalpm/db.h | 9 +-- lib/libalpm/util.c | 2 +- scripts/Makefile.am | 3 + scripts/pacman-db-upgrade.sh.in | 121 ++++++++++++++++++++++++++++++++ scripts/pacman-optimize.sh.in | 6 +- test/pacman/pmdb.py | 62 +++++------------ test/pacman/pmpkg.py | 4 - 10 files changed, 256 insertions(+), 177 deletions(-) create mode 100644 scripts/pacman-db-upgrade.sh.in hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.4.1-18-g5c8083b
by dan＠archlinux.org 13 Dec '10

13 Dec '10

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, maint has been updated via 5c8083baa4a831b16adcc3c1c35d7438f5bdcc3e (commit) via eedf4f4e630f98350090ef2824b0981ffb4b7a4a (commit) via bd08581d2e5c67cc084b7ccb8f67df456bbd9ada (commit) from a91250b7bb718c7e36c9784d51bdcb6e0051c259 (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 ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: lib/libalpm/be_files.c | 7 ++++--- lib/libalpm/trans.c | 2 +- po/ru.po | 2 +- 3 files changed, 6 insertions(+), 5 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.4.1-182-g3030542
by dan＠archlinux.org 13 Dec '10

13 Dec '10

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 3030542d10d4e87997a894a9eb63122cd2426a94 (commit) via 9f96c5433a4867cce9dc0c3ff23cb3ba8029be20 (commit) via ef977865a143eb682baf0d4f598c0c6924aeaa1a (commit) via b7015af0fc9325740950223e719f6dec153e264b (commit) via ec136784d4328b4c75f622c08273cf9dc6cac40f (commit) via 24684a616ed46c635a2236873c8168b9153b5bc1 (commit) via e22aa23c8f5adb0fd4b7ccbee480a4906ede0346 (commit) via 3f0d98c124db1547d6595762bd2d125b4350b861 (commit) via e527699dddca41e3af6c1872e851405a4f5212a0 (commit) via 695656d25279342f80c307035721ced6d37ee08b (commit) via f4e9deb6d74614ac427b37513359d91588dd1542 (commit) via adb10c3ab244cc00eb863d5b036fa7bdd76f8152 (commit) via f0051a7678d138ffc88a4c9c6d9add8966abe4f9 (commit) via fc74ef93b6945dfac299433a9418fdd31798f6b6 (commit) via 8e9a69e8f66b9b1a13e01ef0b67a78f2014754fa (commit) via a44c7b8956f1f445d554a742870c72ca7dacaae5 (commit) via 889c260cbf6cf497169ca9b75df121d04140441d (commit) via 579533d1a07808c3e0458d6042d6039e56c38de2 (commit) via 330951200c8a0bb90937e0578510a41220154726 (commit) via 970ffbb4c77bb2121f5aec8391bcc04e3a54986c (commit) via d4bab6f8fcfea7783c260d28c7c598055617b181 (commit) via 622e7fdd4fffcaf28679ba063505fa18755b7ba0 (commit) via 65a96e900d9b3f970fd184986303092e4735de0e (commit) via 6b928fc0bf6a2dc88c413d4bc88c7923d4741bc6 (commit) via 7c06e66c24953a6c8a15e8460be8ed3b1585c469 (commit) via abefa23341caa995cc0d155d30baa7faeb7a6c0e (commit) via 3a9dec104858a64489b004296d009c8222e7f040 (commit) via c78f5fb99ab7551868b3b2b8cd2b56807d457f59 (commit) via d0d8f605d5f3ff8fc306af90ea2a516174354bfc (commit) via d5a7dc67d93f5e530a6b3611f567eff5afe76efb (commit) via 8791ae0fdac16419d25e18acd245a65879807eb8 (commit) via a03daad0734428ded0398c9d71582f90063f3493 (commit) via c2cce1f46a754c04c3623088a9e43922a6c10e2d (commit) via 0e39cf9275c6a6f965e364792527c2704327bd02 (commit) via a35610beba1b12c37ec26ec878d3701dd669db42 (commit) via 5c8083baa4a831b16adcc3c1c35d7438f5bdcc3e (commit) via eedf4f4e630f98350090ef2824b0981ffb4b7a4a (commit) via bd08581d2e5c67cc084b7ccb8f67df456bbd9ada (commit) via 4fb3cfc48f626f84329c78351d6a8710255a35e3 (commit) via a91250b7bb718c7e36c9784d51bdcb6e0051c259 (commit) via a08638edc8cdacf7840414d82bc687727d7b2363 (commit) via 592211b6dc4b9997a60543309378f1595cf38056 (commit) via d901646f7a2e67d27bcae43f90490299857646fc (commit) from 0f3957ab48f34df8e151ec026dfac0d9d2df5aec (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 3030542d10d4e87997a894a9eb63122cd2426a94 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 20:38:20 2010 -0600 Make reading from any file possible in sync DB Whether it be "desc", "depends", or "deltas", it really doesn't matter- treat them all the same and have the ability to read any data from any file in that list. This continues the work in a44c7b8956. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 9f96c5433a4867cce9dc0c3ff23cb3ba8029be20 Author: Allan McRae <allan(a)archlinux.org> Date: Wed Dec 8 13:51:31 2010 +1000 Explicitly test time difference is greater than zero We are comparing a floating point number so should use an inequality rather than implicitly testing != 0. Prevents warning given by -Wfloat-equal. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ef977865a143eb682baf0d4f598c0c6924aeaa1a Author: Allan McRae <allan(a)archlinux.org> Date: Tue Dec 7 17:48:16 2010 +1000 makepkg: perform sanity checks on variables in package functions Check the over-ridden entries for provides, backup, optdepends and options for illegal entries. Partially fixes FS#16004. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit b7015af0fc9325740950223e719f6dec153e264b Author: Dan McGee <dan(a)archlinux.org> Date: Tue Nov 16 20:12:27 2010 -0600 diskspace style cleanups and small fixes * Use our normal return() function syntax * Rework a few things to reduce number of casts * Fix void function argument declaration * Add missing gettext _() call * Remove need for seperate malloc() of statvfs/statfs structure * Unify argument order of static functions- mountpoints now always passed first * Count all files that start with '.' in a package against the DB * Rename db to db_local in check_diskspace to clarify some code * Fix some line wrapping to respect 80 characters Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit ec136784d4328b4c75f622c08273cf9dc6cac40f Author: Dan McGee <dan(a)archlinux.org> Date: Tue Nov 16 20:12:26 2010 -0600 Refactor statfs/statvfs type check Turn it into a configure-type typedef, which allows us to reduce the amount of duplicated code and clean up some #ifdef magic in the code itself. Adjust some of the other defined checks to look at the headers available rather than trying to pull in the right ones based on configure checks. Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 24684a616ed46c635a2236873c8168b9153b5bc1 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 17:07:43 2010 +1000 Display progress bar for disk space checking Checking disk space needed for a transaction can take a while so add an informative progress bar. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e22aa23c8f5adb0fd4b7ccbee480a4906ede0346 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 16:54:30 2010 +1000 Add configuration option to control disk space checking Disk space checking is likely to be an unnecessary bottleneck to people with reasonable partition sizes so add a configuration option to allow it to be disabled/enabled as wanted. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3f0d98c124db1547d6595762bd2d125b4350b861 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 16:43:45 2010 +1000 Implement disk space checking Pull together the work of the previous commits to implement a check for enough free space before performing an install transaction. Abort if there is not enough free space with an appropriate pm_errno.. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e527699dddca41e3af6c1872e851405a4f5212a0 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 16:30:33 2010 +1000 Add functions to calculate approximate disk usage by packages Two helper function are added to calculate the disk usage from packages that are either currently installed on the system or from a package archive. Some minor approximations have been made: 1. Size for directories is not considered when removing a package from the filesystem to avoid multiple counting across packages. Also, these are reported to take zero size while installing. 2. Symlinks are reported to contribute zero size towards removal as libarchive reports them to have zero size for install. 3. Package data files (.PKGINFO, .INSTALL, .CHANGELOG) are counted towards usage on dbpath on install, but their size is not counted on package removal. 4. No handling of extra size needed for .pacsave/.pacnew files. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 695656d25279342f80c307035721ced6d37ee08b Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 16:24:52 2010 +1000 Add function to match file to mount point For a given file, determine which mount point it is on or will be installed to. Take into account that we might be using an alternative installation root. Add additional helper function added to sort mount point list for easier matching. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f4e9deb6d74614ac427b37513359d91588dd1542 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 16:15:21 2010 +1000 Add function for listing system mount points Add a mount_point_list() function that attempts to portably obtain a list of system mount points and a struct to hold needed mount point information. Abort the transaction if we are unable to determine the mount points. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit adb10c3ab244cc00eb863d5b036fa7bdd76f8152 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Nov 16 15:57:39 2010 +1000 Prototype disk space checking functionality Very basic prototyping for adding functionality to check free disk space before performing package installs. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit f0051a7678d138ffc88a4c9c6d9add8966abe4f9 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Nov 16 20:13:37 2010 -0600 Remove AC_TYPE_SIGNAL usage This macro is deemed unnecessary by even the autoconf guys, so we really don't need to use it. Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit fc74ef93b6945dfac299433a9418fdd31798f6b6 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Nov 16 20:13:36 2010 -0600 dirent usage cleanup We were including the header in a lot of places it is no longer used. Additionally, use the correct autoconf macro for determining whether d_type is available as a member: HAVE_STRUCT_DIRENT_D_TYPE. Signed-off-by: Dan McGee <dan(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8e9a69e8f66b9b1a13e01ef0b67a78f2014754fa Author: Allan McRae <allan(a)archlinux.org> Date: Sun Nov 7 21:39:55 2010 +1000 Move MAX_DELTA_RATIO definition This is a delta specific definition so it makes more sense to put it in the delta specific header file. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a44c7b8956f1f445d554a742870c72ca7dacaae5 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Oct 30 14:50:04 2010 +1000 Combine reading from depends and desc in sync db This will allow us to eventually combine the depends and desc entries within the sync database. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 889c260cbf6cf497169ca9b75df121d04140441d Author: Andres P <aepd87(a)gmail.com> Date: Fri Jun 25 18:46:38 2010 -0430 makepkg: remove dead code from handle_deps The error message that has been removed never gets to print because, given the same condition, handle_deps throws the same error and then immediately exits makepkg. Signed-off-by: Andres P <aepd87(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 579533d1a07808c3e0458d6042d6039e56c38de2 Author: Andres P <aepd87(a)gmail.com> Date: Fri Jun 25 18:46:37 2010 -0430 makepkg: do not ignore errors from pacman when checking deps As check_deps is run in a subshell, exit had the same meaning as return. Since the intention is to halt makepkg when pacman throws an error other than 127, the enclosing function has to handle error control instead. Fixes FS#19840 Signed-off-by: Andres P <aepd87(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 330951200c8a0bb90937e0578510a41220154726 Author: Carlos Diaz <839273(a)gmail.com> Date: Thu Oct 14 12:02:46 2010 -0430 makepkg: use portable escape sequence for terminal escapes The escape string isn't necesarily \033; it's determined by what the particular termcap/info entry for that terminal contains. Bash uses ncurses functions to expand \e to the _correct_ terminal escape. Signed-off-by: Carlos Diaz <839273(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 970ffbb4c77bb2121f5aec8391bcc04e3a54986c Author: Allan McRae <allan(a)archlinux.org> Date: Sat Oct 30 10:42:41 2010 +1000 Add epoch to PKGBUILD prototypes Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d4bab6f8fcfea7783c260d28c7c598055617b181 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 20:23:25 2010 -0600 src/util: update .gitignore Add pactree and sort entries. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 622e7fdd4fffcaf28679ba063505fa18755b7ba0 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Wed Oct 13 18:07:31 2010 +0200 contrib: remove bash pactree This has been rewitten in C which is much much faster. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> commit 65a96e900d9b3f970fd184986303092e4735de0e Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Wed Oct 13 17:11:14 2010 +0200 pactree: use variables for color and tree output This allows to very easily support non-color and linear mode, by just setting the variables to an empty string, very much like it was done in the bash script. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> commit 6b928fc0bf6a2dc88c413d4bc88c7923d4741bc6 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Wed Oct 13 15:50:13 2010 +0200 pactree: separate dependency and print logic The deps walking code simply calls print_start, print, print_end, and all the printing logic is handled there. The unresolvable printing is disabled for now because it does not handle linear mode, and the linear and color output will be re-written. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> commit 7c06e66c24953a6c8a15e8460be8ed3b1585c469 Author: Dave Reisner <d(a)falconindy.com> Date: Tue Oct 12 14:46:12 2010 -0400 pactree: rewrite in C Use the bash script in contrib as the basis for a C rewrite using libalpm. The speedup can go from dozens of seconds to less than one second. Colorized output is preserved. The --graph option generates output that graphviz's `dot' utility will understand to draw us a graph. Output is written to stdout and it is left up to the user to pipe the data and determine the output characteristics. Signed-off-by: Dave Reisner <d(a)falconindy.com> Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> commit abefa23341caa995cc0d155d30baa7faeb7a6c0e Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Tue Oct 12 00:42:04 2010 +0200 alpm/remove.c : respect --dbonly during remove-upgrade When a -Sk or -Uk operation induced a removal of an existing local package, --dbonly was not in effect and the files were all removed. Fixing this behavior was already marked as TODO in database012 pactest ------------ TODO: I honestly think the above should NOT delete the original les, it hould upgrade the DB entry without touching anything on the file stem. E.g. this test should be the same as: pacman -R --dbonly dummy && pacman -U --dbonly dummy.pkg.tar.gz ------------ Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> [Dan: small coding style touchup] Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3a9dec104858a64489b004296d009c8222e7f040 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Thu Oct 14 10:02:38 2010 +0200 pacman: sort --help output Example with pacman -Uh : $ pacman -Uh options: -b, --dbpath <path> set an alternate database location -d, --nodeps skip dependency checks -f, --force force install, overwrite conflicting files -k, --dbonly only modify database entries, not package files -r, --root <path> set an alternate installation root -v, --verbose be verbose --arch <arch> set an alternate architecture --asdeps install packages as non-explicitly installed --asexplicit install packages as explicitly installed --cachedir <dir> set an alternate package cache location --config <path> set an alternate configuration file --debug display debug messages --ignore <pkg> ignore a package upgrade (can be used more than once) --ignoregroup <grp> ignore a group upgrade (can be used more than once) --logfile <path> set an alternate log file --noconfirm do not ask for any confirmation --noprogressbar do not show a progress bar when downloading files --noscriptlet do not execute the install scriptlet if one exists --print only print the targets instead of performing the operation --print-format <string> specify how the targets should be printed Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> [Dan: small coding style touchups] Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c78f5fb99ab7551868b3b2b8cd2b56807d457f59 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Sun Oct 10 18:57:41 2010 +0200 CLI args: update --help and manpage The three parts (help, manpage and code) are now organized in the same way and much easier to compare : - specific options - install/upgrade options for -S and -U - transaction options for -S -R and -U - global options After this re-organization, it was easy to update and sync the three components together. Duplication is also avoided. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d0d8f605d5f3ff8fc306af90ea2a516174354bfc Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Tue Oct 12 21:47:40 2010 +0200 alpm: don't expose alpm_depcmp Either we expose all low levels function dealing with pmdepend_t (splitdep and depfree come to mind), or we don't. Since none of the tools use depcmp, I chose to remove it. In the future, we might want to expose higher level functions such as alpm_find_satisfier, or just lower level functions like splitdep and depfree together with depcmp. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d5a7dc67d93f5e530a6b3611f567eff5afe76efb Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Tue Oct 12 21:41:55 2010 +0200 alpm: kill alpm_deptest This has been replaced by the more flexible alpm_find_satisfier function, and alpm_deptest was completely unsused now. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 8791ae0fdac16419d25e18acd245a65879807eb8 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Tue Oct 12 20:27:49 2010 +0200 pacman/deptest: reimplement with alpm_find_satisfier It's very easy to re-implement the -T feature with the more generic alpm_find_satisfier rather then the more specific and less useful alpm_deptest. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a03daad0734428ded0398c9d71582f90063f3493 Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Tue Oct 12 20:13:32 2010 +0200 alpm: add new alpm_find_satisfier function whatprovides and splitdep were removed, so depcmp alone is quite useless now without splitdep, and deptest is not flexible enough. Introduce a new alpm_find_satisfier which is hopefully more flexible, this should make implementation of deptest very easy, and also help alpm tools such as pactree. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c2cce1f46a754c04c3623088a9e43922a6c10e2d Author: Xavier Chantry <chantry.xavier(a)gmail.com> Date: Sun Oct 31 21:39:31 2010 +0100 Fix a few problems reported by clang-analyzer One missing NULL-check and 3 dead assignments. Signed-off-by: Xavier Chantry <chantry.xavier(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 0e39cf9275c6a6f965e364792527c2704327bd02 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 20:01:14 2010 -0600 Ensure stdin args are correctly terminated And don't require pm_targets to be empty to read from stdin either. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a35610beba1b12c37ec26ec878d3701dd669db42 Merge: 4fb3cfc 5c8083b Author: Dan McGee <dan(a)archlinux.org> Date: Sun Dec 12 19:53:20 2010 -0600 Merge branch 'maint' Conflicts: lib/libalpm/be_local.c lib/libalpm/trans.c commit 4fb3cfc48f626f84329c78351d6a8710255a35e3 Author: Dave Reisner <d(a)falconindy.com> Date: Thu Nov 4 16:05:28 2010 -0400 Support reading package args from stdin Only occurs if no arguments were provided directly. Arguments can be separated by any amount of valid whitespace. This allows for piping into pacman from other programs or from itself, e.g.: pacman -Qdtq | pacman -Rs This is better than using xargs, as xargs will not reconnect stdin to the terminal. The above operation performed using xargs would require the --noconfirm flag to be passed to pacman. Signed-off-by: Dave Reisner <d(a)falconindy.com> ----------------------------------------------------------------------- Summary of changes: NEWS | 6 + PKGBUILD-split.proto | 1 + PKGBUILD.proto | 1 + acinclude.m4 | 28 +++- configure.ac | 15 +- contrib/Makefile.am | 2 - contrib/PKGBUILD.vim | 9 +- contrib/README | 3 - contrib/pactree.in | 322 ------------------------------- doc/pacman.8.txt | 108 +++++------ doc/pacman.conf.5.txt | 4 + etc/pacman.conf.in | 1 + lib/libalpm/Makefile.am | 1 + lib/libalpm/alpm.h | 14 +- lib/libalpm/be_local.c | 2 +- lib/libalpm/be_package.c | 3 +- lib/libalpm/be_sync.c | 40 ++--- lib/libalpm/conflict.c | 2 +- lib/libalpm/db.c | 1 - lib/libalpm/delta.h | 3 + lib/libalpm/deps.c | 41 ++--- lib/libalpm/deps.h | 1 + lib/libalpm/diskspace.c | 319 +++++++++++++++++++++++++++++++ lib/libalpm/{log.h => diskspace.h} | 36 +++-- lib/libalpm/dload.c | 2 +- lib/libalpm/error.c | 2 + lib/libalpm/handle.c | 14 ++ lib/libalpm/handle.h | 1 + lib/libalpm/remove.c | 5 + lib/libalpm/sync.c | 19 ++- lib/libalpm/trans.c | 2 +- lib/libalpm/util.h | 3 - po/es.po | 40 ++-- po/ru.po | 2 +- po/sv.po | 168 +++++++--------- scripts/makepkg.sh.in | 46 +++-- src/pacman/callback.c | 12 +- src/pacman/deptest.c | 14 +- src/pacman/pacman.c | 312 +++++++++++++++++++----------- src/pacman/util.c | 1 - src/pacman/util.h | 1 + src/util/.gitignore | 15 +- src/util/Makefile.am | 5 +- src/util/pactree.c | 372 ++++++++++++++++++++++++++++++++++++ test/pacman/tests/database012.py | 8 +- 45 files changed, 1261 insertions(+), 746 deletions(-) delete mode 100755 contrib/pactree.in create mode 100644 lib/libalpm/diskspace.c copy lib/libalpm/{log.h => diskspace.h} (55%) create mode 100644 src/util/pactree.c hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] Fix possible null pointer deref in check_arch
by Dan McGee 11 Dec '10

11 Dec '10

If we have a corrupted database, a package can come through without an arch, causing the code to blow up when making strcmp() calls. It might even be possible with perfectly valid database entries lacking an 'arch =' line. This behavior was seen as at least one of the problems in FS#21668. Ensure pkgarch is not null before doing anything further. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- lib/libalpm/trans.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index 49fc0f6..02612ec 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -107,7 +107,7 @@ static alpm_list_t *check_arch(alpm_list_t *pkgs) for(i = pkgs; i; i = i->next) { pmpkg_t *pkg = i->data; const char *pkgarch = alpm_pkg_get_arch(pkg); - if(strcmp(pkgarch,arch) && strcmp(pkgarch,"any")) { + if(pkgarch && strcmp(pkgarch, arch) && strcmp(pkgarch, "any")) { char *string; const char *pkgname = alpm_pkg_get_name(pkg); const char *pkgver = alpm_pkg_get_version(pkg); -- 1.7.3.3

1 0

[pacman-dev] Alternative dependencies.
by Xyne 10 Dec '10

10 Dec '10

Hi, Would it be possible to implement support for alternative dependencies, e.g. a package that could use either wget or curl to handle downloads? I'm not asking for this to be implemented. I'm just asking whether there are any technical limitations to such an approach. I know that "provides" can be used to do this, e.g. openjdk6 and jre both provide java-runtime to satisfy that dependency in other packages. The approach is limited though. Using the example above, we couldn't simply add "provides=('download-client')" to both curl and wget because they are not interchangeable. The basic usage case is when an application is capable of detecting any one of a number of different external apps that it can use to satisfy a given dependency. There is currently no way to depend on "at least one" of those. I can imagine a very simple syntax for this too, e.g. "depends=('curl || wget')" so the question is whether the dependency-handling mechanism could be adapted to check for alternatives. If none are found, it could probably use the upcoming "provides" selection dialogue to allow the user to choose the desired dependency. So, would that be possible? Also, would that be acceptable? Again, I'm not asking anyone to do this. I'm just inquiring about the possibility. Regards, Xyne

5 7