30/f96b2f1d9dab5fd57471c7f4894138b2775cd8

   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 ADF78431FBD\r
   6         for <notmuch@notmuchmail.org>; Sun, 20 Jan 2013 14:55:40 -0800 (PST)\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 JBKbOlVeqzvr for <notmuch@notmuchmail.org>;\r
  16         Sun, 20 Jan 2013 14:55:39 -0800 (PST)\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 0C726431FB6\r
  19         for <notmuch@notmuchmail.org>; Sun, 20 Jan 2013 14:55:39 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 530BA100086;\r
  22         Mon, 21 Jan 2013 00:55:28 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Jameson Graef Rollins <jrollins@finestructure.net>, david@tethera.net,\r
  25         notmuch@notmuchmail.org\r
  26 Subject: Re: [PATCH 5/7] CLI: add --leak-report top level option\r
  27 In-Reply-To: <87d2wzwmd9.fsf@servo.finestructure.net>\r
  28 References: <1358619958-21209-1-git-send-email-david@tethera.net>\r
  29         <1358619958-21209-6-git-send-email-david@tethera.net>\r
  30         <87d2wzwmd9.fsf@servo.finestructure.net>\r
  31 User-Agent: Notmuch/0.15+5~g6f527c2 (http://notmuchmail.org) Emacs/24.2.1\r
  32         (x86_64-unknown-linux-gnu)\r
  33 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  34         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  35         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  36 Date: Mon, 21 Jan 2013 00:55:28 +0200\r
  37 Message-ID: <m2ip6ro467.fsf@guru.guru-group.fi>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 Cc: David Bremner <bremner@debian.org>\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Sun, 20 Jan 2013 22:55:40 -0000\r
  54 \r
  55 On Sun, Jan 20 2013, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  56 \r
  57 > On Sat, Jan 19 2013, david@tethera.net wrote:\r
  58 >> This roughly mimics the samba4 argument. The presence of the command\r
  59 >> line argument overrides any value of NOTMUCH_TALLOC_REPORT in the\r
  60 >> environment.\r
  61 >> ---\r
  62 >>  man/man1/notmuch.1 |    8 ++++++++\r
  63 >>  notmuch.c          |   18 +++++++-----------\r
  64 >>  2 files changed, 15 insertions(+), 11 deletions(-)\r
  65 >>\r
  66 >> diff --git a/man/man1/notmuch.1 b/man/man1/notmuch.1\r
  67 >> index 6bf9b2e..5c58c41 100644\r
  68 >> --- a/man/man1/notmuch.1\r
  69 >> +++ b/man/man1/notmuch.1\r
  70 >> @@ -70,6 +70,14 @@ Print a synopsis of available commands and exit.\r
  71 >>  Print the installed version of notmuch, and exit.\r
  72 >>  .RE\r
  73 >>  \r
  74 >> +.RS 4\r
  75 >> +.TP 4\r
  76 >> +.BI \-\-leak-report= path\r
  77 >> +\r
  78 >> +Write a detailed report of all memory allocated via talloc to\r
  79 >> +.I path\r
  80 >> +.RE\r
  81 >\r
  82 > Do we really need a command line option for this?  Why isn't the env var\r
  83 > sufficient?  This just seems to me like it clutters the interface, for\r
  84 > an option that is purely for debugging and should rarely if ever be used\r
  85 > by most users.\r
  86 \r
  87 Jameson does have a point. Now that we already have that environment\r
  88 variable and it can be used in shipped notmuch 0.15 it is perhaps\r
  89 simplest just to stick with that.\r
  90 \r
  91 My thoughts after brief first visit to the patche series has so far\r
  92 being either make the command line usage 1:1 compatible with samba\r
  93 or use option like --leak-report-output=..., --leak-report-file=... or\r
  94 --leak-report-to=... (and attempt to deprecate the env var...)\r
  95 \r
  96 That said, I withdraw my previous suggestion of the command line option...\r
  97 \r
  98 The other changes in this patch series looks initially good -- and changes\r
  99 that drop deprecated features should be get in as early after last release\r
 100 as possible.\r
 101 \r
 102 > jamie.\r
 103 \r
 104 \r
 105 Tomi\r