[pacman-dev] [PATCH 1/4] lib/handle: add get/set methods for useragent

2011-03-28 Thread Dave Reisner
Signed-off-by: Dave Reisner d...@falconindy.com --- lib/libalpm/alpm.h |3 +++ lib/libalpm/dload.c |4 ++-- lib/libalpm/handle.c | 15 +++ lib/libalpm/handle.h |1 + 4 files changed, 21 insertions(+), 2 deletions(-) diff --git a/lib/libalpm/alpm.h b/lib/libalpm

[pacman-dev] [PATCH 0/4] addition of API methods and backend work

2011-03-28 Thread Dave Reisner
I've had the ensuing patches on my tree for a few days. A quick outline of what they accomplish: 1) Replace the old libfetch method of setting useragent (via an environment var) with proper get/set methods. 2) Refactor signature downloading to ensure that the sig file is downloaded from the same

[pacman-dev] [PATCH 4/4] Allow VerifySig to act as a default verification in [options]

2011-03-28 Thread Dave Reisner
* add _alpm_get_sigverify_level * add alpm_option_{get,set}_default_sigverify This requires moving around protos in alpm.h so that pgp_verify_t is known to the get/set methods for default_sigverify. Signed-off-by: Dave Reisner d...@falconindy.com --- lib/libalpm/alpm.h| 36

Re: [pacman-dev] [PATCH 4/4] Allow VerifySig to act as a default verification in [options]

2011-03-28 Thread Dave Reisner
On Mon, Mar 28, 2011 at 09:34:57PM +0200, Rémy Oudompheng wrote: On Mon 28 March 2011 at 15:15 -0400, Dave Reisner wrote: +/* GPG signature verification option */ +typedef enum _pgp_verify_t { + PM_PGP_VERIFY_UNKNOWN, + PM_PGP_VERIFY_ALWAYS, + PM_PGP_VERIFY_OPTIONAL

Re: [pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.5.0-78-gf7b577d

2011-03-23 Thread Dave Reisner
On Wed, Mar 23, 2011 at 03:36:07AM -0400, Dan McGee 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

[pacman-dev] [RFC] Replacing wget with curl in makepkg

2011-03-23 Thread Dave Reisner
The attached patch proposes to replace wget with curl as the default DLAGENT in makepkg for http, https, and ftp. This seems like the sane choice given our switch from fetch to curl. dave From 4e0ad1833fa5c8ef725ce3144e6318c397f60782 Mon Sep 17 00:00:00 2001 From: Dave Reisner d...@falconindy.com

[pacman-dev] [RFC] Proposed fix for FS#20538

2011-03-23 Thread Dave Reisner
2001 From: Dave Reisner d...@falconindy.com Date: Fri, 18 Mar 2011 11:03:28 -0400 Subject: [PATCH] pacman/util: flush terminal input before reading response Addresses FS#20538 Signed-off-by: Dave Reisner d...@falconindy.com --- src/pacman/util.c | 17 + src/pacman/util.h |1

Re: [pacman-dev] [PATCH] Don't initialize progress to zero before calling curl_easy_perform().

2011-03-21 Thread Dave Reisner
On Mon, Mar 21, 2011 at 07:50:37AM -0500, Dan McGee wrote: On Mon, Mar 21, 2011 at 5:53 AM, Lukas Fleischer archli...@cryptocrack.de wrote: Drawing progress bars before calling curl_easy_perform() is needless as the curl progress callback is called with zero progress before actually

Re: [pacman-dev] change to option handling for --debug and -v

2011-03-19 Thread Dave Reisner
On Sat, Mar 19, 2011 at 12:10:58PM -0400, Ray Kohler wrote: I noticed a change in the way 3.5 processes some of its command-line flags: It used to be that --debug and -v were handled immediately, such that their initial output appeared before pacman complained of a usage error. Now, it errors

Re: [pacman-dev] [PATCH] Use sane umask for repo db downloads

2011-03-18 Thread Dave Reisner
On Sat, Mar 19, 2011 at 11:24:22AM +1000, Allan McRae wrote: Fixes FS#23343. Signed-off-by: Allan McRae al...@archlinux.org --- lib/libalpm/be_sync.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index

Re: [pacman-dev] [arch-dev-public] [signoff] pacman 3.5.0-1

2011-03-16 Thread Dave Reisner
On Wed, Mar 16, 2011 at 03:29:24PM -0500, Dan McGee wrote: On Wed, Mar 16, 2011 at 3:26 PM, Aaron Griffin aaronmgrif...@gmail.com wrote: On Wed, Mar 16, 2011 at 3:24 PM, Dan McGee dpmc...@gmail.com wrote: On Wed, Mar 16, 2011 at 3:18 PM, Gaetan Bisson bis...@archlinux.org wrote:

[pacman-dev] [PATCH] Only read from stdin if sole target is '-'

2011-03-16 Thread Dave Reisner
This prevents a regression for people who enjoy piping yes to pacman to avoid prompts. Signed-off-by: Dave Reisner d...@falconindy.com --- src/pacman/pacman.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 984bd1b

Re: [pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.5.0-18-ge6c9076

2011-03-16 Thread Dave Reisner
-0500 Merge branch 'maint' commit cff36093f3b778b3bda9c9d1a3acf24e36dd9f73 Merge: 92630c6 f2eac18 Author: Dan McGee d...@archlinux.org Date: Wed Mar 16 19:25:35 2011 -0500 Merge branch 'download' commit f2eac18a6ec62db3ec53744064e05416024c1b30 Author: Dave Reisner d

[pacman-dev] (no subject)

2011-02-25 Thread Dave Reisner
The ensuing patches revert Allan's breakage^Wchanges to the public API with regard to the introduction of pmpkghash_t. Since we haven't exposed any public facing API calls which can manipulate a pmpkghash_t, there's not much point in exposing the type along with it. alpm_db_get_pkgcache_list is

[pacman-dev] [PATCH 1/2] alpm: remove public visibility of pmpkghash_t

2011-02-25 Thread Dave Reisner
There's no API functions exposed which allow manipulation of this type, so remove it from public view. Also, rename the public and private alpm_db_get_pkgcache symbol to alpm_db_get_pkgcache_has. Signed-off-by: Dave Reisner d...@falconindy.com --- lib/libalpm/alpm.h|2 -- lib/libalpm

[pacman-dev] [PATCH 2/2] alpm: alpm_db_get_pkgcache_list = alpm_db_get_pkgcache

2011-02-25 Thread Dave Reisner
This avoids needless breakage of the public API. Signed-off-by: Dave Reisner d...@falconindy.com --- lib/libalpm/alpm.h |2 +- lib/libalpm/conflict.c |2 +- lib/libalpm/db.c| 12 ++-- lib/libalpm/db.h|2 +- lib/libalpm/deps.c |6 +++--- lib

Re: [pacman-dev] Style questions and concerns

2011-02-25 Thread Dave Reisner
On Fri, Feb 25, 2011 at 10:27:13AM -0600, Dan McGee wrote: I've noticed these things a lot in recent patches, so let the discussion commence. 1) typedef-ed structs. This is just a copy the rest of them habit, but I really feel we should stop doing this, only typedef-ing when we are making a

Re: [pacman-dev] makepkg bug with epoch code

2011-02-09 Thread Dave Reisner
On Wed, Feb 09, 2011 at 08:52:55PM +1000, Allan McRae wrote: Have not had time to look into this, but I get this error with the kernel26 PKGBUILD in Arch Linux [core] repo: makepkg -gc == Retrieving Sources... - Found linux-2.6.37.tar.bz2 - Found patch-2.6.37-4-ARCH.bz2 - Found

Re: [pacman-dev] [PATCH] makepkg: more bash-3.2 compatibility

2011-02-04 Thread Dave Reisner
On Sat, Feb 05, 2011 at 11:54:04AM +1000, Allan McRae wrote: On 05/02/11 11:02, Dave Reisner wrote: On Sat, Feb 05, 2011 at 10:26:20AM +1000, Allan McRae wrote: Adding the || true to the subshell prevents bash-3.2 setting off the error_trap but requires changing the if statement. Signed-off

Re: [pacman-dev] [PATCH] makepkg: more bash-3.2 compatibility

2011-02-04 Thread Dave Reisner
On Sat, Feb 05, 2011 at 12:55:38PM +1000, Allan McRae wrote: It may well be a bug in bash-3.2. Either way, this fixes the issue. I'm open to suggestions for a better fix. Allan Wild. This is as good as any solution I can think of. set -e is somewhat frowned upon for regular usage, so I

Re: [pacman-dev] makepkg bash-3.2 issue

2011-02-02 Thread Dave Reisner
On Thu, Feb 03, 2011 at 10:33:45AM +1000, Allan McRae wrote: Hi, Can anyone give me a fix for this: local optdepends_list eval $(awk '/^[[:space:]]*optdepends=/,/\)/' $BUILDFILE | sed s/optdepends=/optdepends_list+=/) In bash-3.2, this gives optdepends_list with as the first item in

Re: [pacman-dev] [PATCH 2/2] makepkg: add sodepends support

2011-01-31 Thread dave reisner
On Mon, Jan 31, 2011 at 2:15 PM, Florian Pritz bluew...@server-speed.netwrote: From: Florian Pritz bluew...@xssn.at The user adds libaries to the depends array without a version. These must end with .so. Example: depends=(glibc libc.so) find_sodepends() looks for ELF files (not symlinks

[pacman-dev] [RFC] curl-based internal downloader

2011-01-02 Thread Dave Reisner
Greetings, I've been working on and off on a curl-based internal downloader to replace the libfetch based implementation. Why? - libcurl is more widely used in the arch repos. fetch is used for one and only package: pacman. - support for compression over http connections. I see proposals for

[pacman-dev] [PATCH 1/4] add curl to alpm initialization and teardown routines

2011-01-02 Thread Dave Reisner
Signed-off-by: Dave Reisner d...@falconindy.com --- lib/libalpm/alpm.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c index 3f9cfff..44c513e 100644 --- a/lib/libalpm/alpm.c +++ b/lib/libalpm/alpm.c @@ -27,6 +27,9

[pacman-dev] [PATCH 2/4] implement a curl based download_file()

2011-01-02 Thread Dave Reisner
This does not implement any linkage from current code to the curl based download_file() function. libfetch continues to be used unless configured and compiled explicitly with LDFLAGS/CPPFLAGS set for -lcurl and -DHAVE_LIBCURL, respectively. Signed-off-by: Dave Reisner d...@falconindy.com --- lib

[pacman-dev] [PATCH 4/4] Perform a fetch-ectomy, removing libfetch entirely

2011-01-02 Thread Dave Reisner
Signed-off-by: Dave Reisner d...@falconindy.com --- configure.ac| 22 + lib/libalpm/alpm.c | 12 --- lib/libalpm/dload.c | 264 +-- 3 files changed, 6 insertions(+), 292 deletions(-) diff --git a/configure.ac b/configure.ac index

[pacman-dev] [PATCH 3/4] Add configure.ac option for --with-curl

2011-01-02 Thread Dave Reisner
To avoid breaking compilation, fetch defaults to 'no', and curl defaults to 'check'. Signed-off-by: Dave Reisner d...@falconindy.com --- configure.ac | 19 ++- 1 files changed, 18 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 5627fb7..3d536cd

[pacman-dev] [PATCH] Perform a fetch-ectomy, removing libfetch entirely

2011-01-02 Thread Dave Reisner
Signed-off-by: Dave Reisner d...@falconindy.com --- configure.ac| 24 - lib/libalpm/alpm.c | 12 --- lib/libalpm/dload.c | 264 +-- 3 files changed, 3 insertions(+), 297 deletions(-) Resend, because I'm sloppy. diff --git

Re: [pacman-dev] [PATCH 2/4] implement a curl based download_file()

2011-01-02 Thread Dave Reisner
On Sun, Jan 02, 2011 at 08:35:01PM -0600, Dan McGee wrote: On Sun, Jan 2, 2011 at 7:13 PM, Dave Reisner d...@falconindy.com wrote: This does not implement any linkage from current code to the curl based download_file() function. libfetch continues to be used unless configured and compiled

Re: [pacman-dev] makepkg and (n)awk

2010-12-26 Thread Dave Reisner
On Sun, Dec 26, 2010 at 01:59:05AM +0200, Nezmer wrote: Hi, In commit ef977865, awk was added as an external tool needed by makepkg. As you know, there is different awk implementations out there. nawk is the implementation shipped by default in non-GNU systems As far as I know.

Re: [pacman-dev] [PATCH 0/4] Package list find performance improvements

2010-12-14 Thread Dave Reisner
On Tue, Dec 14, 2010 at 12:46:15PM -0600, Dan McGee wrote: 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

Re: [pacman-dev] [PATCH] Warn when synchronizing without upgrading (-Sy)

2010-11-17 Thread Dave Reisner
On Wed, Nov 17, 2010 at 11:53:56PM +0200, Evangelos Foutras wrote: On Wed, Nov 17, 2010 at 11:37 PM, Thomas Bahn thomas-b...@gmx.net wrote: Am Mittwoch 17 November 2010, 22:20:08 schrieb Evangelos Foutras: One problem I see with only printing a warning is that the user doesn't have the

[pacman-dev] [PATCH] Support reading package args from stdin

2010-11-04 Thread Dave Reisner
to the terminal. The above operation performed using xargs would require the --noconfirm flag to be passed to pacman. Revised from the original 7/24 submission at Allan's request to check the return value of freopen(). Signed-off-by: Dave Reisner d...@falconindy.com --- src/pacman/pacman.c | 27

Re: [pacman-dev] [PATCH] Support reading package args from stdin

2010-11-04 Thread Dave Reisner
On Fri, Nov 05, 2010 at 12:42:54AM +1000, Allan McRae wrote: On 05/11/10 00:20, Dave Reisner wrote: 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

Re: [pacman-dev] [PATCH] Support reading package args from stdin

2010-11-04 Thread Dave Reisner
On Thu, Nov 04, 2010 at 10:34:29AM -0500, Dan McGee wrote: On Thu, Nov 4, 2010 at 10:27 AM, Dave Reisner d...@falconindy.com wrote: On Thu, Nov 04, 2010 at 09:40:46AM -0500, Dan McGee wrote: On Thu, Nov 4, 2010 at 9:20 AM, Dave Reisner d...@falconindy.com wrote: Only occurs if no arguments

[pacman-dev] [PATCH] Support reading package args from stdin

2010-11-04 Thread Dave Reisner
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...@falconindy.com --- Resending as per Allan's request with modifications from the origin 7/24 submission. Changes: * Check return value of freopen * Assign '\0

[pacman-dev] [PATCH] Support reading package args from stdin

2010-11-04 Thread Dave Reisner
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...@falconindy.com --- Additional cleanup at Dan's request, and added documentation to man page wrt to this feature. Woo, documentation. doc/pacman.8.txt

[pacman-dev] [RFC] pactree rewrite in C

2010-10-11 Thread Dave Reisner
The recent patchwork over the weekend inspired me to take a closer look at pactree. My first impression was that it would be a great candidate for a rewrite in C. http://www.github.com/falconindy/pactree Results are entertaining: # bash version $ time pactree -r glib2 /dev/null real

Re: [pacman-dev] [RFC] pactree rewrite in C

2010-10-11 Thread Dave Reisner
On Mon, Oct 11, 2010 at 04:49:00PM +0200, Xavier Chantry wrote: On Mon, Oct 11, 2010 at 3:32 PM, Dan McGee dpmc...@gmail.com wrote: On Mon, Oct 11, 2010 at 8:06 AM, Dave Reisner d...@falconindy.com wrote: The recent patchwork over the weekend inspired me to take a closer look at pactree. My

Re: [pacman-dev] [RFC] pactree rewrite in C

2010-10-11 Thread Dave Reisner
On Mon, Oct 11, 2010 at 11:12:47PM +0200, Xavier Chantry wrote: On Mon, Oct 11, 2010 at 5:10 PM, Dave Reisner d...@falconindy.com wrote: Oooh, thanks for the reminder -- I had meant to resolve command line args as providers and never got around to it. I'll need to read up on dot's graph

Re: [pacman-dev] [PATCH] makepkg: canonicalize paths from environmental variables

2010-10-04 Thread Dave Reisner
On Mon, Oct 04, 2010 at 06:57:17AM -0500, Dan McGee wrote: On Mon, Oct 4, 2010 at 5:28 AM, Dave Reisner d...@falconindy.com wrote: On Mon, Oct 04, 2010 at 03:01:50PM +1000, Allan McRae wrote: This prevents circular symlinks and weird final package locations when using commands like

Re: [pacman-dev] [PATCH] bash_completion: negate expression inside brackets

2010-07-26 Thread Dave Reisner
On Sun, Jul 25, 2010 at 07:49:33PM -0500, Dan McGee wrote: On Sun, Jul 25, 2010 at 7:43 PM, Dave Reisner d...@falconindy.com wrote: Avoids letting the shell evaluate the ! as something else (e.g. if ! is an alias). Thanks, applied to maint. -Dan I'm noticing that I misread

[pacman-dev] [PATCH] bash_completion: negate expression inside brackets

2010-07-26 Thread Dave Reisner
Avoids letting the shell evaluate ! as something else (e.g. an alias). Signed-off-by: Dave Reisner d...@falconindy.com --- contrib/bash_completion |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/contrib/bash_completion b/contrib/bash_completion index 1ec2cd5..bdc4754

[pacman-dev] [PATCH] bash_completion: negate expression inside brackets

2010-07-25 Thread Dave Reisner
Avoids letting the shell evaluate the ! as something else (e.g. if ! is an alias). Signed-off-by: Dave Reisner d...@falconindy.com --- contrib/bash_completion |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/contrib/bash_completion b/contrib/bash_completion index 1ec2cd5

Re: [pacman-dev] [PATCH] Support reading package args from stdin

2010-07-24 Thread Dave Reisner
On Sat, Jul 24, 2010 at 10:30:45AM -0400, Daenyth Blank wrote: On Sat, Jul 24, 2010 at 10:21, Dan McGee dpmc...@gmail.com wrote: On Fri, Jul 23, 2010 at 10:26 PM, Dave Reisner d...@falconindy.com wrote: Only occurs if no arguments were provided directly. Arguments can be separated by any

[pacman-dev] [PATCH] Support reading package args from stdin

2010-07-23 Thread Dave Reisner
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 Signed-off-by: Dave Reisner d...@falconindy.com --- src/pacman/pacman.c

[pacman-dev] mmerge with dupe removal

2010-03-29 Thread dave reisner
to go that route. -dave reisner /** * @brief merge sort with duplicate deletion * * @param left left side of merge * @param rightright side of merge * @param fn callback function for comparing data * @param fnf callback function for deleting data * * @return the merged list

<    4   5   6   7   8   9