Re: [pacman-dev] [PATCH 5/6] Update utilities for new initialize/release methods

2011-06-07 Thread Dan McGee
On Fri, Jun 3, 2011 at 6:05 PM, Allan McRae al...@archlinux.org wrote: On 04/06/11 08:41, Dan McGee wrote: Signed-off-by: Dan McGeed...@archlinux.org ---  src/util/cleanupdelta.c |   14 --  src/util/pactree.c      |   23 +++  src/util/testdb.c       |   17

Re: [pacman-dev] [PATCH] Remove incorrect output when downloading only

2011-06-07 Thread Dan McGee
On Mon, Jun 6, 2011 at 11:36 AM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: Patch looks fine to me but some minor comments : 1) just a matter of taste, I would reduce the number of levels with if (prompt download)   install = 1 else if prompt   ... else   ... 2) can we have downloadonly

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.5.2-26-ge62566a

2011-06-07 Thread Dan McGee
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 e62566a7635449e3e35d757c4b3bb768479689ab (commit) via

Re: [pacman-dev] [PATCH 2/6] Add helper methods for setting directory options

2011-06-07 Thread Dan McGee
On Fri, Jun 3, 2011 at 6:49 PM, Allan McRae al...@archlinux.org wrote: On 04/06/11 08:40, Dan McGee wrote: This keeps duplicate code to a minimum and also addresses the issue where calling set_cachedirs() didn't canonicalize the passed-in paths. This will come in more handy as we refactor

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.5.3-199-g8c6a636

2011-06-07 Thread Dan McGee
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 8c6a636cd970c19e7a79bb17c484b28454da877d (commit) via

[pacman-dev] [PATCH 00/13] global handle removal patch series

2011-06-07 Thread Dan McGee
This is a bitch of a diffstat, unfortunately, and the patches aren't all that fun to look through. I have it pushed to my repo as alpm-cleanups if you'd rather grab it there. The bright side is it should successfully build and test after each patch as long as they are applied in the order sent

[pacman-dev] [PATCH 01/13] Require handle argument to alpm_logaction()

2011-06-07 Thread Dan McGee
This is the first in a series of patches to update the API to remove the implicit global handle variable. Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/add.c | 34 +- lib/libalpm/alpm.h|2 +- lib/libalpm/log.c |4 ++--

[pacman-dev] [PATCH 03/13] Require handle argument to all alpm_trans_*() methods

2011-06-07 Thread Dan McGee
Begin enforcing the need to pass a handle. This allows us to remove one more extern handle declaration from the backend. Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h | 26 +- lib/libalpm/deps.c |6 +++--- lib/libalpm/trans.c | 23

[pacman-dev] [PATCH 05/13] Require handle for alpm_sync_sysupgrade()

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |3 ++- lib/libalpm/sync.c |5 + src/pacman/sync.c |2 +- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 50ab06e..2144401 100644 ---

[pacman-dev] [PATCH 04/13] Require handle for alpm_db_register_sync()

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |4 +++- lib/libalpm/db.c|5 + src/pacman/conf.c |2 +- src/util/cleanupdelta.c |2 +- src/util/testdb.c |2 +- 5 files changed, 7 insertions(+), 8 deletions(-) diff --git

[pacman-dev] [PATCH 08/13] Add handle argument to two more alpm methods

2011-06-07 Thread Dan McGee
This takes care of alpm_checkdeps() and alpm_find_dbs_satisfier(). Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |7 --- lib/libalpm/deps.c | 29 +++-- lib/libalpm/deps.h |2 +- lib/libalpm/remove.c | 27

[pacman-dev] [PATCH 06/13] Require handle for alpm_pkg_load()

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |5 +++-- lib/libalpm/be_package.c | 16 lib/libalpm/package.h|6 -- lib/libalpm/sync.c |2 +- src/pacman/query.c |2 +- src/pacman/sync.c|2 +-

[pacman-dev] [PATCH 09/13] Require handle for alpm_find_grp_pkgs()

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |3 ++- lib/libalpm/sync.c |5 +++-- src/pacman/sync.c |2 +- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 821f2fa..c493245 100644 ---

[pacman-dev] [PATCH 07/13] Require handle for alpm_checkconflicts()

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.h |2 +- lib/libalpm/conflict.c | 20 lib/libalpm/conflict.h |2 +- lib/libalpm/sync.c |2 +- src/util/testdb.c |2 +- 5 files changed, 16 insertions(+), 12 deletions(-) diff

[pacman-dev] [PATCH 12/13] Remove global handle variable

2011-06-07 Thread Dan McGee
Signed-off-by: Dan McGee d...@archlinux.org --- lib/libalpm/alpm.c |8 lib/libalpm/handle.c |4 lib/libalpm/log.c|3 --- 3 files changed, 0 insertions(+), 15 deletions(-) diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c index 86b1619..5d475ce 100644 ---

[pacman-dev] [PATCH 13/13] Always pass data to trans_commit()

2011-06-07 Thread Dan McGee
Even though we currently don't use it here in the backend, we might as well pass it in since we used it earlier. Signed-off-by: Dan McGee d...@archlinux.org --- src/pacman/remove.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/pacman/remove.c

Re: [pacman-dev] [PATCH 00/13] global handle removal patch series

2011-06-07 Thread Dan McGee
On Tue, Jun 7, 2011 at 4:36 PM, Dan McGee d...@archlinux.org wrote: This is a bitch of a diffstat, unfortunately, and the patches aren't all that fun to look through. I have it pushed to my repo as alpm-cleanups if you'd rather grab it there. The bright side is it should successfully build and

Re: [pacman-dev] [PATCH 09/13] Require handle for alpm_find_grp_pkgs()

2011-06-07 Thread Dan McGee
On Tue, Jun 7, 2011 at 5:19 PM, Rémy Oudompheng remyoudomph...@gmail.com wrote: On 2011/6/7 Dan McGee d...@archlinux.org wrote: -alpm_list_t SYMEXPORT *alpm_find_grp_pkgs(alpm_list_t *dbs, +alpm_list_t SYMEXPORT *alpm_find_grp_pkgs(pmhandle_t *handle, alpm_list_t *dbs,                const

Re: [pacman-dev] [PATCH 09/13] Require handle for alpm_find_grp_pkgs()

2011-06-07 Thread Allan McRae
On 08/06/11 08:29, Dan McGee wrote: On Tue, Jun 7, 2011 at 5:19 PM, Rémy Oudompheng remyoudomph...@gmail.com wrote: On 2011/6/7 Dan McGeed...@archlinux.org wrote: -alpm_list_t SYMEXPORT *alpm_find_grp_pkgs(alpm_list_t *dbs, +alpm_list_t SYMEXPORT *alpm_find_grp_pkgs(pmhandle_t *handle,

Re: [pacman-dev] [PATCH 3/6] Update Makefile.am and configure.ac for gpgdir

2011-06-07 Thread Allan McRae
On 02/06/11 04:09, Dan McGee wrote: On Wed, Jun 1, 2011 at 4:23 AM, Allan McRaeal...@archlinux.org wrote: On 29/05/11 00:37, Pang Yan Han wrote: install-data-local target in Makefile.am will now create the default GPGDir, trustdb.gpg, pubring.gpg if they do not exist. configure script will

Re: [pacman-dev] [PATCH 09/13] Require handle for alpm_find_grp_pkgs()

2011-06-07 Thread Dan McGee
On Tue, Jun 7, 2011 at 6:38 PM, Allan McRae al...@archlinux.org wrote: On 08/06/11 08:29, Dan McGee wrote: On Tue, Jun 7, 2011 at 5:19 PM, Rémy Oudompheng remyoudomph...@gmail.com  wrote: On 2011/6/7 Dan McGeed...@archlinux.org  wrote: -alpm_list_t SYMEXPORT *alpm_find_grp_pkgs(alpm_list_t

[pacman-dev] [PATCH] Remove incorrect output with download only and IgnorePkg

2011-06-07 Thread Allan McRae
When only downloading a package that is in IgnorePkg, pacman incorrectly asks about installing. e.g. with pkg in IgnorePkg in pacman.conf: pacman -Sddw pkg :: pkg is in IgnorePkg/IgnoreGroup. Install anyway? [Y/n] This output is now silenced when downloading only. Signed-off-by: Allan McRae