[pacman-dev] [PATCH] Make log redirection saner

2011-03-24 Thread Jan Steffens
My main motivation was to remove the "sync", which can stall for minutes on a busy machine (FS#23378). I also cleaned up the redirection. Signed-off-by: Jan Steffens --- scripts/makepkg.sh.in | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/scripts/makepkg.sh.in

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

2011-03-24 Thread Dave Reisner
On Thu, Mar 24, 2011 at 12:18:20AM -0500, Dan McGee wrote: > On Wed, Mar 23, 2011 at 10:41 AM, Dave Reisner wrote: > > https://bugs.archlinux.org/task/20538 > > > > The attached patch flushes the terminal input buffer shortly before > > prompting users for input to answer a question. On the one ha

Re: [pacman-dev] [PATCH] Make log redirection saner

2011-03-24 Thread Dan McGee
On Thu, Mar 24, 2011 at 7:30 AM, Jan Steffens wrote: > My main motivation was to remove the "sync", which can stall for > minutes on a busy machine (FS#23378). I also cleaned up the redirection. > > Signed-off-by: Jan Steffens Fine with the patch, but can I ask how rigorous the testing is this ha

[pacman-dev] [PATCH] Fix use of relative paths for packages in repo-add

2011-03-24 Thread Ray Kohler
I first noticed that checksums weren't calculated if a relative path was used, since they're done after moving into the $tmpdir and relative paths become useless. Then I saw that PGP sigs had basically the same problem, with the addition of the introduction of $startdir, which was never set beforeh

Re: [pacman-dev] [PATCH] Fix use of relative paths for packages in repo-add

2011-03-24 Thread Dan McGee
On Thu, Mar 24, 2011 at 4:46 PM, Ray Kohler wrote: > I first noticed that checksums weren't calculated if a relative path was used, > since they're done after moving into the $tmpdir and relative paths become > useless. Then I saw that PGP sigs had basically the same problem, with the > addition o

[pacman-dev] [PATCH] Fix use of relative paths for packages in repo-add

2011-03-24 Thread Ray Kohler
Move checksum and pgpsig calcluation before changing into the tmpdir, otherwise we can't find the files if a relative path was used. Signed-off-by: Ray Kohler --- scripts/repo-add.sh.in | 26 ++ 1 files changed, 14 insertions(+), 12 deletions(-) diff --git a/scripts/re

Re: [pacman-dev] [PATCH] Fix use of relative paths for packages in repo-add

2011-03-24 Thread Dan McGee
On Thu, Mar 24, 2011 at 5:05 PM, Ray Kohler wrote: > Move checksum and pgpsig calcluation before changing into the > tmpdir, otherwise we can't find the files if a relative path > was used. Thanks! I made a few small touchups to the local var declaration to ensure sha256sum and pgpsig made it int

[pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Ray Kohler
Signed-off-by: Ray Kohler --- scripts/pacman-key.sh.in |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in index 5746e64..ccc7f92 100644 --- a/scripts/pacman-key.sh.in +++ b/scripts/pacman-key.sh.in @@ -246,6 +246,12 @

[pacman-dev] [PATCH] Add -T, --deptest to usage message

2011-03-24 Thread Ray Kohler
Fixes FS #23369 Signed-off-by: Ray Kohler --- src/pacman/pacman.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 0487ee8..d10cefe 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -123,6 +123,7 @@ static void u

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Denis A . Altoé Falqueto
On Thu, Mar 24, 2011 at 8:05 PM, Ray Kohler wrote: > > Signed-off-by: Ray Kohler > --- >  scripts/pacman-key.sh.in |    6 ++ >  1 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in > index 5746e64..ccc7f92 100644 > --- a/scripts

Re: [pacman-dev] [PATCH] Make log redirection saner

2011-03-24 Thread Allan McRae
On 25/03/11 05:07, Dan McGee wrote: On Thu, Mar 24, 2011 at 7:30 AM, Jan Steffens wrote: My main motivation was to remove the "sync", which can stall for minutes on a busy machine (FS#23378). I also cleaned up the redirection. Signed-off-by: Jan Steffens Fine with the patch, but can I ask how

Re: [pacman-dev] [PATCH] Add -T, --deptest to usage message

2011-03-24 Thread Dan McGee
On Thu, Mar 24, 2011 at 6:17 PM, Ray Kohler wrote: > Fixes FS #23369 But only partially, or at least when one takes a closer look. See below: dmcgee@galway ~/projects/pacman-maint (maint) $ pacman -T --help -b, --dbpath set an alternate database location -r, --root set an alternate ins

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Allan McRae
On 25/03/11 11:00, Denis A. Altoé Falqueto wrote: On Thu, Mar 24, 2011 at 8:05 PM, Ray Kohler wrote: Signed-off-by: Ray Kohler --- scripts/pacman-key.sh.in |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in index

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Dan McGee
On Thu, Mar 24, 2011 at 8:00 PM, Denis A. Altoé Falqueto wrote: > On Thu, Mar 24, 2011 at 8:05 PM, Ray Kohler wrote: >> >> Signed-off-by: Ray Kohler >> --- >>  scripts/pacman-key.sh.in |    6 ++ >>  1 files changed, 6 insertions(+), 0 deletions(-) >> >> diff --git a/scripts/pacman-key.sh.in

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Ray Kohler
On Thu, Mar 24, 2011 at 9:36 PM, Allan McRae wrote: > I agree.   I still think these folder should be created by "make install" > but it makes sense for pacman-key to create the folder if it does not exist. > > My pacman-git PKGBUILD has this in it > > install -dm700 $pkgdir/etc/pacman.d/gnupg

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Allan McRae
On 25/03/11 11:32, Dan McGee wrote: On Thu, Mar 24, 2011 at 8:00 PM, Denis A. Altoé Falqueto wrote: On Thu, Mar 24, 2011 at 8:05 PM, Ray Kohler wrote: Signed-off-by: Ray Kohler --- scripts/pacman-key.sh.in |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/scri

Re: [pacman-dev] [PATCH] Create pacman keyring directory if missing

2011-03-24 Thread Ray Kohler
On Thu, Mar 24, 2011 at 9:49 PM, Allan McRae wrote: > On 25/03/11 11:32, Dan McGee wrote: >> >> On Thu, Mar 24, 2011 at 8:00 PM, Denis A. Altoé Falqueto >>  wrote: >>> >>> On Thu, Mar 24, 2011 at 8:05 PM, Ray Kohler >>>  wrote: Signed-off-by: Ray Kohler ---  scripts/pacman-key

[pacman-dev] [PATCH] Add -T, --deptest to usage message

2011-03-24 Thread Ray Kohler
Fixes FS #23369 Signed-off-by: Ray Kohler --- src/pacman/pacman.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 0487ee8..da6b71e 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -123,6 +123,7 @@ static voi

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.5.1-3-gc37c9c5

2011-03-24 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 c37c9c5dcae06c1585b4da4c0b838f5b44e1498e (commit) via

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.5.1-71-g86e7f60

2011-03-24 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 86e7f6075671e08c456f389d8ecba47190c03f0f (commit) vi