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 dd265921559b187cba59e83bfdbfda11bcec2f1e (commit) via 57c5afd69c4c70ad85676bab32150d6182a3def9 (commit) via 0284cf2128eb0179a3598ee21a951c5b3c11be40 (commit) via 33240e87b99e5aebd0e64a672ea307a698edb32f (commit) via 25fab402c79aeaa5b72959c7bfd2e541a1b34045 (commit) via 6942bba75db596a70d5ba408785bf461e691bc9c (commit) via 55bff19b7692e01a0bf276c742c77f03fb445160 (commit) via fe6e90c21f1561ee54deedbc707f5afc964d3cae (commit) via 9a82cb92a446bce05186460bc5b5b12c4df7e5a1 (commit) via d0c327df172942da18f1e5c9b6afb183e4d9ee18 (commit) via eda4d9ec00be1108ab4336a438299a283c5a0a90 (commit) via b04a56dbe90f43622158410234fabea96d0d7f07 (commit) via 0d4dd09993971924d379be4d0944d72f4c77b021 (commit) via 7ce90bb135c04b4421ca2824099e6e72659721b1 (commit) via a9cbd15260b396c38b7d5ffe5669994e65f9a86f (commit) via 3e1bdfa93c0c0c1a966ba24f8013754a8a87f0ec (commit) via 5f140a62debb7d55436af7f84b0ecf5696f46c86 (commit) via 842cbc9ea4036036cf00b3b2eeccc11ad531cd2e (commit) via d03b57f459fb9ab9288991a70c4e7297a7c1d150 (commit) from 281a4c0a4f2de217b5d23939fb78b3bbfccc34ca (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 dd265921559b187cba59e83bfdbfda11bcec2f1e Merge: 57c5afd 25fab40 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 21:20:12 2011 -0600 Merge branch 'diskspace-fixes' commit 57c5afd69c4c70ad85676bab32150d6182a3def9 Merge: 0284cf2 33240e8 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 21:20:10 2011 -0600 Merge branch 'fix-double-load' commit 0284cf2128eb0179a3598ee21a951c5b3c11be40 Merge: fe6e90c d0c327d Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 21:20:07 2011 -0600 Merge branch 'repo-add-improvements' commit 33240e87b99e5aebd0e64a672ea307a698edb32f Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 18:46:10 2011 -0600 Fix double filelist issue when upgrading a package Due to the way we funk around with package data loading, we had a condition where the filelist got doubled up because it was loaded twice. Packages are originally loaded with INFRQ_BASE. In an upgrade/sync, the package is checked for file conflicts next, leaving us in an "INFRQ_BASE | INFRQ_FILES" state. Later, when committing a single package, we have an explicit call to _alpm_local_db_read() with INFRQ_ALL as the level. Because the package's level did not match this, we skipped over our previous "does the incoming level match where I'm at" shortcut, and continued to load things again, because of a lack of fine-grained checking for each of DESC, FILES, and INSTALL. The end result is we loaded the filelist twice, causing our remove logic to iterate twice over the installed files, spewing a bunch of "cannot find file X" messages. Fix the problem by doing a bit more bitmasking logic throughout the load method, and also fix the sanity check at the beginning of the function- this should *only* be used for local packages as opposed to the "not a package" check that was there before. A debug log message was added to upgraderemove as well to match the one already in the normal remove codepath. Signed-off-by: Dan McGee <dan@archlinux.org> commit 25fab402c79aeaa5b72959c7bfd2e541a1b34045 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 21:13:04 2011 -0600 Call archive_read_data_skip() while checking diskspace libarchive eventually calls it anyway, but backtraces make a lot more sense if we call it, as well as matching our precedent from alpm_pkg_load(). Signed-off-by: Dan McGee <dan@archlinux.org> commit 6942bba75db596a70d5ba408785bf461e691bc9c Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 19:49:27 2011 -0600 Add error message stating which partition is full This is helpful anyway to the user, and should also be helpful to us if we see problems cropping up in the check during development. Also add a missing ->used = 0 initialization in the code path less taken. Signed-off-by: Dan McGee <dan@archlinux.org> commit 55bff19b7692e01a0bf276c742c77f03fb445160 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 18:52:30 2011 -0600 Unify two free diskspace error messages Although they won't be the same in the gettext catalog because of the '\n' we should still use the same text. Signed-off-by: Dan McGee <dan@archlinux.org> commit fe6e90c21f1561ee54deedbc707f5afc964d3cae Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 21:12:08 2011 -0600 Add a progressbar for package integrity checking This can take a while too, and it is really easy to add the necessary callback stuff for adding a progressbar. Signed-off-by: Dan McGee <dan@archlinux.org> commit 9a82cb92a446bce05186460bc5b5b12c4df7e5a1 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 20:16:54 2011 -0600 Small fix to download size lookup and a logger These were just two small things I came across today and found could be fixed or helpful, so I've added them and I'm not sure what else to bundle them with. commit_count++ Signed-off-by: Dan McGee <dan@archlinux.org> commit d0c327df172942da18f1e5c9b6afb183e4d9ee18 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 17:16:12 2011 -0600 doc: add docs for repo-add -d option This never got added when the option was brought in, so fix it. Signed-off-by: Dan McGee <dan@archlinux.org> commit eda4d9ec00be1108ab4336a438299a283c5a0a90 Author: PyroPeter <abi1789@googlemail.com> Date: Tue Jan 11 23:04:20 2011 +0100 repo-add: Create/modify files databases Implements FS#11302. Dan: updated docs to not reference pkgfile. Signed-off-by: PyroPeter <abi1789@googlemail.com> Signed-off-by: Dan McGee <dan@archlinux.org> commit b04a56dbe90f43622158410234fabea96d0d7f07 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 18:44:26 2011 -0600 Add two pactests with non-trivial file counts These are probably useful anyway, but also exposed the double file list bug that will be fixed in a later commit. Signed-off-by: Dan McGee <dan@archlinux.org> commit 0d4dd09993971924d379be4d0944d72f4c77b021 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 18:43:28 2011 -0600 pactest: build the filelist using a set() This will prevent duplicates, which we had plenty of once I made a few tests that had a list of files greater than the normal two. The previous logic was not working quite right. Signed-off-by: Dan McGee <dan@archlinux.org> commit 7ce90bb135c04b4421ca2824099e6e72659721b1 Author: Dan McGee <dan@archlinux.org> Date: Tue Jan 11 15:15:07 2011 -0600 repo-add: use pushd/popd Rather than explicit cd calls, we can use the directory stack to our advantage. This also removes the need to store and restore $startdir, so kill the variable entirely. Signed-off-by: Dan McGee <dan@archlinux.org> commit a9cbd15260b396c38b7d5ffe5669994e65f9a86f Author: Dan McGee <dan@archlinux.org> Date: Mon Jan 3 18:32:43 2011 -0600 pactest: correctly write epoch and force as necessary We were missing this in a few places; also add the ability to check the outcome via a new rule type. Signed-off-by: Dan McGee <dan@archlinux.org> commit 3e1bdfa93c0c0c1a966ba24f8013754a8a87f0ec Author: Dan McGee <dan@archlinux.org> Date: Mon Jan 10 08:44:23 2011 -0600 Use double rather than float everywhere No real need to use the smaller floating point types here. Signed-off-by: Dan McGee <dan@archlinux.org> commit 5f140a62debb7d55436af7f84b0ecf5696f46c86 Author: Dan McGee <dan@archlinux.org> Date: Sun Jan 9 21:40:36 2011 -0600 Progress callback cleanups and fixes * Remove a stale comment * Fix a logic error- the conditional disagreed with the comments * Remove some unnecessary floating point casts Signed-off-by: Dan McGee <dan@archlinux.org> commit 842cbc9ea4036036cf00b3b2eeccc11ad531cd2e Author: Dan McGee <dan@archlinux.org> Date: Sun Jan 9 20:47:47 2011 -0600 Ensure we use local package when calculating removed size We were checking if a package existed locally, but then using the incoming package to calculate removed size rather than the currently installed package. Also adjust the local variable in the replaces loop to make it more clear that we are always dealing with local packages here. Signed-off-by: Dan McGee <dan@archlinux.org> commit d03b57f459fb9ab9288991a70c4e7297a7c1d150 Author: Dan McGee <dan@archlinux.org> Date: Sun Jan 9 20:36:42 2011 -0600 Remove need for floating point division in backend All of these can be done with integer division; the only slightly interesting part is ensuring we round up like before with calling the ceil() function. We can also remove the math library from requirements; now that the only ceil() calls are gone, we don't need this anymore. Signed-off-by: Dan McGee <dan@archlinux.org> ----------------------------------------------------------------------- Summary of changes: configure.ac | 3 -- doc/repo-add.8.txt | 12 ++++++- lib/libalpm/add.c | 22 ++++++------ lib/libalpm/alpm.h | 1 + lib/libalpm/be_local.c | 22 +++++++------ lib/libalpm/be_package.c | 2 + lib/libalpm/conflict.c | 4 +- lib/libalpm/diskspace.c | 52 ++++++++++++++++++----------- lib/libalpm/error.c | 2 +- lib/libalpm/remove.c | 25 +++++++++----- lib/libalpm/sync.c | 12 ++++++- scripts/repo-add.sh.in | 34 ++++++++++++------- src/pacman/callback.c | 69 +++++++++++++++++++-------------------- src/pacman/package.c | 6 ++-- src/pacman/util.c | 7 ++-- test/pacman/pmdb.py | 12 +++--- test/pacman/pmpkg.py | 4 ++ test/pacman/pmrule.py | 3 ++ test/pacman/tests/remove002.py | 12 +++++++ test/pacman/tests/upgrade006.py | 18 ++++++++++ 20 files changed, 203 insertions(+), 119 deletions(-) create mode 100644 test/pacman/tests/remove002.py create mode 100644 test/pacman/tests/upgrade006.py hooks/post-receive -- The official pacman repository