a1/28f3503caeb40cede8d572eaf6cfcb00504197

   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 1AA51429E30\r
   6         for <notmuch@notmuchmail.org>; Sun, 17 Nov 2013 07:28:27 -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 4OmqkeZ4UPlh for <notmuch@notmuchmail.org>;\r
  16         Sun, 17 Nov 2013 07:28:16 -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 B2070429E27\r
  19         for <notmuch@notmuchmail.org>; Sun, 17 Nov 2013 07:28:16 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 784E9100030;\r
  22         Sun, 17 Nov 2013 17:28:09 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: David Bremner <david@tethera.net>, Jani Nikula <jani@nikula.org>,\r
  25         notmuch@notmuchmail.org\r
  26 Subject: Re: [PATCH v2 5/5] compact: provide user more information on\r
  27         after-compaction failures\r
  28 In-Reply-To: <8738mvz2fy.fsf@zancas.localnet>\r
  29 References: <1384362167-12740-1-git-send-email-tomi.ollila@iki.fi>\r
  30         <1384362167-12740-6-git-send-email-tomi.ollila@iki.fi>\r
  31         <871u2jnkai.fsf@nikula.org> <87y54rx8sf.fsf@unb.ca>\r
  32         <m28uwquccd.fsf@guru.guru-group.fi>\r
  33         <8738mvz2fy.fsf@zancas.localnet>\r
  34 User-Agent: Notmuch/0.16+174~g9a06aa2 (http://notmuchmail.org) Emacs/24.3.1\r
  35         (x86_64-unknown-linux-gnu)\r
  36 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  37         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  38         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  39 Date: Sun, 17 Nov 2013 17:28:09 +0200\r
  40 Message-ID: <m2pppzm4ja.fsf@guru.guru-group.fi>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: text/plain\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Sun, 17 Nov 2013 15:28:27 -0000\r
  56 \r
  57 On Sun, Nov 17 2013, David Bremner <david@tethera.net> wrote:\r
  58 \r
  59 > Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  60 >\r
  61 >> The log hook in it's current form is problematic as it doesn't provide\r
  62 >> way to distinguish progress reporting from error reporting.\r
  63 >\r
  64 > Is this _more_ problematic than more output to stderr?\r
  65 >\r
  66 >>  Currently\r
  67 >> lib/database.cc writes error messages with fprintf(stderr, ...) everywhere.\r
  68 >\r
  69 > Sure. But I'm trying to understand why a partial fix isn't better than\r
  70 > nothing.  Is the argument just that the effort is wasted, or that the\r
  71 > result is somehow less satisfactory than the status quo.\r
  72 \r
  73 The partial fix (using current log hook) would mean we either write all\r
  74 messages to stdout or to stderr.\r
  75 \r
  76 To the end user this would mean inconsistent behaviour in compact compared\r
  77 to other commands (like 'notmuch new' which prints progress to stdout and\r
  78 errors to stderr).\r
  79 \r
  80 I personally think doing things this way for 0.17 is tolerable\r
  81 (considering current schedule and risks involved) so that user experience\r
  82 is stable.\r
  83 \r
  84 >> I suggest that this problem is fixed in one big sweep during 0.18\r
  85 >> development -- the suggestion Jani pastebin'd a few days ago is\r
  86 >> a good one and I'm willing to take part of that development...\r
  87 >> And now take this approach of fprintf()ing (basically I would\r
  88 >> also ask developers using the library wait for 0.18 before starting\r
  89 >> to use the compact functionality (if ever), as the we have yet\r
  90 >> another soname bump with changing interface coming...\r
  91 >\r
  92 > I guess we can mark this interface as unstable for the moment?\r
  93 > "Asking developers not to use it" sounds pretty bad.\r
  94 \r
  95 I was thinking naming the function notmuch_database_compact_internal ()\r
  96 as one option (I also though of notmuch_database_compact_unstable () -- but\r
  97 that sounds so "unstable" (at least outside Debian people ;D)) --\r
  98 could be one option. Then developers should understand the API is not\r
  99 fixed there...\r
 100 \r
 101 Although "Informing developers to prepare for upcoming changes to the API"\r
 102 (which is goind to happen early on 0.18 development cycle) should suffice.\r
 103 \r
 104 > d\r
 105 \r
 106 Tomi\r