Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-16 Thread Pieter Praet
On Fri, 11 Nov 2011 10:09:17 +0200, Tomi Ollila wrote: > On Fri, 11 Nov 2011 01:17:37 +0100, Pieter Praet wrote: > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > > wrote: > > > From: Tomi Ollila > > > > > > dtach is lighter than screen and is not setuid/setgid program so > > >

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-15 Thread Pieter Praet
On Fri, 11 Nov 2011 10:09:17 +0200, Tomi Ollila wrote: > On Fri, 11 Nov 2011 01:17:37 +0100, Pieter Praet wrote: > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > finestructure.net> wrote: > > > From: Tomi Ollila > > > > > > dtach is lighter than screen and is not setuid/setgid p

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-12 Thread Dmitry Kurochkin
On Fri, 11 Nov 2011 22:48:13 +0200, Tomi Ollila wrote: > dtach is simpler than screen and is not setuid/setgid program so > TMPDIR does not get cleared by dynamic loader when executed. > --- > > Updated version after discussion with DmitryKurochkin and amdragon > on IRC. Thank you. > Looks good

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed --- The 2 Fixes based Dmitry's comments + Jameson's one too. test/test-lib.sh | 19 ++- 1 files changed, 10 insertions(+), 9 deletions(-) diff --git a/t

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread David Bremner
On Fri, 11 Nov 2011 23:33:58 +0200, Tomi Ollila wrote: > dtach is simpler than screen and is not setuid/setgid program so > TMPDIR does not get cleared by dynamic loader when executed Pushed.

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
On Sat, 12 Nov 2011 01:06:40 +0400, Dmitry Kurochkin wrote: > On Fri, 11 Nov 2011 22:48:13 +0200, Tomi Ollila wrote: > > dtach is simpler than screen and is not setuid/setgid program so > > TMPDIR does not get cleared by dynamic loader when executed. > > --- > > > > Updated version after discuss

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed. --- Updated version after discussion with DmitryKurochkin and amdragon on IRC. Thank you. test/test-lib.sh | 25 + 1 files changed, 13 insertion

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed. --- Updated version after discussion with DmitryKurochkin and amdragon on IRC. Thank you. test/test-lib.sh | 25 + 1 files changed, 13 insertion

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread David Bremner
On Fri, 11 Nov 2011 23:33:58 +0200, Tomi Ollila wrote: > dtach is simpler than screen and is not setuid/setgid program so > TMPDIR does not get cleared by dynamic loader when executed Pushed. ___ notmuch mailing list notmuch@notmuchmail.org http://notmu

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Dmitry Kurochkin
On Fri, 11 Nov 2011 13:29:48 +0200, Tomi Ollila wrote: > On Fri, 11 Nov 2011 12:41:11 +0400, Dmitry Kurochkin gmail.com> wrote: > > Hi Tomi. > > > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > finestructure.net> wrote: > > > From: Tomi Ollila > > > > > > dtach is lighter than

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Jameson Graef Rollins
On Fri, 11 Nov 2011 20:02:52 +0400, Dmitry Kurochkin wrote: > My point is that if a system can not run "emacs -nw" because it lacks > corresponding terminfo, it is not notmuch who should work around these > issues. If you are using such system by whatever reason, it is your > responsibility to r

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Jameson Graef Rollins
On Fri, 11 Nov 2011 20:02:52 +0400, Dmitry Kurochkin wrote: > My point is that if a system can not run "emacs -nw" because it lacks > corresponding terminfo, it is not notmuch who should work around these > issues. If you are using such system by whatever reason, it is your > responsibility to ru

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed --- The 2 Fixes based Dmitry's comments + Jameson's one too. test/test-lib.sh | 19 ++- 1 files changed, 10 insertions(+), 9 deletions(-) diff --git a/t

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Tomi Ollila
On Fri, 11 Nov 2011 12:41:11 +0400, Dmitry Kurochkin wrote: > Hi Tomi. > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins finestructure.net> wrote: > > From: Tomi Ollila > > > > dtach is lighter than screen and is not setuid/setgid program so > > TMPDIR does not get reset by dynamic

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
On Sat, 12 Nov 2011 01:06:40 +0400, Dmitry Kurochkin wrote: > On Fri, 11 Nov 2011 22:48:13 +0200, Tomi Ollila wrote: > > dtach is simpler than screen and is not setuid/setgid program so > > TMPDIR does not get cleared by dynamic loader when executed. > > --- > > > > Updated version after discus

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Dmitry Kurochkin
On Fri, 11 Nov 2011 22:48:13 +0200, Tomi Ollila wrote: > dtach is simpler than screen and is not setuid/setgid program so > TMPDIR does not get cleared by dynamic loader when executed. > --- > > Updated version after discussion with DmitryKurochkin and amdragon > on IRC. Thank you. > Looks good

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed. --- Updated version after discussion with DmitryKurochkin and amdragon on IRC. Thank you. test/test-lib.sh | 25 + 1 files changed, 13 insertion

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests

2011-11-11 Thread Tomi Ollila
dtach is simpler than screen and is not setuid/setgid program so TMPDIR does not get cleared by dynamic loader when executed. --- Updated version after discussion with DmitryKurochkin and amdragon on IRC. Thank you. test/test-lib.sh | 25 + 1 files changed, 13 insertion

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Dmitry Kurochkin
Hi Tomi. On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins wrote: > From: Tomi Ollila > > dtach is lighter than screen and is not setuid/setgid program so > TMPDIR does not get reset by dynamic loader when executed. > > Signed-off-by: Jameson Graef Rollins > --- > This tweaks the orig

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Tomi Ollila
On Fri, 11 Nov 2011 01:17:37 +0100, Pieter Praet wrote: > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins finestructure.net> wrote: > > From: Tomi Ollila > > > > dtach is lighter than screen and is not setuid/setgid program so > > TMPDIR does not get reset by dynamic loader when execu

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Dmitry Kurochkin
On Fri, 11 Nov 2011 13:29:48 +0200, Tomi Ollila wrote: > On Fri, 11 Nov 2011 12:41:11 +0400, Dmitry Kurochkin > wrote: > > Hi Tomi. > > > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > > wrote: > > > From: Tomi Ollila > > > > > > dtach is lighter than screen and is not setuid

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Tomi Ollila
On Fri, 11 Nov 2011 12:41:11 +0400, Dmitry Kurochkin wrote: > Hi Tomi. > > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > wrote: > > From: Tomi Ollila > > > > dtach is lighter than screen and is not setuid/setgid program so > > TMPDIR does not get reset by dynamic loader when ex

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Pieter Praet
On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins wrote: > From: Tomi Ollila > > dtach is lighter than screen and is not setuid/setgid program so > TMPDIR does not get reset by dynamic loader when executed. > `dtach' may be lighter than `screen', but contrary to my expectations, it app

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Dmitry Kurochkin
Hi Tomi. On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins wrote: > From: Tomi Ollila > > dtach is lighter than screen and is not setuid/setgid program so > TMPDIR does not get reset by dynamic loader when executed. > > Signed-off-by: Jameson Graef Rollins > --- > This tweaks the ori

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Tomi Ollila
On Fri, 11 Nov 2011 01:17:37 +0100, Pieter Praet wrote: > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins > wrote: > > From: Tomi Ollila > > > > dtach is lighter than screen and is not setuid/setgid program so > > TMPDIR does not get reset by dynamic loader when executed. > > > > `

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-11 Thread Tomi Ollila
dtach is lighter than screen and is not setuid/setgid program so TMPDIR does not get reset by dynamic loader when executed. --- test/test-lib.sh |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh index c81c709..af723ad 100755 --- a/test/

Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-10 Thread Pieter Praet
On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins wrote: > From: Tomi Ollila > > dtach is lighter than screen and is not setuid/setgid program so > TMPDIR does not get reset by dynamic loader when executed. > `dtach' may be lighter than `screen', but contrary to my expectations, it ap

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-10 Thread Jameson Graef Rollins
From: Tomi Ollila dtach is lighter than screen and is not setuid/setgid program so TMPDIR does not get reset by dynamic loader when executed. Signed-off-by: Jameson Graef Rollins --- This tweaks the original patch slightly by removing the no-longer needed screenrc variables. test/test-lib.sh

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-10 Thread Jameson Graef Rollins
From: Tomi Ollila dtach is lighter than screen and is not setuid/setgid program so TMPDIR does not get reset by dynamic loader when executed. Signed-off-by: Jameson Graef Rollins --- This tweaks the original patch slightly by removing the no-longer needed screenrc variables. test/test-lib.sh

[PATCH] test: use dtach(1) instead of screen(1) in emacs tests.

2011-11-10 Thread Tomi Ollila
dtach is lighter than screen and is not setuid/setgid program so TMPDIR does not get reset by dynamic loader when executed. --- test/test-lib.sh |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh index c81c709..af723ad 100755 --- a/test/