From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id D51FA6DE01D8 for ; Sun, 28 Oct 2018 05:56:43 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: 0.003 X-Spam-Level: X-Spam-Status: No, score=0.003 tagged_above=-999 required=5 tests=[AWL=0.004, SPF_PASS=-0.001] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6znfPoFVWjRU for ; Sun, 28 Oct 2018 05:56:42 -0700 (PDT) Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197]) by arlo.cworth.org (Postfix) with ESMTPS id BCA136DE00EF for ; Sun, 28 Oct 2018 05:56:42 -0700 (PDT) Received: from remotemail by fethera.tethera.net with local (Exim 4.89) (envelope-from ) id 1gGkcS-0004j9-9O; Sun, 28 Oct 2018 08:56:40 -0400 Received: (nullmailer pid 14962 invoked by uid 1000); Sun, 28 Oct 2018 12:56:39 -0000 From: David Bremner To: mawww@kakoune.org, notmuch@notmuchmail.org Subject: Re: [PATCH] cli: Support --include-html and --body=false for --format=text In-Reply-To: <648ba1a6be0f1653e078245a235cffc5@kakoune.org> References: <31d73dc7587ea4ee45c176e345f7cb3d@kakoune.org> <87r2gd7t54.fsf@nikula.org> <648ba1a6be0f1653e078245a235cffc5@kakoune.org> Date: Sun, 28 Oct 2018 09:56:39 -0300 Message-ID: <87lg6iqlfc.fsf@tethera.net> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Oct 2018 12:56:43 -0000 mawww@kakoune.org writes: > On 2018-10-26 05:55, Jani Nikula wrote: >> It's debatable whether parsing the --format=text output correctly is >> convenient or not. Particularly for notmuch show I think the text >> format >> output is basically legacy, and I don't think we're all that fond of >> adding new features to it. > > I would argue that it is easier to work with with shell tools than > sexpression or json, but even if it was not, this change is pretty > trivial > and does not add a brand new feature to the text format, it just makes > it possible to use existing features (include-html and body=false) with > it. > >> In any case, this patch has two independent changes in one, and should >> be split. > > Following are the splitted commits. > Please re-send using git-send-email when you have a chance. d