Date: Thu, 31 Dec 2015 08:43:12 -0800
From: "W. Trevor King" <wking@tremily.us>
To: Tomi Ollila <tomi.ollila@iki.fi>
Cc: notmuch@notmuchmail.org
Subject: Re: [PATCH 0/2] nmbug-status: h1 title and unbalanced <p> fixups to
 default templates
Message-ID: <20151231164312.GJ3680@odin.tremily.us>
References: <cover.1451509249.git.wking@tremily.us>
 <m21ta2d92t.fsf@guru.guru-group.fi>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
 protocol="application/pgp-signature"; boundary="4vpci17Ql0Nrbul2"
Content-Disposition: inline
In-Reply-To: <m21ta2d92t.fsf@guru.guru-group.fi>
OpenPGP: id=39A2F3FA2AB17E5D8764F388FC29BDCDF15F5BE8;
 url=http://tremily.us/pubkey.txt
User-Agent: Mutt/1.5.23 (2014-03-12)
Precedence: list


--4vpci17Ql0Nrbul2
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Dec 31, 2015 at 03:46:50PM +0200, Tomi Ollila wrote:
> This series LGTM. (html) tidy complains about imo irrelevant things
> -- or I just did not know how to use it correctly -- as `| tidy
> -eq`.

That doesn't complain about anything with the current tidy 5.1.25 [1]
with output built by this branch (although I ran it on output for a
non-notmuch tag-set).

> PS: it was a bit difficult to test, had to resort to reading sources
> and experment with some git commands. So (i.e. Note to self):
>=20
> $ git --git-dir ./.nmbug cat-file -p remotes/origin/config:status-config.=
json > ~/tmp/nmbug...

There's a sample status-config.json in devel/nmbug/, so you don't have
to check with the version in the nmbug repository if you don't want
to.

> $ PYTHONPATH=3D$PWD/bindings/python python2.7 devel/nmbug/nmbug-status --=
config ~/tmp/nmbug...

Yeah, if you don't have the =E2=80=98notmuch=E2=80=99 package installed glo=
bally,
you'll need to do something like this (although nmbug-status should
work with Python 2.6+, including the 3.x line).  The error you get
=66rom not having notmuch installed should be fairly understandable
though:

  $ nmbug-status
  Traceback (most recent call last):
    File "./nmbug-status", line 422, in <module>
      import notmuch
  ImportError: No module named 'notmuch'

So I'd recommend either:

  $ PYTHONPATH=3D$PWD/bindings/python python2.7 devel/nmbug/nmbug-status --=
config devel/nmbug/status-config.json

or installing notmuch with its Python bindings using your package
manager and running:

  $ ./devel/nmbug/nmbug-status --config devel/nmbug/status-config.json

Cheers,
Trevor

[1]: http://www.html-tidy.org/

     =E2=80=9CTidy reborn, Thanks for the efforts of HTACG and prominent
     contributors, HTML Tidy has a whole new heartbeat and a whole new
     life.=E2=80=9D

     Maybe your distro-packaged tidy is just old?  I have Tidy
     20090325 from Gentoo, but 5.1.25 was cut on 2015-11-23 [2].

[2]: https://github.com/htacg/tidy-html5/releases

--=20
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy

--4vpci17Ql0Nrbul2
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBAgAGBQJWhVseAAoJEAPqygegUbGsNxcP/ihsgdYpm7mhduvEOHHgmrSc
01qVLQoPvhN50f1xJ4KKNGqKv481vvaGpRZ7r3anKjt1MUDuWEAayhD1Ycm964pw
tvjqZtzFuLtUUAxMHMkyXC7c/Yq4MgSAZHS6QOeGUMCPWbJPTMcBoc74frcFbFbL
b3Zo90N4rXw8R9voUqQZ25kBBIJGnPqWTr01KH8ZtzJtL95S273VPF5P6QtQet82
nW/UT/xkKsAAwPyuUfsLQxmZDzKsAqv/5bnUf6ZxND3C4pCOs5XkbKGeZgouj3OB
4doYShB3v7VAFSlFITnAh4xIMYL+bn9mviyErE8GwnMz3F62ZnC3Uf/qj7CGJvAy
O9U+inQKQtheUp2w4eCV5mmdl4aKJQIdhqy/oJ4rRVGLpYpyPVl1Q9YgsKXnR32s
o0fxzLExPy1F2jz+XM3DyhVjDZZij88BkC3YIMWgZvkiujWy7Le3+7efhSW+FHfZ
Ld6VyxbyrGFgD1872oXdmiJ59G61AuuL541sNcvgBhITGWDIsN+lrRONGjNmKCf9
gFQCCV/cEJ6U77xqAN/e2in1daZhWjVLAf7gjjUjnPsuNkod2JxWwAGMngfcroLM
K0tnLn94up+jzLxrjg4jZfOavnePsMlixrw5g3Uwf49fUwT//6pyB0zYhmb1k14A
d2SHy8tQcFdCAVLJTWU4
=W6Aa
-----END PGP SIGNATURE-----

--4vpci17Ql0Nrbul2--