e1/caf85553232c02e1af8c8e46f766fb20287c01

   1 Return-Path: <Sebastian@SSpaeth.de>\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 F25FC431FD0\r
   6         for <notmuch@notmuchmail.org>; Thu, 29 Sep 2011 00:45:44 -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.09\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.09 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         RCVD_IN_DNSWL_NONE=-0.0001, T_MIME_NO_TEXT=0.01] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id hyXmDRIBxpcr for <notmuch@notmuchmail.org>;\r
  17         Thu, 29 Sep 2011 00:45:44 -0700 (PDT)\r
  18 Received: from homiemail-a18.g.dreamhost.com (caiajhbdcahe.dreamhost.com\r
  19         [208.97.132.74])\r
  20         by olra.theworths.org (Postfix) with ESMTP id 410AC431FB6\r
  21         for <notmuch@notmuchmail.org>; Thu, 29 Sep 2011 00:45:44 -0700 (PDT)\r
  22 Received: from homiemail-a18.g.dreamhost.com (localhost [127.0.0.1])\r
  23         by homiemail-a18.g.dreamhost.com (Postfix) with ESMTP id 807E9250079;\r
  24         Thu, 29 Sep 2011 00:45:43 -0700 (PDT)\r
  25 DomainKey-Signature: a=rsa-sha1; c=nofws; d=SSpaeth.de; h=from:to:cc:subject\r
  26         :in-reply-to:references:date:message-id:mime-version:\r
  27         content-type; q=dns; s=sspaeth.de; b=avYIMh0PiipaeVmtvUEN4YBbzdh\r
  28         GqtvfqYuAUWwkoXc/w/fBJrgrS8cG0YwbFsdxORGvxE0pGvqzN64PlHnMn8nj0Rg\r
  29         j+atBGMTYEYvr9G4UBaXIkA93YJ6okQQG9sXuTfMX+enWcDDFEsC1l/juHuQIOPR\r
  30         HssOO1jI3BDvyVPA=\r
  31 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=SSpaeth.de; h=from:to:cc\r
  32         :subject:in-reply-to:references:date:message-id:mime-version:\r
  33         content-type; s=sspaeth.de; bh=OrVQHs1GQZBipZSf3NSI+VnuLsQ=; b=M\r
  34         5XfYXC0o/dmIiI5gwn48M0ek6E635QLtEW4in3FzCRXZyeCr3+PwjWUmLtCATKYd\r
  35         z64ikFKY8JH/aR616j/MXiwxEk0wrntx6DpzcDAcbQKGfwAnJ099SwFYF+X2qV5j\r
  36         MAc4EjwL5WETAOVurPvs6TldQe4sHIIBoziyCLiWkw=\r
  37 Received: from spaetzbook.sspaeth.de (unknown [84.55.203.213])\r
  38         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  39         (No client certificate requested)\r
  40         (Authenticated sender: fax@sspaeth.de)\r
  41         by homiemail-a18.g.dreamhost.com (Postfix) with ESMTPSA id 8996D250075; \r
  42         Thu, 29 Sep 2011 00:45:41 -0700 (PDT)\r
  43 Received: by spaetzbook.sspaeth.de (sSMTP sendmail emulation);\r
  44         Thu, 29 Sep 2011 09:45:38 +0200\r
  45 From: Sebastian Spaeth <Sebastian@SSpaeth.de>\r
  46 To: Justus Winter <4winter@informatik.uni-hamburg.de>, notmuch@notmuchmail.org\r
  47 Subject: Re: [PATCH 1/9] python: add a .gitignore file and refine the toplevel\r
  48         one\r
  49 In-Reply-To:\r
  50  <1316999137-28257-1-git-send-email-4winter@informatik.uni-hamburg.de>\r
  51 References:\r
  52  <1316999137-28257-1-git-send-email-4winter@informatik.uni-hamburg.de>\r
  53 User-Agent: Notmuch/0.7-19-gee4579a (http://notmuchmail.org) Emacs/23.2.1\r
  54         (x86_64-pc-linux-gnu)\r
  55 Date: Thu, 29 Sep 2011 09:45:38 +0200\r
  56 Message-ID: <871uuzlrkt.fsf@SSpaeth.de>\r
  57 MIME-Version: 1.0\r
  58 Content-Type: multipart/signed; boundary="=-=-=";\r
  59         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  60 X-BeenThere: notmuch@notmuchmail.org\r
  61 X-Mailman-Version: 2.1.13\r
  62 Precedence: list\r
  63 List-Id: "Use and development of the notmuch mail system."\r
  64         <notmuch.notmuchmail.org>\r
  65 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  67 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  68 List-Post: <mailto:notmuch@notmuchmail.org>\r
  69 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  70 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  72 X-List-Received-Date: Thu, 29 Sep 2011 07:45:45 -0000\r
  73 \r
  74 --=-=-=\r
  75 \r
  76 On Mon, 26 Sep 2011 03:05:29 +0200, Justus Winter wrote:\r
  77 \r
  78 #1) APPLIED\r
  79 #2) APPLIED\r
  80 #3) reorder the arguments of NotmuchError.__init__(): NOT APPLIED\r
  81 \r
  82 The python tutorial gives an example of custom TransitionError with\r
  83 three arguments, a custom message as the third. In addition, a STATUS\r
  84 value is always expected to be given, while the additional explanatory\r
  85 msg is optional, so STATUS makes for a more logical 1st parameter IMHO.\r
  86 Even if it were the case, it makes for lots of code churn, longer code\r
  87 (status=foo) to all Exceptions, and existing third party code would be\r
  88 broken. Overall, I think there is more potential for trouble than\r
  89 cleanup.\r
  90 \r
  91 #4) APPLIED\r
  92 status is always expected to be existing, but bullet proofing never hurts,\r
  93 so this patch makes sense.\r
  94 \r
  95 #5&#6) APPLIED\r
  96 Modified the patches to apply again, as some changes had been made.\r
  97 \r
  98 #7) NOT APPLIED, INPUT SOUGHT :)\r
  99 I do see the value of more fine grained exceptions, but I am not sure,\r
 100 we need this level of fine-grainedness. It would also make things more\r
 101 tricky (the API is still actively evolving, and e.g. 4 days ago, a new\r
 102 error status was added), so users of the bindings would now have\r
 103  +    NotmuchError,\r
 104  +    OutOfMemoryError,\r
 105  +    ReadOnlyDatabaseError,\r
 106  +    XapianError,\r
 107  +    FileError,\r
 108  +    FileNotEmailError,\r
 109  +    DuplicateMessageIdError,\r
 110  +    NullPointerError,\r
 111  +    TagTooLongError,\r
 112  +    UnbalancedFreezeThawError,\r
 113  +    UnbalancedAtomicError,\r
 114  +    NotInitializedError\r
 115 \r
 116 to check where e.g. Xapian could also hide an Out of Memory. Do people\r
 117 really want to catch, say UnbalancedAtomic errors specifically, rather\r
 118 than testing whether an operation succeeded, and check the status code\r
 119 if not? I could see the case for NotInitializedError, as that is a bit\r
 120 specific to the python bindings and users might want to catch it separately.\r
 121 \r
 122 Also, not all "status" are an error, e.g. DuplicateMessageId denotes\r
 123 success rather than failure, it just communicates a status.\r
 124 \r
 125 What do people (&user of the bindings) think would make sense here?\r
 126 I am not opposed, but want more discussion and input before such a\r
 127 change is made.\r
 128 \r
 129 #8) Not merged, as it depends on #7\r
 130 \r
 131 #9) APPLIED\r
 132 \r
 133 Thanks for the patches, most of them are quite nice. For 7&8, I'd like\r
 134 to hear more opinions.\r
 135 \r
 136 Sebastian\r
 137 \r
 138 --=-=-=\r
 139 Content-Type: application/pgp-signature\r
 140 \r
 141 -----BEGIN PGP SIGNATURE-----\r
 142 Version: GnuPG v1.4.11 (GNU/Linux)\r
 143 \r
 144 iEYEARECAAYFAk6EIiIACgkQVYX1jMgnoGIiFACggHuxiQ3uCVGkk9zJxvit1nHn\r
 145 KJoAn3GD4FLcUFMW6+00yr3+iMYCQmnW\r
 146 =b+jk\r
 147 -----END PGP SIGNATURE-----\r
 148 --=-=-=--\r