unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] test: use --quick when starting emacs.
@ 2014-05-29  0:26 David Bremner
  2014-05-29  0:33 ` David Bremner
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: David Bremner @ 2014-05-29  0:26 UTC (permalink / raw)
  To: notmuch

At least in emacs24, this removes the "site-lisp" directories from the
load path in addition to enforcing --no-site-lisp --no-init-file.

This works around a slightly mysterious bug on Debian that causes
test-lib.el not to load when there is cl-lib.el(c) in some site-lisp
directory.  It should be harmless in general since we really don't
want to load any files from addon packages to emacs.
---
 test/test-lib.sh | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/test/test-lib.sh b/test/test-lib.sh
index 8697d6a..17deaab 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -1081,15 +1081,14 @@ export NOTMUCH_CONFIG=$NOTMUCH_CONFIG
 
 # Here's what we are using here:
 #
-# --no-init-file	Don't load users ~/.emacs
-#
-# --no-site-file	Don't load the site-wide startup stuff
+# --quick              Use minimal customization. This implies --no-init-file,
+#		       --no-site-file and (emacs 24) --no-site-lisp
 #
 # --directory		Ensure that the local elisp sources are found
 #
 # --load		Force loading of notmuch.el and test-lib.el
 
-exec ${TEST_EMACS} --no-init-file --no-site-file \
+exec ${TEST_EMACS} --quick \
 	--directory "$TEST_DIRECTORY/../emacs" --load notmuch.el \
 	--directory "$TEST_DIRECTORY" --load test-lib.el \
 	"\$@"
-- 
2.0.0.rc2

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] test: use --quick when starting emacs.
  2014-05-29  0:26 [PATCH] test: use --quick when starting emacs David Bremner
@ 2014-05-29  0:33 ` David Bremner
  2014-05-29  6:57 ` Tomi Ollila
  2014-06-14  2:09 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: David Bremner @ 2014-05-29  0:33 UTC (permalink / raw)
  To: notmuch

David Bremner <david@tethera.net> writes:

> At least in emacs24, this removes the "site-lisp" directories from the
> load path in addition to enforcing --no-site-lisp --no-init-file.
>

that should say, --no-site-file. fixed in git.

d

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] test: use --quick when starting emacs.
  2014-05-29  0:26 [PATCH] test: use --quick when starting emacs David Bremner
  2014-05-29  0:33 ` David Bremner
@ 2014-05-29  6:57 ` Tomi Ollila
  2014-06-14  2:09 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: Tomi Ollila @ 2014-05-29  6:57 UTC (permalink / raw)
  To: David Bremner, notmuch

On Thu, May 29 2014, David Bremner <david@tethera.net> wrote:

> At least in emacs24, this removes the "site-lisp" directories from the
> load path in addition to enforcing --no-site-lisp --no-init-file.
>
> This works around a slightly mysterious bug on Debian that causes
> test-lib.el not to load when there is cl-lib.el(c) in some site-lisp
> directory.  It should be harmless in general since we really don't
> want to load any files from addon packages to emacs.
> ---

LGTM. works with emacs 23.1.1 and emacs 24.3.1

Funnily, if make test is run w/o setting EMACS run_emacs.sh will have:

./test/tmp.T350-crypto/run_emacs:15:exec emacs --quick --quick --directory ...


Tomi

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] test: use --quick when starting emacs.
  2014-05-29  0:26 [PATCH] test: use --quick when starting emacs David Bremner
  2014-05-29  0:33 ` David Bremner
  2014-05-29  6:57 ` Tomi Ollila
@ 2014-06-14  2:09 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: David Bremner @ 2014-06-14  2:09 UTC (permalink / raw)
  To: notmuch

David Bremner <david@tethera.net> writes:

> At least in emacs24, this removes the "site-lisp" directories from the
> load path in addition to enforcing --no-site-lisp --no-init-file.
>

pushed, 

d

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-06-14  2:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-29  0:26 [PATCH] test: use --quick when starting emacs David Bremner
2014-05-29  0:33 ` David Bremner
2014-05-29  6:57 ` Tomi Ollila
2014-06-14  2:09 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).