unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: "John Lenz" <lenz@math.uic.edu>
To: "Tomi Ollila" <tomi.ollila@iki.fi>, notmuch@notmuchmail.org
Subject: Re: cli: add --include-html option to notmuch show
Date: Sat, 17 Aug 2013 10:52:28 -0500	[thread overview]
Message-ID: <notmuch-web-1376754748.336647450@www.wuzzeb.org> (raw)
In-Reply-To: <m2zjsxs0g1.fsf@guru.guru-group.fi>

On Sun Aug  4 14:47 -0500 2013, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> The next question is should we have new option as
>
> --include-html
>
> or as
>
> --include-html=(true|false)
>
> or even
>
> --body=(true|false|text-and-html)
>
> See --exclude option in http://notmuchmail.org/manpages/notmuch-search-1/
> and --body option in http://notmuchmail.org/manpages/notmuch-show-1/
> for comparison...
>

I have no preference here, although I guess I would vote for
--include-html=(true|false) since adding it to --body makes the --body
options confusing: to make sense the body options should be
--body=(text|text-and-html|none) but of course you can't change that and
break the command line API.  Well, maybe you could add all three
--body=(text|text-and-html|none)  and still accept true/false for
compatibility.

  reply	other threads:[~2013-08-17 15:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-02  0:19 cli: add --include-html option to notmuch show John Lenz
2013-07-21 20:23 ` Tomi Ollila
2013-07-22 16:49   ` John Lenz
2013-07-25  2:36   ` John Lenz
2013-08-04 19:47     ` Tomi Ollila
2013-08-17 15:52       ` John Lenz [this message]
2013-08-18 11:25         ` Jani Nikula
2013-08-18 18:30           ` Tomi Ollila
2013-08-24  8:11             ` Mark Walters
2013-08-24 10:59             ` Jani Nikula
2013-08-24 15:29 ` [PATCH 1/1] test: test notmuch show --include-html option Tomi Ollila
2013-08-24 21:36   ` Mark Walters
2013-08-27 11:04   ` David Bremner
2013-08-27 11:03 ` cli: add --include-html option to notmuch show 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=notmuch-web-1376754748.336647450@www.wuzzeb.org \
    --to=lenz@math.uic.edu \
    --cc=notmuch@notmuchmail.org \
    --cc=tomi.ollila@iki.fi \
    /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).