[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.1.4-182-gb48f703
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 b48f703aa6da7d9863501df1488f851313998ebe (commit) via 009f89c4d21fc08f16d648d8341137d36b0cde96 (commit) via b32aa81b5eb00b5e07295f6bdfeb294a340842b8 (commit) via 62ee1bfff08b358a11e08194adc5774a8a1cdf51 (commit) via 1cca4ef764519c00326dc81669b2bbd4bf1b978f (commit) via b262ddb2edb8a4622a9cd3c50f245a6fa7f35f75 (commit) via f30dab4b5a382bc4886af6b64f721fd2b4884283 (commit) via c1a5616c26a5c175c65dea61ca74a622c6930df8 (commit) via da1c11cc30d0286f07d46bf2d910482c9a72f31c (commit) via ad54b286802fd739532c9fd6f3bc6c7c431bddf4 (commit) via ae40d1c05b2a164e922a19ddece1334bf5e55330 (commit) via 9577c07d867eb0887bd524dc1cdaaa188fbe82f3 (commit) via c33cabd6751d1d5903898c911701ce20cf77598d (commit) via 20ae871940b8b7f773b0fdc31f8e053cc3cf4884 (commit) via b9369a747d0c2fa3ee8371f99f5ca7acbc89f7f9 (commit) via 149839c5391e9a93465f86dbb8d095a0150d755d (commit) via 1d71079c5b57af2feb53fbfa98f93122a32ca154 (commit) via 398d4aff2dc25585e51310ba2f7f8d64a9f52081 (commit) from bf2964dc5841f49d1d5ab60181b0caa4670c1874 (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 b48f703aa6da7d9863501df1488f851313998ebe Author: Dan McGee <dan@archlinux.org> Date: Thu May 29 17:21:46 2008 -0500 Add missing NULL set in group printing If we don't set the pkgname var to NULL, we run into all sorts of beautiful segfault behavior when a group spans multiple repositories and we try to print out the location of the former list. Easy fix. This regression was introduced in bf8670036907b3ede18d37b7a3f0f7e14542a5ac. Signed-off-by: Dan McGee <dan@archlinux.org> commit 009f89c4d21fc08f16d648d8341137d36b0cde96 Merge: b32aa81b5eb00b5e07295f6bdfeb294a340842b8 398d4aff2dc25585e51310ba2f7f8d64a9f52081 Author: Dan McGee <dan@archlinux.org> Date: Thu May 29 06:39:29 2008 -0500 Merge branch 'maint' commit b32aa81b5eb00b5e07295f6bdfeb294a340842b8 Author: Geoffroy Carrier <geoffroy.carrier@koon.fr> Date: Mon May 26 23:35:28 2008 +0200 Add trailing / to DESTDIR for stupid Makefiles. Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 62ee1bfff08b358a11e08194adc5774a8a1cdf51 Author: Allan McRae <mcrae_allan@hotmail.com> Date: Mon May 26 16:27:06 2008 +1000 Tidy up of the handle struct in libalpm Removed unused handle->uid from pmhandle_t. The need to check permissions should be determined by the frontend (and is in pacman). Fixed comment on noextract in pmhandle_t. Signed-off-by: Allan McRae <mcrae_allan@hotmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 1cca4ef764519c00326dc81669b2bbd4bf1b978f Author: Xavier Chantry <shiningxc@gmail.com> Date: Mon May 26 23:28:34 2008 +0200 Remove PM_TRANS_EVT_EXTRACT_START. This event was unused, was missing the equivalent EXTRACT_DONE event, and was useless because we already have ADD / UPGRADE START and DONE events. Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit b262ddb2edb8a4622a9cd3c50f245a6fa7f35f75 Author: Xavier Chantry <shiningxc@gmail.com> Date: Thu May 22 23:59:10 2008 +0200 be_files : stores REPLACES and FORCE in desc. repo-add and db_read both assume that REPLACES and FORCE fields are in the desc file, so do that for db_write as well (instead of depends file). Note that db_write is currently only used on the local database. And the only purpose of replaces and force in local database is for information purpose (available on -Qi operations). So this is not a big problem. Ref: http://www.archlinux.org/pipermail/pacman-dev/2008-May/011859.html Acked-by: Dan McGee <dan@archlinux.org> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit f30dab4b5a382bc4886af6b64f721fd2b4884283 Author: Allan McRae <mcrae_allan@hotmail.com> Date: Mon May 19 21:39:59 2008 +1000 Pactests for sync operations with NoUpgrade and NoExtract This adds the sync equivalents of upgrade010.py and upgrade070.py. Both additional test pass. Signed-off-by: Allan McRae <mcrae_allan@hotmail.com> Acked-by: Dan McGee <dan@archlinux.org> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit c1a5616c26a5c175c65dea61ca74a622c6930df8 Author: Allan McRae <mcrae_allan@hotmail.com> Date: Tue May 13 20:26:11 2008 +1000 makepkg - add check for valid options in PKGBUILD This patch removes the code block in makepkg that checked for depreciated options in a PKGBUILD and provided a workaround. Unknown and depreciated options are upgraded to error conditions. Also, removed TODO regarding including install script if exists and $install is unset. That should never happen. Signed-off-by: Allan McRae <mcrae_allan@hotmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit da1c11cc30d0286f07d46bf2d910482c9a72f31c Author: Nagy Gabor <ngaba@bibl.u-szeged.hu> Date: Sat May 3 19:59:11 2008 +0200 Add some verbosity to _alpm_db_read. Name and version are computed from "/var/lib/pacman/..." pathname. And the %NAME% and %VERSION% fields from the desc file were not even read. So now, when we read the desc file, we make sure the %NAME% and %VERSION% fields are consistent. Signed-off-by: Nagy Gabor <ngaba@bibl.u-szeged.hu> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit ad54b286802fd739532c9fd6f3bc6c7c431bddf4 Author: Nagy Gabor <ngaba at bibl.u-szeged.hu> Date: Sat Apr 19 22:31:22 2008 +0200 Fix for remove052.py by moving recursedeps before checkdeps. In the case of -Rs operation, first pulling the dependencies with recursedeps before calling checkdeps takes care of the dependency chain of remove052 pactest. In the case of -Rcs, we can keep the old behavior because we have no problem there (any dependency returned by checkdeps will be added to the remove list because of -Rc) and we have to run recursedeps on the final remove list anyway to catch all orphans. Ref.: http://www.archlinux.org/pipermail/pacman-dev/2008-April/011569.html Signed-off-by: Nagy Gabor <ngaba at bibl.u-szeged.hu> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit ae40d1c05b2a164e922a19ddece1334bf5e55330 Author: Allan McRae <mcrae_allan at hotmail.com> Date: Thu Apr 3 16:35:14 2008 +1000 Pactest for removing multiple items in a dependency chain. This adds a test for when removing multilpe packages recursively from a chain of dependent packages. This situation can occur when removing installed dependencies with makepkg if a "makedepend" recursively depends on a "depend" or if redundant dependancies are included. Signed-off-by: Allan McRae <mcrae_allan at hotmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 9577c07d867eb0887bd524dc1cdaaa188fbe82f3 Author: Allan McRae <mcrae_allan at hotmail.com> Date: Thu Apr 3 01:07:25 2008 +1000 No error in makepkg when removing deps fails Catches error from when pacman is unable to remove dependencies after successfully building package and prints warning. Fixes FS#10039. Signed-off-by: Allan McRae <mcrae_allan at hotmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit c33cabd6751d1d5903898c911701ce20cf77598d Author: Sebastian Nowicki <sebnow@gmail.com> Date: Wed May 28 03:22:08 2008 +0800 Use chown 0:0 instead of root.root On BSD systems using a dot as a separator is not allowed. On Mac OSX it is deprecated. A colon should be used instead. BSD systems also use the "wheel" group instead of "root" to indicate the "super user" group. Both groups use the id of 0. Signed-off-by: Sebastian Nowicki <sebnow@gmail.com> Acked-by: Aaron Griffin <aaronmgriffin@gmail.com> Acked-by: Dan McGee <dan@archlinux.org> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 20ae871940b8b7f773b0fdc31f8e053cc3cf4884 Author: Sebastian Nowicki <sebnow@gmail.com> Date: Wed May 28 03:23:56 2008 +0800 Use "legacy mode" on Mac OSX (for file command) For some reason `file` on Mac OSX has different arguments than BSD and Linux; -i no longer prints out the mime strings. With the environment variable COMMAND_MODE set to "legacy", `file` behaves more like it does on Linux and BSD, i.e., `file -i` prints the mime type. Signed-off-by: Sebastian Nowicki <sebnow@gmail.com> Acked-by: Dan McGee <dan@archlinux.org> Acked-by: Aaron Griffin <aaronmgriffin@gmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit b9369a747d0c2fa3ee8371f99f5ca7acbc89f7f9 Author: Sebastian Nowicki <sebnow@gmail.com> Date: Tue May 27 14:00:51 2008 +0800 Allow an unprivileged user to create packages without fakeroot When fakeroot was not in BUILDENV and the user was not root, makepkg still tried to use fakeroot for building packages. BUILDENV is now checked to see if fakeroot is enabled. If it is not enabled the package can still be built, but root will not have ownership of files. This is useful when users want to make packages for personal use and don't care about ownership. Closes FS#10450. Signed-off-by: Sebastian Nowicki <sebnow@gmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 149839c5391e9a93465f86dbb8d095a0150d755d Author: Xavier Chantry <shiningxc@gmail.com> Date: Mon May 26 23:46:01 2008 +0200 du -b is not available on BSD, use du -k instead. This fixes FS#10459. There is apparently no portable ways to get the apparent size of a file, like du -b does. So the best compromise seems to get the block size in kB, and then convert that to byte so that we keep compatibility. Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit 1d71079c5b57af2feb53fbfa98f93122a32ca154 Author: Sebastian Nowicki <sebnow@gmail.com> Date: Wed May 21 01:51:38 2008 +0800 Replace usage of "cp -s" with the more portable "ln -s" The "-s" argument does not exist on BSD, and neither does "--remove-destination". This patch replaces the calls to "cp -s --remove-destination" with the equivalent "rm -f" and "ln -s" calls, in order to increase portability. Signed-off-by: Sebastian Nowicki <sebnow@gmail.com> Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org> ----------------------------------------------------------------------- Summary of changes: PKGBUILD.proto | 2 +- lib/libalpm/add.c | 1 - lib/libalpm/alpm.h | 2 - lib/libalpm/be_files.c | 41 ++++-- lib/libalpm/handle.c | 3 - lib/libalpm/handle.h | 3 +- lib/libalpm/remove.c | 8 +- pactest/tests/{remove051.py => remove052.py} | 9 +- pactest/tests/{upgrade070.py => sync500.py} | 12 +- pactest/tests/{upgrade010.py => sync501.py} | 10 +- po/zh_CN.po | 178 +++++++++++++------------- scripts/makepkg.sh.in | 74 +++++++----- scripts/repo-add.sh.in | 4 +- src/pacman/callback.c | 1 - src/pacman/sync.c | 6 +- 15 files changed, 192 insertions(+), 162 deletions(-) copy pactest/tests/{remove051.py => remove052.py} (63%) copy pactest/tests/{upgrade070.py => sync500.py} (50%) copy pactest/tests/{upgrade010.py => sync501.py} (76%) hooks/post-receive -- The official pacman repository
On Thu, May 29, 2008 at 9:11 PM, Dan McGee <dan@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".
The branch, master has been updated via b48f703aa6da7d9863501df1488f851313998ebe (commit) via 009f89c4d21fc08f16d648d8341137d36b0cde96 (commit) via b32aa81b5eb00b5e07295f6bdfeb294a340842b8 (commit) via 62ee1bfff08b358a11e08194adc5774a8a1cdf51 (commit) via 1cca4ef764519c00326dc81669b2bbd4bf1b978f (commit) via b262ddb2edb8a4622a9cd3c50f245a6fa7f35f75 (commit) via f30dab4b5a382bc4886af6b64f721fd2b4884283 (commit) via c1a5616c26a5c175c65dea61ca74a622c6930df8 (commit) via da1c11cc30d0286f07d46bf2d910482c9a72f31c (commit) via ad54b286802fd739532c9fd6f3bc6c7c431bddf4 (commit) via ae40d1c05b2a164e922a19ddece1334bf5e55330 (commit) via 9577c07d867eb0887bd524dc1cdaaa188fbe82f3 (commit) via c33cabd6751d1d5903898c911701ce20cf77598d (commit) via 20ae871940b8b7f773b0fdc31f8e053cc3cf4884 (commit) via b9369a747d0c2fa3ee8371f99f5ca7acbc89f7f9 (commit) via 149839c5391e9a93465f86dbb8d095a0150d755d (commit) via 1d71079c5b57af2feb53fbfa98f93122a32ca154 (commit) via 398d4aff2dc25585e51310ba2f7f8d64a9f52081 (commit) from bf2964dc5841f49d1d5ab60181b0caa4670c1874 (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 ----------------------------------------------------------------- <snip> -----------------------------------------------------------------------
Summary of changes: PKGBUILD.proto | 2 +- lib/libalpm/add.c | 1 - lib/libalpm/alpm.h | 2 - lib/libalpm/be_files.c | 41 ++++-- lib/libalpm/handle.c | 3 - lib/libalpm/handle.h | 3 +- lib/libalpm/remove.c | 8 +- pactest/tests/{remove051.py => remove052.py} | 9 +- pactest/tests/{upgrade070.py => sync500.py} | 12 +- pactest/tests/{upgrade010.py => sync501.py} | 10 +- po/zh_CN.po | 178 +++++++++++++------------- scripts/makepkg.sh.in | 74 +++++++----- scripts/repo-add.sh.in | 4 +- src/pacman/callback.c | 1 - src/pacman/sync.c | 6 +- 15 files changed, 192 insertions(+), 162 deletions(-) copy pactest/tests/{remove051.py => remove052.py} (63%) copy pactest/tests/{upgrade070.py => sync500.py} (50%) copy pactest/tests/{upgrade010.py => sync501.py} (76%)
I just want to say thanks to Xavier for managing the patch queue while I was on vacation, and he assembled most of the patches above in his tree. I'm sure he had a lot of fun doing it. :) There has been a good amount of activity here lately, so if there are any outstanding patches that got lost in the shuffle, be sure to poke the thread so it comes to Xavier and I's attention. One currently known issue I am working on: my versioncmp change a while back definitely broke something, and I've written a short vercmp test script that I plan on incorporating into our codebase. You can see that on my working branch. -Dan
Dan McGee wrote:
One currently known issue I am working on: my versioncmp change a while back definitely broke something, and I've written a short vercmp test script that I plan on incorporating into our codebase. You can see that on my working branch.
You probably should add the mirror cases for these -1 return situations. +# with pkgrel, simple +runtest 1.5.0-1 1.5.0-1 0 +runtest 1.5.0-1 1.5.0-2 -1 +runtest 1.5.0-1 1.5.1-1 -1 +runtest 1.5.0-2 1.5.1-1 -1 + +# with pkgrel, mixed lengths +runtest 1.5-1 1.5.1-1 -1 +runtest 1.5-2 1.5.1-1 -1 +runtest 1.5-2 1.5.1-2 -1 I've been bitten by bugs not picked up because of missing test cases too many times in the past not to include them.
On Thu, May 29, 2008 at 10:42 PM, Allan McRae <mcrae_allan@hotmail.com> wrote:
Dan McGee wrote:
One currently known issue I am working on: my versioncmp change a while back definitely broke something, and I've written a short vercmp test script that I plan on incorporating into our codebase. You can see that on my working branch.
You probably should add the mirror cases for these -1 return situations.
+# with pkgrel, simple +runtest 1.5.0-1 1.5.0-1 0 +runtest 1.5.0-1 1.5.0-2 -1 +runtest 1.5.0-1 1.5.1-1 -1 +runtest 1.5.0-2 1.5.1-1 -1 + +# with pkgrel, mixed lengths +runtest 1.5-1 1.5.1-1 -1 +runtest 1.5-2 1.5.1-1 -1 +runtest 1.5-2 1.5.1-2 -1
I've been bitten by bugs not picked up because of missing test cases too many times in the past not to include them.
I know what you mean. I've already added another two test cases tonight after I got all the existing ones to pass and there were still problems. :) I made this script super easy to add test cases to though, so no real excuse to not include them. -Dan
On Fri, May 30, 2008 at 5:42 AM, Allan McRae <mcrae_allan@hotmail.com> wrote:
You probably should add the mirror cases for these -1 return situations.
That makes perfect sense, I was thinking the same. But it is much better to automate the mirror cases, as Dan did with his last patch :)
participants (4)
-
Allan McRae
-
Dan McGee
-
Dan McGee
-
Xavier