77/c23c637615ea16ae87d69165072d1f8de18c89

   1 Return-Path: <david@tethera.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5  by arlo.cworth.org (Postfix) with ESMTP id E85516DE0298\r
   6  for <notmuch@notmuchmail.org>; Mon, 28 Sep 2015 04:02:18 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.107\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.107 tagged_above=-999 required=5 tests=[AWL=0.107]\r
  12  autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id 3Iz4Xi_Rze5l for <notmuch@notmuchmail.org>;\r
  16  Mon, 28 Sep 2015 04:02:16 -0700 (PDT)\r
  17 Received: from gitolite.debian.net (gitolite.debian.net [87.98.215.224])\r
  18  by arlo.cworth.org (Postfix) with ESMTPS id 97E6B6DE0173\r
  19  for <notmuch@notmuchmail.org>; Mon, 28 Sep 2015 04:02:15 -0700 (PDT)\r
  20 Received: from remotemail by gitolite.debian.net with local (Exim 4.80)\r
  21  (envelope-from <david@tethera.net>)\r
  22  id 1ZgWA3-0003ft-Ee; Mon, 28 Sep 2015 10:59:59 +0000\r
  23 Received: (nullmailer pid 12310 invoked by uid 1000); Mon, 28 Sep 2015\r
  24  10:59:40 -0000\r
  25 From: David Bremner <david@tethera.net>\r
  26 To: Tomi Ollila <tomi.ollila@iki.fi>, Jani Nikula <jani@nikula.org>,\r
  27  notmuch@notmuchmail.org\r
  28 Subject: Re: [PATCH v4 5/5] notmuch-emacs-mua: do not create a frame by\r
  29  default with --client\r
  30 In-Reply-To: <m21tepvpm5.fsf@guru.guru-group.fi>\r
  31 References: <cover.1439629384.git.jani@nikula.org>\r
  32  <455026447d3f75b1038ac11cebe734431604be9d.1439629384.git.jani@nikula.org>\r
  33  <m21tepvpm5.fsf@guru.guru-group.fi>\r
  34 User-Agent: Notmuch/0.20.2+93~g33c8777 (http://notmuchmail.org) Emacs/24.5.1\r
  35  (x86_64-pc-linux-gnu)\r
  36 Date: Mon, 28 Sep 2015 07:59:40 -0300\r
  37 Message-ID: <87wpva3j37.fsf@zancas.localnet>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.18\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44  <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Mon, 28 Sep 2015 11:02:19 -0000\r
  53 \r
  54 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  55 \r
  56 \r
  57 > if [ -n "$AUTO_DAEMON" -a -z "$CREATE_FRAME" ]; then\r
  58 >     echo "$0: --auto-daemon is only applicable with --create-frame." >&2\r
  59 >     exit 1       \r
  60 > fi\r
  61 >\r
  62 > without this one may execute ./notmuch-emacs-mua --client --auto-daemon\r
  63 > which yields starting emacs in daemon mode (in this example it is expected\r
  64 > emacs is not running; otherwise --auto-daemon has no use in this example)\r
  65 > -- but no ui to that newly-running emacs is provided. Similar behaviour\r
  66 > can be observed by the following\r
  67 >\r
  68 \r
  69 I think what you propose is fine for a followup patch; note that the\r
  70 scenario you worry about also needs --client to be a problem. Apparently\r
  71 nothing is uncontroversial here, but if auto-daemon only works with\r
  72 create frame, then perhaps the followup would be to have auto-daemon\r
  73 imply create-frame\r
  74 \r
  75 >>  \r
  76 >> +# Kill the terminal/frame if we're creating one.\r
  77 >> +if [ -z "$USE_EMACSCLIENT" -o -n "$CREATE_FRAME" -o -n "$NO_WINDOW" ]; then\r
  78 >> +    ELISP="${ELISP} (setq message-exit-actions (list #'save-buffers-kill-terminal))"\r
  79 >> +fi\r
  80 >\r
  81 > I am not very happy that message-exit-actions was added to $ELISP when\r
  82 > not using emacsclient; when emacs is started its sole (initial) purpose is\r
  83 > to serve mail sending (and not lending a frame in some other emacs) -- in\r
  84 > this case it would be nice to be able to retrieve the sent mail buffer.\r
  85 \r
  86 I'm somewhat less sympathetic here. AIUI, the goal of notmuch-emacs-mua\r
  87 is to provide a drop in tool to replace mutt for sending mail in the\r
  88 shell or from other programs. In this case, I think the most common\r
  89 expectation is to terminate after sending mail (e.g. to return use of\r
  90 the shell to the user, or allow the spawning program to continue).\r
  91 However, it seems like this would be relatively easy to set up some\r
  92 customization in notmuch-message-mode so that it could check if running\r
  93 from the cli, and then either kill-or-not emacs depending on some\r
  94 variable. So I think I'd like to keep the current behaviour as default,\r
  95 and make it customizable at the emacs level.\r
  96 \r