From: Jameson Graef Rollins <jrollins@finestructure.net>
To: David Bremner <david@tethera.net>, Notmuch Mail <notmuch@notmuchmail.org>
Subject: Re: have "notmuch help" call man?
In-Reply-To: <87ehvxtgoi.fsf@zancas.localnet>
References: <87zkescxp5.fsf@zancas.localnet> <87iplbyj0w.fsf@zancas.localnet>
	<87ehvxtgoi.fsf@zancas.localnet>
User-Agent: Notmuch/0.10.2+93~ga124635 (http://notmuchmail.org) Emacs/23.3.1
	(x86_64-pc-linux-gnu)
Date: Wed, 21 Dec 2011 20:53:08 -0800
Message-ID: <87k45pqkaz.fsf@servo.finestructure.net>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha256; protocol="application/pgp-signature"
Precedence: list

--=-=-=
Content-Transfer-Encoding: quoted-printable

On Wed, 21 Dec 2011 23:43:09 -0400, David Bremner <david@tethera.net> wrote:
> The man pages are now installed, a notmuch help (built from this branch)
> uses them.

Hey, David this is really great.  Thank you so much for working on this.
This will make things much easier to maintain down the line.

I think what you've done looks great.  A couple small issues:

* There are a couple of formatting issues (notmuch.1, notmuch-config.1,
  notmuch-reply.1).

* The notmuch part command is completely deprecated, so I don't think
  there's really any reason to include a man page for it.

* There's no man page for restore (notmuch-restore.1).

It also occurs to me while looking through this: do we really need a
separate "setup" command?  Does that ever really get run?  Doesn't
notmuch new run the setup if it doesn't detect a config?  If there's
really reason to run it again at a later point, why don't we just have a
=2D-setup option to notmuch new instead?  It would also make notmuch.1
cleaner if we removed it.

> I'm not sure if posting the patches make sense; it's up to 74k of diffs,
> even using "git format-patch -D".  I'm open to suggestions, as always.

While in general I think it's nice to have patches to the list, but I'm
not sure how much benefit there is in this case.  I would be fine to
just see the patch merged directly from your branch.

Again, thanks for the work on this, David.  This=20

jamie.

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iQIcBAEBCAAGBQJO8re0AAoJEO00zqvie6q8f8AP/RZOt/9QeYweAn6hKWAG/zy4
hjHtZ7nkeCO77CquiexUlIv7oLvrSDgVMes1RhiLzaOFDBPo8YosP61/3tEhffb3
mp0GWDdDBGkhZ/6DQMoK+SPNwNkBXkrmao03ZzU1Ah3wi51jVP7HaTrxzzqtzkpB
j6aEqY/Lcgy7W491lz1wYl9tpMxD6fZqyNK0Ph1+6Zfgk2+8hs0rS8Bmj1KVPnu8
vxIG5PQ0fQQoW5BFs/dCe0YcWyCxjfyQ9vfO08b4pLiASZsBiM09P+TZaFF3iLZX
1RMlzsVEVgoNI5E1ZPOeHLbu4HaXlIa+5LEWeYLL3aRGHWBFxa38tjsbETy5NR12
nABZQmLJnYqUGd+wBRxzbJQgoOxxmMd1lqqrSPWfTBD5LyyOLcpqlzdYjx6U1A+3
qFKMAH8yFWD/8RYBTh3IVTSPXmDfKefCF+p5eTU/48pIT8P7eX1bk2jR/NO70pPz
4cCvcGLqwZbBLq0pMEK1Uvh+4NAJVnUnPpsVBfmN08fueA7iD8Uqt/zNZI116X6d
YG5RFXOFLyFoGaEU15n7FDXOYK4LhSO/sg3rXCgGTQ1Pz7oN5BxoPIuvPsG5tHkS
Y+nMgOLXSvvX8qCDJSGMmU07rrKwhC6T/OPRr0xSKb3Yu35v5Sor9HamKjM7XZOd
Tlkj1UV/7MTR0lwVd6Uz
=fjJC
-----END PGP SIGNATURE-----
--=-=-=--