Patchworkβ exec emacs at the end of run_emacs script

login
register
about
Submitter Tomi Ollila
Date 2011-11-10 23:43:03
Message ID <yf6zkg335o8.fsf@taco2.nixu.fi>
Download mbox | patch
Permalink /patch/1485/
State New
Headers show

Comments

Tomi Ollila - 2011-11-10 23:43:03
In the last line of run_emacs, exec the emacs process.
With one fork less the process list is (also) neater.
---
 test/test-lib.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Austin Clements - 2011-11-12 16:20:53
LGTM

Quoth Tomi Ollila on Nov 11 at  1:43 am:
> In the last line of run_emacs, exec the emacs process.
> With one fork less the process list is (also) neater.
> ---
>  test/test-lib.sh |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 57cb088..3ad325a 100755
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -834,7 +834,7 @@ export NOTMUCH_CONFIG=$NOTMUCH_CONFIG
>  #
>  # --load		Force loading of notmuch.el and test-lib.el
>  
> -emacs --no-init-file --no-site-file \
> +exec emacs --no-init-file --no-site-file \
>  	--directory "$TEST_DIRECTORY/../emacs" --load notmuch.el \
>  	--directory "$TEST_DIRECTORY" --load test-lib.el \
>  	"\$@"
David Bremner - 2011-11-13 12:17:34
On Fri, 11 Nov 2011 01:43:03 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> In the last line of run_emacs, exec the emacs process.
> With one fork less the process list is (also) neater.

pushed.

Patch

diff --git a/test/test-lib.sh b/test/test-lib.sh
index 57cb088..3ad325a 100755
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -834,7 +834,7 @@  export NOTMUCH_CONFIG=$NOTMUCH_CONFIG
 #
 # --load		Force loading of notmuch.el and test-lib.el
 
-emacs --no-init-file --no-site-file \
+exec emacs --no-init-file --no-site-file \
 	--directory "$TEST_DIRECTORY/../emacs" --load notmuch.el \
 	--directory "$TEST_DIRECTORY" --load test-lib.el \
 	"\$@"