66/cd0e05979e5f8ccbe84770d77945f65bd1b753

   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 B5228431FBC\r
   6         for <notmuch@notmuchmail.org>; Fri, 25 May 2012 13:53:13 -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 NETJrnVIqgml for <notmuch@notmuchmail.org>;\r
  16         Fri, 25 May 2012 13:53:10 -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 C5DB2431FB6\r
  19         for <notmuch@notmuchmail.org>; Fri, 25 May 2012 13:53:10 -0700 (PDT)\r
  20 Received: by guru.guru-group.fi (Postfix, from userid 501)\r
  21         id 387BE100641; Fri, 25 May 2012 23:53:20 +0300 (EEST)\r
  22 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  23 To: Mike Kelly <pioto@pioto.org>, notmuch@notmuchmail.org\r
  24 Subject: Re: FreeBSD Support Patches\r
  25 In-Reply-To: <1337953411-21000-1-git-send-email-pioto@pioto.org>\r
  26 References: <1337953411-21000-1-git-send-email-pioto@pioto.org>\r
  27 User-Agent: Notmuch/0.13+38~g944a859 (http://notmuchmail.org) Emacs/23.1.1\r
  28         (x86_64-redhat-linux-gnu)\r
  29 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  30         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  31         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  32 Date: Fri, 25 May 2012 23:53:20 +0300\r
  33 Message-ID: <m2k400geyn.fsf@guru.guru-group.fi>\r
  34 MIME-Version: 1.0\r
  35 Content-Type: text/plain; charset=us-ascii\r
  36 X-BeenThere: notmuch@notmuchmail.org\r
  37 X-Mailman-Version: 2.1.13\r
  38 Precedence: list\r
  39 List-Id: "Use and development of the notmuch mail system."\r
  40         <notmuch.notmuchmail.org>\r
  41 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  43 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  44 List-Post: <mailto:notmuch@notmuchmail.org>\r
  45 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  46 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  48 X-List-Received-Date: Fri, 25 May 2012 20:53:13 -0000\r
  49 \r
  50 On Fri, May 25 2012, Mike Kelly <pioto@pioto.org> wrote:\r
  51 \r
  52 > Hi,\r
  53 >\r
  54 > These patches add some changes necessary for a clean build on FreeBSD,\r
  55 > and for most of the tests to pass.\r
  56 >\r
  57 > Also mixed in are a few patches to disable emacs tests when you don't\r
  58 > have emacs, along with tests that depend upon emacs. I could split those\r
  59 > off onto a separate branch if required.\r
  60 \r
  61 Yes...\r
  62 \r
  63 Thank you for your efforts to make notmuch better; some general\r
  64 comments on this patch series:\r
  65 \r
  66 \r
  67 Commit messages are somewhat terse\r
  68 check what http://notmuchmail.org/patchformatting/\r
  69 has to say about this.\r
  70 \r
  71 MANPATH setting has hit me too -- but that shold be set\r
  72 in the same place where PATH is adjusted to find notmuch\r
  73 command instead of hardcoding '../../man'.\r
  74 \r
  75 s/executable/+111/ (and following sed (which could be combined\r
  76 into next sed but that's another matter)) is good (that hit me\r
  77 once too) -- This particular patch has the worst commit message,\r
  78 though (Make test/basic more portable) ;/\r
  79 \r
  80 in 6/10 there is missing space before sed in '...wc -l |sed...'.\r
  81 \r
  82 -p overrides -n in perl, so perl -pe... (-npe confuses, at least\r
  83 me...)\r
  84 \r
  85 The patch series is supposed to add FreeBSD support; \r
  86 prereq additions to take care missing emacs should\r
  87 be in separate series.\r
  88 \r
  89 Some of the patches could be marked as 'trivial', putting those\r
  90 to 'maybe ready' section in http://nmbug.tethera.net/status/\r
  91 -- just if those had more comprehensive commit messages.\r
  92 \r
  93 > You can find the latest version of these patches on my github repo's\r
  94 > fbsd-support branch:\r
  95 >\r
  96 >   https://github.com/pioto/notmuch/compare/master...fbsd-support \r
  97 \r
  98 Tomi\r