unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH v6] devel/emacs: add devel/try-emacs-mua
@ 2016-01-06 19:04 Tomi Ollila
  2016-01-06 21:48 ` Michal Sojka
  2016-01-13 14:42 ` David Bremner
  0 siblings, 2 replies; 3+ messages in thread
From: Tomi Ollila @ 2016-01-06 19:04 UTC (permalink / raw)
  To: notmuch, sojkam1; +Cc: tomi.ollila

devel/try-emacs-mua provides an easy way to try and experiment
with the notmuch emacs client distributed in emacs subdirectory of
the notmuch source tree.

try-emacs-mua starts a new emacs process and if initial checks pass
*scratch* buffer is filled with information of how to begin.

Normal emacs command line arguments can be used, like -q or -Q.
These arguments are appended verbatim to the starting emacs process.

If the emacs version in use is smaller than 24.4, special care is taken
to ensure that notmuch*.elc files older than corresponding .el files
are not loaded. Since emacs 24.4, setting `load-prefer-newer' variable
takes care of this.
---

less talk, more diff, since v5
( id:1451946112-23573-1-git-send-email-tomi.ollila@iki.fi )

(line numbers missing, as I did this by diffing patches...)

= thinko

/// -;; Try the notmuch emacs client located in ../notmuch/emacs directory
*** +;; Try the notmuch emacs client located in ../emacs/ directory

= the change

/// -(setq initial-buffer-choice t) ;; *scratch* buffer
*** +(setq initial-buffer-choice nil
*** +      inhibit-startup-screen t)

= combined setq (verified with (setq a 1 b a) ... => b = 1)

/// -  (setq try-notmuch-source-directory (directory-file-name pdir))
/// -  (setq try-notmuch-emacs-directory (concat pdir "emacs/"))
/// -  (setq load-path (cons try-notmuch-emacs-directory load-path)))
*** +  (setq try-notmuch-source-directory (directory-file-name pdir)
*** +        try-notmuch-emacs-directory (concat pdir "emacs/")
*** +        load-path (cons try-notmuch-emacs-directory load-path)))

= how did I miss this change in v5?

/// -  (insert "Notmuch cli executable "
*** +  (insert "Notmuch CLI executable "

= clearer (it is effectively the same)

/// -       (set-buffer "*scratch*")
/// -       (lisp-interaction-mode)
/// -       (goto-char (point-min))
/// -       (forward-line 2)
/// -       (set-buffer-modified-p nil)))
*** +       (with-current-buffer "*scratch*"
*** +         (lisp-interaction-mode)
*** +         (goto-char (point-min))
*** +         (forward-line 2)
*** +         (set-buffer-modified-p nil))))


 devel/try-emacs-mua | 157 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 157 insertions(+)
 create mode 100755 devel/try-emacs-mua

diff --git a/devel/try-emacs-mua b/devel/try-emacs-mua
new file mode 100755
index 000000000000..b0a62c25b28f
--- /dev/null
+++ b/devel/try-emacs-mua
@@ -0,0 +1,157 @@
+#!/bin/sh
+:; set -x; exec "${EMACS:-emacs}" --debug-init --load "$0" "$@"; exit
+;;
+;; Try the notmuch emacs client located in ../emacs/ directory
+;;
+;; Run this without arguments; emacs window opens with some usage information
+;;
+;; Authors: Tomi Ollila <tomi.ollila@iki.fi>
+;;
+;; http://www.emacswiki.org/emacs/EmacsScripts was a useful starting point...
+;;
+;; Licence: GPLv3+
+;;
+
+(message "Starting '%s'" load-file-name)
+
+(set-buffer "*scratch*")
+
+(setq initial-buffer-choice nil
+      inhibit-startup-screen t)
+
+(when (featurep 'notmuch)
+  (insert "
+Notmuch has been loaded to this emacs (during processing of the init file)
+which means it is (most probably) loaded from different source than expected.
+
+Please run \"" (file-name-nondirectory load-file-name)
+"\" with '-q' (or '-Q') as an argument, to disable
+processing of the init file -- you can load it after emacs has started\n
+exit emacs (y or n)? ")
+  (if (y-or-n-p "exit emacs")
+      (kill-emacs)
+    (error "Stopped reading %s" load-file-name)))
+
+(let ((pdir (file-name-directory
+	     (directory-file-name (file-name-directory load-file-name)))))
+  (unless (file-exists-p (concat pdir "emacs/notmuch-lib.el"))
+    (insert "Cannot find notmuch-emacs source directory
+while looking at: " pdir "emacs\n\nexit emacs (y or n)? ")
+    (if (y-or-n-p "exit emacs")
+	(kill-emacs)
+      (error "Stopped reading %s" load-file-name)))
+  (setq try-notmuch-source-directory (directory-file-name pdir)
+	try-notmuch-emacs-directory (concat pdir "emacs/")
+	load-path (cons try-notmuch-emacs-directory load-path)))
+
+;; they say advice doesn't work for primitives (functions from c source)
+;; well, these 'before' advice works for emacs 23.1 - 24.5 (at least)
+;; ...and for our purposes 24.3 is enough (there is no load-prefer-newer there)
+;; note also that the old, "obsolete" defadvice mechanism was used, but that
+;; is the only one available for emacs 23 and 24 up to 24.3.
+
+(if (boundp 'load-prefer-newer)
+    (defadvice require (before before-require activate)
+      (unless (featurep feature)
+	(message "require: %s" feature)))
+  ;; else: special require "short-circuit"; after load feature is provided...
+  ;; ... in notmuch sources we always use require and there are no loops
+  (defadvice require (before before-require activate)
+    (unless (featurep feature)
+      (message "require: %s" feature)
+      (let ((name (symbol-name feature)))
+	(if (and (string-match "^notmuch" name)
+		 (file-newer-than-file-p
+		  (concat try-notmuch-emacs-directory name ".el")
+		  (concat try-notmuch-emacs-directory name ".elc")))
+	    (load (concat try-notmuch-emacs-directory name ".el") nil nil t t)
+	  )))))
+
+(insert "Found notmuch emacs client in " try-notmuch-emacs-directory "\n")
+
+(let ((notmuch-path (executable-find "notmuch")))
+  (insert "Notmuch CLI executable "
+	  (if notmuch-path (concat "is " notmuch-path) "not found!") "\n"))
+
+(condition-case err
+;; "opportunistic" load-prefer-newer -- will be effective since emacs 24.4
+    (let ((load-prefer-newer t)
+	  (force-load-messages t))
+      (require 'notmuch))
+  ;; specifying `debug' here lets the debugger run
+  ;; if `debug-on-error' is non-nil.
+  ((debug error)
+   (let ((error-message-string (error-message-string err)))
+     (insert "\nLoading notmuch failed: " error-message-string "\n")
+     (message "Loading notmuch failed: %s" error-message-string)
+     (insert "See *Messages* buffer for more information.\n")
+     (if init-file-user
+	 (message "Hint: %s -q (or -Q) may help" load-file-name))
+     (pop-to-buffer "*Messages*")
+     (error "Stopped reading %s" load-file-name))))
+
+(insert "
+Go to the end of the following lines and type C-x C-e to evaluate
+(or C-j which is shorter but inserts evaluation results into buffer)
+
+To \"disable\" mail sending, evaluate
+* (setq message-send-mail-function (lambda () t))
+")
+
+(if (file-exists-p (concat try-notmuch-source-directory "/notmuch"))
+    (insert "
+To use accompanied notmuch binary from the same source, evaluate
+* (setq exec-path (cons \"" try-notmuch-source-directory  "\" exec-path))
+Note: Evaluating the above may be followed by unintended database
+upgrade and getting back to old version may require dump & restore.
+"))
+
+(if init-file-user ;; nil, if '-q' or '-Q' is given, but no '-u' 'USER'
+    (insert "
+Your init file was processed during emacs startup. If you want to test
+notmuch emacs mail client without your emacs init file interfering, Run\n\""
+(file-name-nondirectory load-file-name) "\" with '-q' (or '-Q') as an argument.
+")
+  (let ((emacs-init-file-name) (notmuch-init-file-name))
+    ;; determining init file name in startup.el/command-line is too complicated
+    ;; to be duplicated here; these 3 file names covers most of the users
+    (mapc (lambda (fn) (if (file-exists-p fn) (setq emacs-init-file-name fn)))
+	  '("~/.emacs.d/init.el" "~/.emacs" "~/.emacs.el"))
+    (setq notmuch-init-file-name "~/.emacs.d/notmuch-config.el")
+    (unless (file-exists-p notmuch-init-file-name)
+	(setq notmuch-init-file-name nil))
+    (if (and emacs-init-file-name notmuch-init-file-name)
+	(insert "
+If you want to load your initialization files now, evaluate\n* (progn")
+      (if (or emacs-init-file-name notmuch-init-file-name)
+	  (insert "
+If you want to load your initialization file now, evaluate\n*")))
+    (if emacs-init-file-name
+	(insert " (load \"" emacs-init-file-name "\")"))
+    (if notmuch-init-file-name
+	(insert " (load \"" notmuch-init-file-name "\")"))
+    (if (and emacs-init-file-name notmuch-init-file-name)
+	(insert ")"))
+    (if (or emacs-init-file-name notmuch-init-file-name)
+	(insert "\n")))
+  (if (>= emacs-major-version 24)
+      (insert "
+If you want to use packages (e.g. company from elpa) evaluate
+* (progn (require 'package) (package-initialize))
+")))
+
+(insert "
+To start notmuch (hello) screen, evaluate
+* (notmuch-hello)")
+
+(add-hook 'emacs-startup-hook
+	  (lambda ()
+	    (with-current-buffer "*scratch*"
+	      (lisp-interaction-mode)
+	      (goto-char (point-min))
+	      (forward-line 2)
+	      (set-buffer-modified-p nil))))
+
+;; Local Variables:
+;; mode: emacs-lisp
+;; End:
-- 
2.6.4

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

* Re: [PATCH v6] devel/emacs: add devel/try-emacs-mua
  2016-01-06 19:04 [PATCH v6] devel/emacs: add devel/try-emacs-mua Tomi Ollila
@ 2016-01-06 21:48 ` Michal Sojka
  2016-01-13 14:42 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: Michal Sojka @ 2016-01-06 21:48 UTC (permalink / raw)
  To: Tomi Ollila, notmuch; +Cc: tomi.ollila

On Wed, Jan 06 2016, Tomi Ollila wrote:
> devel/try-emacs-mua provides an easy way to try and experiment
> with the notmuch emacs client distributed in emacs subdirectory of
> the notmuch source tree.

LGTM. I tested it with various use cases I have it my mind and it works
as I expect.

-Michal

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

* Re: [PATCH v6] devel/emacs: add devel/try-emacs-mua
  2016-01-06 19:04 [PATCH v6] devel/emacs: add devel/try-emacs-mua Tomi Ollila
  2016-01-06 21:48 ` Michal Sojka
@ 2016-01-13 14:42 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: David Bremner @ 2016-01-13 14:42 UTC (permalink / raw)
  To: Tomi Ollila, notmuch, sojkam1

Tomi Ollila <tomi.ollila@iki.fi> writes:

> devel/try-emacs-mua provides an easy way to try and experiment
> with the notmuch emacs client distributed in emacs subdirectory of
> the notmuch source tree.
>

pushed, thanks to you both for revisions.

d

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

end of thread, other threads:[~2016-01-13 14:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-06 19:04 [PATCH v6] devel/emacs: add devel/try-emacs-mua Tomi Ollila
2016-01-06 21:48 ` Michal Sojka
2016-01-13 14:42 ` 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).