Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id B05EF431FBC for ; Wed, 12 Dec 2012 17:46:20 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -0.7 X-Spam-Level: X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5 tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled Received: from olra.theworths.org ([127.0.0.1]) by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jyNdEfAh-VDj for ; Wed, 12 Dec 2012 17:46:20 -0800 (PST) Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU [18.9.25.12]) by olra.theworths.org (Postfix) with ESMTP id 1BFE5431FB6 for ; Wed, 12 Dec 2012 17:46:20 -0800 (PST) X-AuditID: 1209190c-b7f886d000000936-fa-50c9336b7d6a Received: from mailhub-auth-1.mit.edu ( [18.9.21.35]) by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP id 45.33.02358.B6339C05; Wed, 12 Dec 2012 20:46:19 -0500 (EST) Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103]) by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id qBD1kIjF023868; Wed, 12 Dec 2012 20:46:19 -0500 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91]) (authenticated bits=0) (User authenticated as amdragon@ATHENA.MIT.EDU) by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id qBD1kHxo023174 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Wed, 12 Dec 2012 20:46:18 -0500 (EST) Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.80) (envelope-from ) id 1TixsP-0000JG-55; Wed, 12 Dec 2012 20:46:17 -0500 Date: Wed, 12 Dec 2012 20:46:17 -0500 From: Austin Clements To: Mark Walters Subject: Re: [PATCH 00/10] CLI output versioning Message-ID: <20121213014617.GD6187@mit.edu> References: <1354416002-3557-1-git-send-email-amdragon@mit.edu> <87pq2sx86o.fsf@awakening.csail.mit.edu> <878v98hoyp.fsf@qmul.ac.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <878v98hoyp.fsf@qmul.ac.uk> User-Agent: Mutt/1.5.21 (2010-09-15) X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpkleLIzCtJLcpLzFFi42IR4hRV1s02PhlgMPWkpsXquTwW12/OZHZg 8tg56y67x7NVt5gDmKK4bFJSczLLUov07RK4Mo4f+MJc8F+sYvrfmgbG54JdjJwcEgImEpuf NTND2GISF+6tZ+ti5OIQEtjHKLHh10lWCGcDo8SFG/uZIZyLTBLrO8+yg7QICSxhlJjTBTSK g4NFQFXiWIcGSJhNQENi2/7ljCC2iICOxO1DC8DKmQWkJb79bmYCsYUFDCQW/lgGVsMroC1x /NEHqM0djBIfLy9lhkgISpyc+YQFollL4sa/l0wgu0AGLf/HARLmBNp1te8L2HxRARWJKSe3 sU1gFJqFpHsWku5ZCN0LGJlXMcqm5Fbp5iZm5hSnJusWJyfm5aUW6Rrq5WaW6KWmlG5iBIe0 JM8OxjcHlQ4xCnAwKvHwXnh5PECINbGsuDL3EKMkB5OSKK+q0ckAIb6k/JTKjMTijPii0pzU 4kOMEhzMSiK8ZbpAOd6UxMqq1KJ8mJQ0B4uSOO/llJv+QgLpiSWp2ampBalFMFkZDg4lCd7b hkCNgkWp6akVaZk5JQhpJg5OkOE8QMNfgtTwFhck5hZnpkPkTzEqSonz7gdJCIAkMkrz4Hph KecVozjQK8K8T0CqeIDpCq77FdBgJqDBcZeOgwwuSURISTUwTg6L+e3H8efg/fBCkYiMQmNO je6T3JPmhXwqXx+o+yBt8mI55ceO5nwSeRqP9ETXTLi5IPzT3rxrB29NnhYT8HZP3J7In5tc YlxuhmwQlsrcxRo1MWhp5m3Zr4mRFp1/LPcFG2106HadsSXFjfHk5mnb7199UfDI5vg/4xL+ k+WLmL2VvTJKlViKMxINtZiLihMBzK1wsxQDAAA= Cc: notmuch@notmuchmail.org X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.13 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: Thu, 13 Dec 2012 01:46:20 -0000 Quoth Mark Walters on Dec 08 at 9:29 am: > > Hi > > Overall this series looks good. As we discussed on irc I think i would > prefer global NOTMUCH_SCHEMA_MIN as I am a little worried about these > proliferating (eg if someone decides text output also needs versioning > etc) In addition, if we do find the distinction useful it would be easy > to add at a later date. Will do. > One tiny comment on the manpage updates: now that you mention two return > values explicitly should the other possibilities be mentioned too or are > they so obvious it is not needed? I thought about this, but couldn't figure out what to say for exit status 1 other than "Unspecified error". Hence I carefully worded the man page to say *special* exit status codes. > Would it be worth having some emacs test for the error handling? (eg set > notmuch-command to something giving some stderr and an error) Inherently > these code paths won't be tested much so I think tests could be > particularly useful. Good idea. > Best wishes > > Mark > > > > > > > On Mon, 03 Dec 2012, Austin Clements wrote: > > (Sorry; I forgot to include a cover letter.) > > > > This series is intended to help with our long-standing output format > > versioning issue. While the JSON format is amenable to extension, > > there's still a high barrier to extensions because of the need to > > support them going forward, and an even higher barrier to modifications > > that break backwards compatibility. Versioning will make the format > > more dynamic, enabling us to easily improve and iterate on it. It will > > also address the slew of confusing bugs that people encounter when they > > use a mismatched CLI and front-end. > > > > On IRC we've talking about adding version information to the output > > format itself. This series takes a different and, I think, better > > approach: callers request a specific output format version on the > > command line. This allows notmuch to remain backwards compatible with > > older format versions when it's easy or necessary. This also doesn't > > require shoehorning a version number into the output, which would be > > awkward for both the CLI and the consumer. > > > > I called the argument --use-schema, but I'm open to other suggestions. > > --use-schema is technically accurate, but perhaps not as self-describing > > as something like --schema-version or --format-version (to parallel > > --format). > > _______________________________________________ > > notmuch mailing list > > notmuch@notmuchmail.org > > http://notmuchmail.org/mailman/listinfo/notmuch -- Austin Clements MIT/'06/PhD/CSAIL amdragon@mit.edu http://web.mit.edu/amdragon Somewhere in the dream we call reality you will find me, searching for the reality we call dreams.