From: Tomi Ollila <tomi.ollila@iki.fi>
To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>,
Notmuch Mail <notmuch@notmuchmail.org>
Subject: Re: [PATCH] cli/help: give a hint about notmuch-emacs-mua
Date: Sun, 03 Dec 2017 15:26:32 +0200 [thread overview]
Message-ID: <m2fu8s7xiv.fsf@guru.guru-group.fi> (raw)
In-Reply-To: <87po7z4zyf.fsf@fifthhorseman.net>
On Thu, Nov 30 2017, Daniel Kahn Gillmor wrote:
> On Thu 2017-11-30 11:08:05 +0200, Tomi Ollila wrote:
>> On Thu, Nov 30 2017, Daniel Kahn Gillmor wrote:
>>
>>> On Thu 2017-10-26 18:27:51 -0400, Daniel Kahn Gillmor wrote:
>>>> "notmuch help" doesn't mention "notmuch-emacs-mua" even though we
>>>> support it through the try_external_command() mechanism.
>>>>
>>>> In addition, "notmuch help emacs-mua" doesn't work, even though we
>>>> ship the appropriate manpage.
>>>>
>>>> This changeset fixes both of these problems.
>>>
>>> Ping! i don't think this patch is complex or objectionable. Can it be
>>> merged?
>>
>> it may (*) look like we don't have #if WITH_EMACS (**) cpp macros defined ???
>>
>> Tomi
>>
>> (*) did not check...
>>
>> (**) this is emacs-specific stull, less generic that other hints (if that
>> matters) ...
>>
>> Sorry to cause confusion...
>
> confusion is right! i don't understand what you're saying, sorry. are
> you suggesting a different change somehow? Want to propose a different
> patch?
My thought was that even though user may compile notmuch without emacs
support (and notmuch-emacs-mua not installed) we would be hinting
`notmuch emacs-mua` command to exist.
Tomi
>
> --dkg
next prev parent reply other threads:[~2017-12-03 13:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-26 21:27 [PATCH] cli/help: give a hint about notmuch-emacs-mua Daniel Kahn Gillmor
2017-11-30 5:55 ` Daniel Kahn Gillmor
2017-11-30 9:08 ` Tomi Ollila
2017-11-30 14:13 ` Daniel Kahn Gillmor
2017-12-03 13:26 ` Tomi Ollila [this message]
2017-12-04 15:44 ` Daniel Kahn Gillmor
2017-12-04 18:44 ` Daniel Kahn Gillmor
2017-12-04 18:43 ` [PATCH v2] " Daniel Kahn Gillmor
2017-12-04 20:56 ` Tomi Ollila
2017-12-07 12:31 ` David Bremner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m2fu8s7xiv.fsf@guru.guru-group.fi \
--to=tomi.ollila@iki.fi \
--cc=dkg@fifthhorseman.net \
--cc=notmuch@notmuchmail.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).