9a/0720de70f3dd37595fd5a7aa960d524330fb68

   1 Return-Path: <tomi.ollila@iki.fi>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id AE31E431FCB\r
   6         for <notmuch@notmuchmail.org>; Sat, 12 Jul 2014 23:05:53 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id lA8bfTjAyQAT for <notmuch@notmuchmail.org>;\r
  16         Sat, 12 Jul 2014 23:05:46 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 5200A431FBD\r
  19         for <notmuch@notmuchmail.org>; Sat, 12 Jul 2014 23:05:46 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id F05E71000B3;\r
  22         Sun, 13 Jul 2014 09:05:41 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: "W. Trevor King" <wking@tremily.us>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH v3 0/5] rst2man.py support and doc-build cleanups\r
  26 In-Reply-To: <cover.1405220724.git.wking@tremily.us>\r
  27 References: <cover.1405220724.git.wking@tremily.us>\r
  28 User-Agent: Notmuch/0.18.1+25~gdaf4b6f (http://notmuchmail.org) Emacs/24.3.1\r
  29         (x86_64-unknown-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Sun, 13 Jul 2014 09:05:41 +0300\r
  34 Message-ID: <m2wqbhiyiy.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=utf-8\r
  37 Content-Transfer-Encoding: quoted-printable\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sun, 13 Jul 2014 06:05:53 -0000\r
  51 \r
  52 On Sun, Jul 13 2014, "W. Trevor King" <wking@tremily.us> wrote:\r
  53 \r
  54 > Changes since v2 [1]:\r
  55 >\r
  56 > * In patches 1/5 and 5/5, use for loops to check for rst2man[.py] and\r
  57 >   rst2html[.py].\r
  58 > * In patches 1/5 and 5/5, store the command names, not the full paths\r
  59 >   (command -v =E2=80=A6) [2].\r
  60 > * In patch 3/5, I've added argparse handling to the newly-renamed\r
  61 >   rst-man2any.py.  Tomi suggested modeling the name and UI on texi2any\r
  62 >   [3], but texi2any is for converting a single file, while\r
  63 >   rst-man2any.py is for converting a whole tree (and it takes an\r
  64 >   arbitrary ReST-to-$x converter to do the real work).  I've basically\r
  65 >   just cleaned things up, but if folks aren't satisfied I'm going to\r
  66 >   need more concrete suggestions ;).\r
  67 \r
  68 The series now looks good to me, I don't have time to test it now but will\r
  69 do it after Jul 20th.\r
  70 \r
  71 I am satisfied with rst-man2any.py, but as being normal picky me I wonder\r
  72 whether the command prefix 'rst-' is being too generic i.e. is invading\r
  73 that "namespace". If no one else has the same feeling (or the feeling is\r
  74 just wrong (or insignificant)) then this can be forgotten :D\r
  75 \r
  76 >\r
  77 > Cheers,\r
  78 > Trevor\r
  79 \r
  80 Thanks, good work!\r
  81 \r
  82 \r
  83 Tomi\r
  84 \r
  85 >\r
  86 > [1]: id:cover.1399740604.git.wking@tremily.us\r
  87 >      http://thread.gmane.org/gmane.mail.notmuch.general/18291\r
  88 > [2]: id:m2ion3dn0r.fsf@guru.guru-group.fi\r
  89 >      http://article.gmane.org/gmane.mail.notmuch.general/18652\r
  90 > [3]: id:m2lhrzj8kb.fsf@guru.guru-group.fi\r
  91 >      http://article.gmane.org/gmane.mail.notmuch.general/18653\r
  92 >\r
  93 > W. Trevor King (5):\r
  94 >   doc: Allow rst2man.py as an alternative to rst2man\r
  95 >   doc/prerst2man.py: Convert execfile to import\r
  96 >   doc/rst-man2any.py: Adjust to handle any output format, not just man\r
  97 >     pages\r
  98 >   doc: Consolidate Makefile targets around {build|install}-{format}\r
  99 >   doc: Add rst2html support for building HTML docs\r
 100 >\r
 101 >  Makefile.local     |  14 ++++++-\r
 102 >  configure          |  45 ++++++++++++++++++++---\r
 103 >  doc/INSTALL        |  47 +++++++++++++++++-------\r
 104 >  doc/Makefile.local |  48 ++++++++++++++----------\r
 105 >  doc/prerst2man.py  |  63 --------------------------------\r
 106 >  doc/rst-man2any.py | 105 +++++++++++++++++++++++++++++++++++++++++++++++=\r
 107 ++++++\r
 108 >  6 files changed, 217 insertions(+), 105 deletions(-)\r
 109 >  delete mode 100644 doc/prerst2man.py\r
 110 >  create mode 100755 doc/rst-man2any.py\r
 111 >\r
 112 > --=20\r
 113 > 1.9.1.353.gc66d89d\r
 114 >\r
 115 > _______________________________________________\r
 116 > notmuch mailing list\r
 117 > notmuch@notmuchmail.org\r
 118 > http://notmuchmail.org/mailman/listinfo/notmuch\r