9a/e2dda3583504dbc4e69a0f009023b32aa2c087

   1 Return-Path: <fgeller@gmail.com>\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 C03A1431FD0\r
   6         for <notmuch@notmuchmail.org>; Sat, 16 Jul 2011 06:25:12 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] 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 I3RZ0wnru8Kb for <notmuch@notmuchmail.org>;\r
  17         Sat, 16 Jul 2011 06:25:10 -0700 (PDT)\r
  18 Received: from mail-fx0-f46.google.com (mail-fx0-f46.google.com\r
  19         [209.85.161.46]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 404CF431FB6\r
  22         for <notmuch@notmuchmail.org>; Sat, 16 Jul 2011 06:25:10 -0700 (PDT)\r
  23 Received: by fxh19 with SMTP id 19so4074099fxh.19\r
  24         for <notmuch@notmuchmail.org>; Sat, 16 Jul 2011 06:25:09 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type;\r
  28         bh=5KVQBqJG08NI9GOUv9QhDu+DDHOxv3Ct+sfApWLM2Ns=;\r
  29         b=RR7S1lKtBsnBg/sh8cbr4WbmU7St0SCkB0DvA9IdruayOlMPxUFv+0fOcC48IEjnMA\r
  30         PIQFCuOk9p9Qq+NfLB5zDKdmpmfBPIqmTnQEjyLmn0kzDIyIur0twUSjli5GzVxoc/hB\r
  31         Ion/wdQxzIeTjPGq+E3pxdKHmIJVo+b7Q/h3E=\r
  32 Received: by 10.223.23.143 with SMTP id r15mr7061631fab.29.1310822708837;\r
  33         Sat, 16 Jul 2011 06:25:08 -0700 (PDT)\r
  34 Received: from localhorst.example.com (brln-4dba6039.pool.mediaWays.net\r
  35         [77.186.96.57])\r
  36         by mx.google.com with ESMTPS id l22sm1520803fam.9.2011.07.16.06.25.05\r
  37         (version=TLSv1/SSLv3 cipher=OTHER);\r
  38         Sat, 16 Jul 2011 06:25:07 -0700 (PDT)\r
  39 Received: by localhorst.example.com (Postfix, from userid 501)\r
  40         id B875D470E7A8; Sat, 16 Jul 2011 15:25:03 +0200 (CEST)\r
  41 From: Felix Geller <fgeller@gmail.com>\r
  42 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  43  notmuch@notmuchmail.org\r
  44 Subject: Re: [PATCH] Emacs: Add variable to toggle message indentation in a\r
  45         thread\r
  46 In-Reply-To: <m2ipr7nju7.fsf@gmail.com>\r
  47 References: <m2r55xi4yr.fsf@gmail.com>\r
  48         <87aackwzf6.fsf@servo.factory.finestructure.net>\r
  49         <m2ipr7nju7.fsf@gmail.com>\r
  50 User-Agent: Emacs/23.3.1 (x86_64-apple-darwin10.8.0)\r
  51 Date: Sat, 16 Jul 2011 15:25:02 +0200\r
  52 Message-ID: <m2vcv2mk7l.fsf@gmail.com>\r
  53 MIME-Version: 1.0\r
  54 Content-Type: multipart/signed; boundary="==-=-=";\r
  55         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Sat, 16 Jul 2011 13:25:13 -0000\r
  69 \r
  70 --==-=-=\r
  71 Content-Type: multipart/mixed; boundary="=-=-="\r
  72 \r
  73 --=-=-=\r
  74 \r
  75 On Wed, 13 Jul 2011 01:46:24 +0200, Felix Geller <fgeller@gmail.com> wrote:\r
  76 \r
  77 > I attached two commits. One that includes the changes (which have most\r
  78 > comments incorporated, only that I stick to when rather than if) and one\r
  79 > that includes a test. I still can't run the tests myself, I attached an\r
  80 > excerpt of what errors come up for the emacs subset, so I haven't tested\r
  81 > the test itself :( However, it is a simple copy of an existing one, but\r
  82 > I use a let to change the value of the new variable and adapted one of\r
  83 > the existing expected outputs to lack indentation. I'd be grateful if\r
  84 > someone could test it...\r
  85 \r
  86 Got the test suite to run on my machine and I fixed a editing typo of\r
  87 the expected output. After that the test runs fine on my machine. The\r
  88 patch that has the actual changes needs no changing. Let me know if\r
  89 something's wrong/missing.\r
  90 \r
  91 \r
  92 Cheers,\r
  93 Felix\r
  94 \r
  95 \r
  96 --=-=-=\r
  97 Content-Type: text/x-patch\r
  98 Content-Disposition: attachment;\r
  99  filename=0001-Test-for-toggling-message-indentation.patch\r
 100 Content-Transfer-Encoding: quoted-printable\r
 101 \r
 102 From=203b3145ad0411dec9f1c6f93635184a1a308471a7 Mon Sep 17 00:00:00 2001\r
 103 From: Felix Geller <fgeller@gmail.com>\r
 104 Date: Wed, 13 Jul 2011 01:32:14 +0200\r
 105 Subject: [PATCH] Test for toggling message indentation\r
 106 \r
 107 =2D--\r
 108  test/emacs                                         |    7 +\r
 109  ...show-thread-maildir-storage-without-indentation |  215 ++++++++++++++++=\r
 110 ++++\r
 111  2 files changed, 222 insertions(+), 0 deletions(-)\r
 112  create mode 100644 test/emacs.expected-output/notmuch-show-thread-maildir-=\r
 113 storage-without-indentation\r
 114 \r
 115 diff --git a/test/emacs b/test/emacs\r
 116 index 53f455a..c65d6d8 100755\r
 117 =2D-- a/test/emacs\r
 118 +++ b/test/emacs\r
 119 @@ -55,6 +55,13 @@ test_emacs "(notmuch-show \"$maildir_storage_thread\")\r
 120             (test-output)"\r
 121  test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-thread-maildir-storage\r
 122 =20\r
 123 +test_begin_subtest "Basic notmuch-show view in emacs without indentation"\r
 124 +maildir_storage_thread=3D$(notmuch search --output=3Dthreads id:2009111719=\r
 125 0054.GU3165@dottiness.seas.harvard.edu)\r
 126 +test_emacs "(let ((notmuch-show-indent-messages-in-thread nil))\r
 127 +             (notmuch-show \"$maildir_storage_thread\")\r
 128 +             (test-output))"\r
 129 +test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-thread-maildir-storag=\r
 130 e-without-indentation\r
 131 +\r
 132  test_begin_subtest "notmuch-show for message with invalid From"\r
 133  add_message "[subject]=3D\"message-with-invalid-from\"" \\r
 134             "[from]=3D\"\\\"Invalid \\\" From\\\" <test_suite@notmuchmail.org>\""\r
 135 diff --git a/test/emacs.expected-output/notmuch-show-thread-maildir-storage=\r
 136 -without-indentation b/test/emacs.expected-output/notmuch-show-thread-maild=\r
 137 ir-storage-without-indentation\r
 138 new file mode 100644\r
 139 index 0000000..00bd006\r
 140 =2D-- /dev/null\r
 141 +++ b/test/emacs.expected-output/notmuch-show-thread-maildir-storage-withou=\r
 142 t-indentation\r
 143 @@ -0,0 +1,215 @@\r
 144 +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)\r
 145 +Subject: [notmuch] Working with Maildir storage?\r
 146 +To: notmuch@notmuchmail.org\r
 147 +Date: Tue, 17 Nov 2009 14:00:54 -0500\r
 148 +\r
 149 +[ multipart/mixed ]\r
 150 +[ multipart/signed ]\r
 151 +[ text/plain ]\r
 152 +I saw the LWN article and decided to take a look at notmuch.  I'm\r
 153 +currently using mutt and mairix to index and read a collection of\r
 154 +Maildir mail folders (around 40,000 messages total).\r
 155 +\r
 156 +notmuch indexed the messages without complaint, but my attempt at\r
 157 +searching bombed out. Running, for example:\r
 158 +\r
 159 +  notmuch search storage\r
 160 +\r
 161 +Resulted in 4604 lines of errors along the lines of:\r
 162 +\r
 163 +  Error opening\r
 164 +  /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostname,U=\r
 165 =3D3026:2,S:\r
 166 +  Too many open files\r
 167 +\r
 168 +I'm curious if this is expected behavior (i.e., notmuch does not work\r
 169 +with Maildir) or if something else is going on.\r
 170 +\r
 171 +Cheers,\r
 172 +\r
 173 +[ 5-line signature. Click/Enter to show. ]\r
 174 +--=20\r
 175 +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 176 +Senior Technologist, Computing and Information Technology\r
 177 +Harvard University School of Engineering and Applied Sciences\r
 178 +\r
 179 +[ application/pgp-signature ]\r
 180 +[ text/plain ]\r
 181 +[ 4-line signature. Click/Enter to show. ]\r
 182 +_______________________________________________\r
 183 +notmuch mailing list\r
 184 +notmuch@notmuchmail.org\r
 185 +http://notmuchmail.org/mailman/listinfo/notmuch\r
 186 +Mikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox signed unrea=\r
 187 d)\r
 188 +Subject: Re: [notmuch] Working with Maildir storage?\r
 189 +To: notmuch@notmuchmail.org\r
 190 +Date: Wed, 18 Nov 2009 01:02:38 +0600\r
 191 +\r
 192 +[ multipart/mixed ]\r
 193 +[ multipart/signed ]\r
 194 +[ text/plain ]\r
 195 +\r
 196 +Twas brillig at 14:00:54 17.11.2009 UTC-05 when lars@seas.harvard.edu did =\r
 197 gyre and gimble:\r
 198 +\r
 199 + LK> Resulted in 4604 lines of errors along the lines of:\r
 200 +\r
 201 + LK>   Error opening\r
 202 + LK>   /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostn=\r
 203 ame,U=3D3026:2,S:\r
 204 + LK>   Too many open files\r
 205 +\r
 206 +See the patch just posted here.\r
 207 +\r
 208 +[ 2-line signature. Click/Enter to show. ]\r
 209 +--=20\r
 210 +  http://fossarchy.blogspot.com/\r
 211 +[ application/pgp-signature ]\r
 212 +[ text/plain ]\r
 213 +[ 4-line signature. Click/Enter to show. ]\r
 214 +_______________________________________________\r
 215 +notmuch mailing list\r
 216 +notmuch@notmuchmail.org\r
 217 +http://notmuchmail.org/mailman/listinfo/notmuch\r
 218 +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed un=\r
 219 read)\r
 220 +Subject: Re: [notmuch] Working with Maildir storage?\r
 221 +To: Mikhail Gusarov <dottedmag@dottedmag.net>\r
 222 +Cc: notmuch@notmuchmail.org\r
 223 +Date: Tue, 17 Nov 2009 15:33:01 -0500\r
 224 +\r
 225 +[ multipart/mixed ]\r
 226 +[ multipart/signed ]\r
 227 +[ text/plain ]\r
 228 +> See the patch just posted here.\r
 229 +\r
 230 +Is the list archived anywhere?  The obvious archives\r
 231 +(http://notmuchmail.org/pipermail/notmuch/) aren't available, and I\r
 232 +think I subscribed too late to get the patch (I only just saw the\r
 233 +discussion about it).\r
 234 +\r
 235 +It doesn't look like the patch is in git yet.\r
 236 +\r
 237 +-- Lars\r
 238 +\r
 239 +[ 5-line signature. Click/Enter to show. ]\r
 240 +--=20\r
 241 +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 242 +Senior Technologist, Computing and Information Technology\r
 243 +Harvard University School of Engineering and Applied Sciences\r
 244 +\r
 245 +[ application/pgp-signature ]\r
 246 +[ text/plain ]\r
 247 +[ 4-line signature. Click/Enter to show. ]\r
 248 +_______________________________________________\r
 249 +notmuch mailing list\r
 250 +notmuch@notmuchmail.org\r
 251 +http://notmuchmail.org/mailman/listinfo/notmuch\r
 252 +Mikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox unread)\r
 253 +Subject: [notmuch] Working with Maildir storage?\r
 254 +To: notmuch@notmuchmail.org\r
 255 +Date: Wed, 18 Nov 2009 02:50:48 +0600\r
 256 +\r
 257 +\r
 258 +Twas brillig at 15:33:01 17.11.2009 UTC-05 when lars at seas.harvard.edu d=\r
 259 id gyre and gimble:\r
 260 +\r
 261 + LK> Is the list archived anywhere?  The obvious archives\r
 262 + LK> (http://notmuchmail.org/pipermail/notmuch/) aren't available, and I\r
 263 + LK> think I subscribed too late to get the patch (I only just saw the\r
 264 + LK> discussion about it).\r
 265 +\r
 266 + LK> It doesn't look like the patch is in git yet.\r
 267 +\r
 268 +Just has been pushed\r
 269 +\r
 270 +[ 10-line signature. Click/Enter to show. ]\r
 271 +--=20\r
 272 +  http://fossarchy.blogspot.com/\r
 273 +-------------- next part --------------\r
 274 +A non-text attachment was scrubbed...\r
 275 +Name: not available\r
 276 +Type: application/pgp-signature\r
 277 +Size: 834 bytes\r
 278 +Desc: not available\r
 279 +URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20091118/0e33d9=\r
 280 64/attachment.pgp>\r
 281 +\r
 282 +Keith Packard <keithp@keithp.com> (2009-11-17) (inbox unread)\r
 283 +Subject: [notmuch] Working with Maildir storage?\r
 284 +To: notmuch@notmuchmail.org\r
 285 +Date: Tue, 17 Nov 2009 13:24:13 -0800\r
 286 +\r
 287 +On Tue, 17 Nov 2009 15:33:01 -0500, Lars Kellogg-Stedman <lars at seas.har=\r
 288 vard.edu> wrote:\r
 289 +> > See the patch just posted here.\r
 290 +\r
 291 +I've also pushed a slightly more complicated (and complete) fix to my\r
 292 +private notmuch repository\r
 293 +\r
 294 +git://keithp.com/git/notmuch\r
 295 +\r
 296 +> Is the list archived anywhere?\r
 297 +\r
 298 +Oops. Looks like Carl's mail server is broken. He's traveling to\r
 299 +Barcelona today and so it won't get fixed for a while.\r
 300 +\r
 301 +Thanks to everyone for trying out notmuch!\r
 302 +\r
 303 +-keith\r
 304 +\r
 305 +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-18) (inbox signed un=\r
 306 read)\r
 307 +Subject: Re: [notmuch] Working with Maildir storage?\r
 308 +To: Keith Packard <keithp@keithp.com>\r
 309 +Cc: notmuch@notmuchmail.org\r
 310 +Date: Tue, 17 Nov 2009 19:50:40 -0500\r
 311 +\r
 312 +[ multipart/mixed ]\r
 313 +[ multipart/signed ]\r
 314 +[ text/plain ]\r
 315 +> I've also pushed a slightly more complicated (and complete) fix to my\r
 316 +> private notmuch repository\r
 317 +\r
 318 +The version of lib/messages.cc in your repo doesn't build because it's\r
 319 +missing "#include <stdint.h>" (for the uint32_t on line 466).\r
 320 +\r
 321 +[ 5-line signature. Click/Enter to show. ]\r
 322 +--=20\r
 323 +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 324 +Senior Technologist, Computing and Information Technology\r
 325 +Harvard University School of Engineering and Applied Sciences\r
 326 +\r
 327 +[ application/pgp-signature ]\r
 328 +[ text/plain ]\r
 329 +[ 4-line signature. Click/Enter to show. ]\r
 330 +_______________________________________________\r
 331 +notmuch mailing list\r
 332 +notmuch@notmuchmail.org\r
 333 +http://notmuchmail.org/mailman/listinfo/notmuch\r
 334 +Carl Worth <cworth@cworth.org> (2009-11-18) (inbox unread)\r
 335 +Subject: [notmuch] Working with Maildir storage?\r
 336 +To: notmuch@notmuchmail.org\r
 337 +Date: Wed, 18 Nov 2009 02:08:10 -0800\r
 338 +\r
 339 +On Tue, 17 Nov 2009 14:00:54 -0500, Lars Kellogg-Stedman <lars at seas.har=\r
 340 vard.edu> wrote:\r
 341 +> I saw the LWN article and decided to take a look at notmuch.  I'm\r
 342 +> currently using mutt and mairix to index and read a collection of\r
 343 +> Maildir mail folders (around 40,000 messages total).\r
 344 +\r
 345 +Welcome, Lars!\r
 346 +\r
 347 +I hadn't even seen that Keith's blog post had been picked up by lwn.net.\r
 348 +That's very interesting. So, thanks for coming and trying out notmuch.\r
 349 +\r
 350 +>   Error opening\r
 351 +>   /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostname=\r
 352 ,U=3D3026:2,S:\r
 353 +>   Too many open files\r
 354 +\r
 355 +Sadly, the lwn article coincided with me having just introduced this\r
 356 +bug, and then getting on a Trans-Atlantic flight. So I fixed the bug\r
 357 +fairly quickly, but there was quite a bit of latency before I could push\r
 358 +the fix out. It should be fixed now.\r
 359 +\r
 360 +> I'm curious if this is expected behavior (i.e., notmuch does not work\r
 361 +> with Maildir) or if something else is going on.\r
 362 +\r
 363 +Notmuch works just fine with maildir---it's one of the things that it\r
 364 +likes the best.\r
 365 +\r
 366 +Happy hacking,\r
 367 +\r
 368 +-Carl\r
 369 +\r
 370 =2D-=20\r
 371 1.7.6\r
 372 \r
 373 \r
 374 --=-=-=\r
 375 Content-Transfer-Encoding: quoted-printable\r
 376 \r
 377 \r
 378 \r
 379 \r
 380 >\r
 381 >=20\r
 382 > Didn't know about format-patch, thanks :)=20\r
 383 >=20\r
 384 >=20\r
 385 > Cheers,\r
 386 > Felix\r
 387 >=20\r
 388 >=20\r
 389 > >\r
 390 > >=20\r
 391 > > > +(defcustom notmuch-show-indent-messages-in-thread nil\r
 392 > > > +  "Should the messages in a thread be indented according to their re=\r
 393 spective depth in the thread?"\r
 394 > > > +  :group 'notmuch\r
 395 > > > +  :type 'boolean)\r
 396 > >=20\r
 397 > > I agres with Dmitry that this should default to 't', to be consistent\r
 398 > > with the current default behavior.\r
 399 > >=20\r
 400 > > > -    (insert (notmuch-show-spaces-n depth)\r
 401 > > > +    (insert (if notmuch-show-indent-messages-in-thread\r
 402 > > > +         (notmuch-show-spaces-n depth)\r
 403 > > > +       "")\r
 404 > >=20\r
 405 > > I also agree with Dmitry's suggestion here to use the following slightly\r
 406 > > simpler construct:\r
 407 > >=20\r
 408 > >  (if notmuch-show-indent-messages-in-thread\r
 409 > >      (insert (notmuch-show-spaces-n depth)))\r
 410 > >=20\r
 411 > > Finally, as Dmitry also points out, you'll almost certainly need to\r
 412 > > construct a test for this feature, since it constitutes a pretty big\r
 413 > > formatting change.  It should probably test for both cases of the\r
 414 > > customization variable.  Check out the tests in tests/emacs for\r
 415 > > guidance.\r
 416 > >=20\r
 417 > > hth.\r
 418 > >=20\r
 419 > > jamie.\r
 420 > Non-text part: application/pgp-signature\r
 421 >=20\r
 422 > From 94570ada2eae1ff1a1feebbda5dff04d223da6b4 Mon Sep 17 00:00:00 2001\r
 423 > From: Felix Geller <fgeller@gmail.com>\r
 424 > Date: Wed, 13 Jul 2011 01:30:51 +0200\r
 425 > Subject: [PATCH 1/2] Added variable to toggle message indendation in Emac=\r
 426 s'\r
 427 >  notmuch-show\r
 428 >=20\r
 429 > ---\r
 430 >  emacs/notmuch-show.el |   13 ++++++++++---\r
 431 >  1 files changed, 10 insertions(+), 3 deletions(-)\r
 432 >=20\r
 433 > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
 434 > index f96743b..48df4c5 100644\r
 435 > --- a/emacs/notmuch-show.el\r
 436 > +++ b/emacs/notmuch-show.el\r
 437 > @@ -90,6 +90,11 @@ any given message."\r
 438 >    :group 'notmuch\r
 439 >    :type 'boolean)\r
 440 >=20=20\r
 441 > +(defcustom notmuch-show-indent-messages-in-thread t\r
 442 > +  "Should messages be indented according to their depth in a thread?"\r
 443 > +  :group 'notmuch\r
 444 > +  :type 'boolean)\r
 445 > +\r
 446 >  (defcustom notmuch-show-indent-multipart nil\r
 447 >    "Should the sub-parts of a multipart/* part be indented?"\r
 448 >    ;; dme: Not sure which is a good default.\r
 449 > @@ -237,8 +242,9 @@ unchanged ADDRESS if parsing fails."\r
 450 >    "Insert a notmuch style headerline based on HEADERS for a\r
 451 >  message at DEPTH in the current thread."\r
 452 >    (let ((start (point)))\r
 453 > -    (insert (notmuch-show-spaces-n depth)\r
 454 > -         (notmuch-show-clean-address (plist-get headers :From))\r
 455 > +    (when notmuch-show-indent-messages-in-thread\r
 456 > +      (insert (notmuch-show-spaces-n depth)))\r
 457 > +    (insert (notmuch-show-clean-address (plist-get headers :From))\r
 458 >           " ("\r
 459 >           date\r
 460 >           ") ("\r
 461 > @@ -733,7 +739,8 @@ current buffer, if possible."\r
 462 >      (setq content-end (point-marker))\r
 463 >=20=20\r
 464 >      ;; Indent according to the depth in the thread.\r
 465 > -    (indent-rigidly content-start content-end depth)\r
 466 > +    (when notmuch-show-indent-messages-in-thread\r
 467 > +      (indent-rigidly content-start content-end depth))\r
 468 >=20=20\r
 469 >      (setq message-end (point-max-marker))\r
 470 >=20=20\r
 471 > --=20\r
 472 > 1.7.6\r
 473 >=20\r
 474 > From 016ee41930eed6b53e10c35c62c40fa6bb35b79e Mon Sep 17 00:00:00 2001\r
 475 > From: Felix Geller <fgeller@gmail.com>\r
 476 > Date: Wed, 13 Jul 2011 01:32:14 +0200\r
 477 > Subject: [PATCH 2/2] Test for toggling message indentation\r
 478 >=20\r
 479 > ---\r
 480 >  test/emacs                                         |    7 +\r
 481 >  ...show-thread-maildir-storage-without-indentation |  215 ++++++++++++++=\r
 482 ++++++\r
 483 >  2 files changed, 222 insertions(+), 0 deletions(-)\r
 484 >  create mode 100644 test/emacs.expected-output/notmuch-show-thread-maildi=\r
 485 r-storage-without-indentation\r
 486 >=20\r
 487 > diff --git a/test/emacs b/test/emacs\r
 488 > index 53f455a..c65d6d8 100755\r
 489 > --- a/test/emacs\r
 490 > +++ b/test/emacs\r
 491 > @@ -55,6 +55,13 @@ test_emacs "(notmuch-show \"$maildir_storage_thread\")\r
 492 >           (test-output)"\r
 493 >  test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-thread-maildir-stor=\r
 494 age\r
 495 >=20=20\r
 496 > +test_begin_subtest "Basic notmuch-show view in emacs without indentation"\r
 497 > +maildir_storage_thread=3D$(notmuch search --output=3Dthreads id:20091117=\r
 498 190054.GU3165@dottiness.seas.harvard.edu)\r
 499 > +test_emacs "(let ((notmuch-show-indent-messages-in-thread nil))\r
 500 > +           (notmuch-show \"$maildir_storage_thread\")\r
 501 > +           (test-output))"\r
 502 > +test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-thread-maildir-stor=\r
 503 age-without-indentation\r
 504 > +\r
 505 >  test_begin_subtest "notmuch-show for message with invalid From"\r
 506 >  add_message "[subject]=3D\"message-with-invalid-from\"" \\r
 507 >           "[from]=3D\"\\\"Invalid \\\" From\\\" <test_suite@notmuchmail.org>\=\r
 508 ""\r
 509 > diff --git a/test/emacs.expected-output/notmuch-show-thread-maildir-stora=\r
 510 ge-without-indentation b/test/emacs.expected-output/notmuch-show-thread-mai=\r
 511 ldir-storage-without-indentation\r
 512 > new file mode 100644\r
 513 > index 0000000..cdc8fbd\r
 514 > --- /dev/null\r
 515 > +++ b/test/emacs.expected-output/notmuch-show-thread-maildir-storage-with=\r
 516 out-indentation\r
 517 > @@ -0,0 +1,215 @@\r
 518 > +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)\r
 519 > +Subject: [notmuch] Working with Maildir storage?\r
 520 > +To: notmuch@notmuchmail.org\r
 521 > +Date: Tue, 17 Nov 2009 14:00:54 -0500\r
 522 > +\r
 523 > +[ multipart/mixed ]\r
 524 > +[ multipart/signed ]\r
 525 > +[ text/plain ]\r
 526 > +I saw the LWN article and decided to take a look at notmuch.  I'm\r
 527 > +currently using mutt and mairix to index and read a collection of\r
 528 > +Maildir mail folders (around 40,000 messages total).\r
 529 > +\r
 530 > +notmuch indexed the messages without complaint, but my attempt at\r
 531 > +searching bombed out. Running, for example:\r
 532 > +\r
 533 > +  notmuch search storage\r
 534 > +\r
 535 > +Resulted in 4604 lines of errors along the lines of:\r
 536 > +\r
 537 > +  Error opening\r
 538 > +  /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostname=\r
 539 ,U=3D3026:2,S:\r
 540 > +  Too many open files\r
 541 > +\r
 542 > +I'm curious if this is expected behavior (i.e., notmuch does not work\r
 543 > +with Maildir) or if something else is going on.\r
 544 > +\r
 545 > +Cheers,\r
 546 > +\r
 547 > +[ 5-line signature. Click/Enter to show. ]\r
 548 > +--=20\r
 549 > +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 550 > +Senior Technologist, Computing and Information Technology\r
 551 > +Harvard University School of Engineering and Applied Sciences\r
 552 > +\r
 553 > +[ application/pgp-signature ]\r
 554 > +[ text/plain ]\r
 555 > +[ 4-line signature. Click/Enter to show. ]\r
 556 > +_______________________________________________\r
 557 > +notmuch mailing list\r
 558 > +notmuch@notmuchmail.org\r
 559 > +http://notmuchmail.org/mailman/listinfo/notmuch\r
 560 > +Mikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox signed unr=\r
 561 ead)\r
 562 > +Subject: Re: [notmuch] Working with Maildir storage?\r
 563 > +To: notmuch@notmuchmail.org\r
 564 > +Date: Wed, 18 Nov 2009 01:02:38 +0600\r
 565 > +\r
 566 > +[ multipart/mixed ]\r
 567 > +[ multipart/signed ]\r
 568 > +[ text/plain ]\r
 569 > +\r
 570 > +Twas brillig at 14:00:54 17.11.2009 UTC-05 when lars@seas.harvard.edu di=\r
 571 d gyre and gimble:\r
 572 > +\r
 573 > + LK> Resulted in 4604 lines of errors along the lines of:\r
 574 > +\r
 575 > + LK>   Error opening\r
 576 > + LK>   /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hos=\r
 577 tname,U=3D3026:2,S:\r
 578 > + LK>   Too many open files\r
 579 > +\r
 580 > +See the patch just posted here.\r
 581 > +\r
 582 > +[ 2-line signature. Click/Enter to show. ]\r
 583 > +--=20\r
 584 > +http://fossarchy.blogspot.com/\r
 585 > +[ application/pgp-signature ]\r
 586 > +[ text/plain ]\r
 587 > +[ 4-line signature. Click/Enter to show. ]\r
 588 > +_______________________________________________\r
 589 > +notmuch mailing list\r
 590 > +notmuch@notmuchmail.org\r
 591 > +http://notmuchmail.org/mailman/listinfo/notmuch\r
 592 > +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed =\r
 593 unread)\r
 594 > +Subject: Re: [notmuch] Working with Maildir storage?\r
 595 > +To: Mikhail Gusarov <dottedmag@dottedmag.net>\r
 596 > +Cc: notmuch@notmuchmail.org\r
 597 > +Date: Tue, 17 Nov 2009 15:33:01 -0500\r
 598 > +\r
 599 > +[ multipart/mixed ]\r
 600 > +[ multipart/signed ]\r
 601 > +[ text/plain ]\r
 602 > +> See the patch just posted here.\r
 603 > +\r
 604 > +Is the list archived anywhere?  The obvious archives\r
 605 > +(http://notmuchmail.org/pipermail/notmuch/) aren't available, and I\r
 606 > +think I subscribed too late to get the patch (I only just saw the\r
 607 > +discussion about it).\r
 608 > +\r
 609 > +It doesn't look like the patch is in git yet.\r
 610 > +\r
 611 > +-- Lars\r
 612 > +\r
 613 > +[ 5-line signature. Click/Enter to show. ]\r
 614 > +--=20\r
 615 > +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 616 > +Senior Technologist, Computing and Information Technology\r
 617 > +Harvard University School of Engineering and Applied Sciences\r
 618 > +\r
 619 > +[ application/pgp-signature ]\r
 620 > +[ text/plain ]\r
 621 > +[ 4-line signature. Click/Enter to show. ]\r
 622 > +_______________________________________________\r
 623 > +notmuch mailing list\r
 624 > +notmuch@notmuchmail.org\r
 625 > +http://notmuchmail.org/mailman/listinfo/notmuch\r
 626 > +ikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox unread)\r
 627 > +ubject: [notmuch] Working with Maildir storage?\r
 628 > +o: notmuch@notmuchmail.org\r
 629 > +ate: Wed, 18 Nov 2009 02:50:48 +0600\r
 630 > +\r
 631 > +\r
 632 > +was brillig at 15:33:01 17.11.2009 UTC-05 when lars at seas.harvard.edu =\r
 633 did gyre and gimble:\r
 634 > +\r
 635 > +LK> Is the list archived anywhere?  The obvious archives\r
 636 > +LK> (http://notmuchmail.org/pipermail/notmuch/) aren't available, and I\r
 637 > +LK> think I subscribed too late to get the patch (I only just saw the\r
 638 > +LK> discussion about it).\r
 639 > +\r
 640 > +LK> It doesn't look like the patch is in git yet.\r
 641 > +\r
 642 > +ust has been pushed\r
 643 > +\r
 644 > + 10-line signature. Click/Enter to show. ]\r
 645 > +-=20\r
 646 > +ttp://fossarchy.blogspot.com/\r
 647 > +------------- next part --------------\r
 648 > + non-text attachment was scrubbed...\r
 649 > +ame: not available\r
 650 > +ype: application/pgp-signature\r
 651 > +ize: 834 bytes\r
 652 > +esc: not available\r
 653 > +RL: <http://notmuchmail.org/pipermail/notmuch/attachments/20091118/0e33d=\r
 654 964/attachment.pgp>\r
 655 > +\r
 656 > +eith Packard <keithp@keithp.com> (2009-11-17) (inbox unread)\r
 657 > +ubject: [notmuch] Working with Maildir storage?\r
 658 > +o: notmuch@notmuchmail.org\r
 659 > +ate: Tue, 17 Nov 2009 13:24:13 -0800\r
 660 > +\r
 661 > +n Tue, 17 Nov 2009 15:33:01 -0500, Lars Kellogg-Stedman <lars at seas.ha=\r
 662 rvard.edu> wrote:\r
 663 > + > See the patch just posted here.\r
 664 > +\r
 665 > +'ve also pushed a slightly more complicated (and complete) fix to my\r
 666 > +rivate notmuch repository\r
 667 > +\r
 668 > +it://keithp.com/git/notmuch\r
 669 > +\r
 670 > + Is the list archived anywhere?\r
 671 > +\r
 672 > +ops. Looks like Carl's mail server is broken. He's traveling to\r
 673 > +arcelona today and so it won't get fixed for a while.\r
 674 > +\r
 675 > +hanks to everyone for trying out notmuch!\r
 676 > +\r
 677 > +keith\r
 678 > +\r
 679 > +Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-18) (inbox signed =\r
 680 unread)\r
 681 > +Subject: Re: [notmuch] Working with Maildir storage?\r
 682 > +To: Keith Packard <keithp@keithp.com>\r
 683 > +Cc: notmuch@notmuchmail.org\r
 684 > +Date: Tue, 17 Nov 2009 19:50:40 -0500\r
 685 > +\r
 686 > +[ multipart/mixed ]\r
 687 > +[ multipart/signed ]\r
 688 > +[ text/plain ]\r
 689 > +> I've also pushed a slightly more complicated (and complete) fix to my\r
 690 > +> private notmuch repository\r
 691 > +\r
 692 > +The version of lib/messages.cc in your repo doesn't build because it's\r
 693 > +missing "#include <stdint.h>" (for the uint32_t on line 466).\r
 694 > +\r
 695 > +[ 5-line signature. Click/Enter to show. ]\r
 696 > +--=20\r
 697 > +Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 698 > +Senior Technologist, Computing and Information Technology\r
 699 > +Harvard University School of Engineering and Applied Sciences\r
 700 > +\r
 701 > +[ application/pgp-signature ]\r
 702 > +[ text/plain ]\r
 703 > +[ 4-line signature. Click/Enter to show. ]\r
 704 > +_______________________________________________\r
 705 > +notmuch mailing list\r
 706 > +notmuch@notmuchmail.org\r
 707 > +http://notmuchmail.org/mailman/listinfo/notmuch\r
 708 > +Carl Worth <cworth@cworth.org> (2009-11-18) (inbox unread)\r
 709 > +Subject: [notmuch] Working with Maildir storage?\r
 710 > +To: notmuch@notmuchmail.org\r
 711 > +Date: Wed, 18 Nov 2009 02:08:10 -0800\r
 712 > +\r
 713 > +On Tue, 17 Nov 2009 14:00:54 -0500, Lars Kellogg-Stedman <lars at seas.h=\r
 714 arvard.edu> wrote:\r
 715 > +> I saw the LWN article and decided to take a look at notmuch.  I'm\r
 716 > +> currently using mutt and mairix to index and read a collection of\r
 717 > +> Maildir mail folders (around 40,000 messages total).\r
 718 > +\r
 719 > +Welcome, Lars!\r
 720 > +\r
 721 > +I hadn't even seen that Keith's blog post had been picked up by lwn.net.\r
 722 > +That's very interesting. So, thanks for coming and trying out notmuch.\r
 723 > +\r
 724 > +>   Error opening\r
 725 > +>   /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostna=\r
 726 me,U=3D3026:2,S:\r
 727 > +>   Too many open files\r
 728 > +\r
 729 > +Sadly, the lwn article coincided with me having just introduced this\r
 730 > +bug, and then getting on a Trans-Atlantic flight. So I fixed the bug\r
 731 > +fairly quickly, but there was quite a bit of latency before I could push\r
 732 > +the fix out. It should be fixed now.\r
 733 > +\r
 734 > +> I'm curious if this is expected behavior (i.e., notmuch does not work\r
 735 > +> with Maildir) or if something else is going on.\r
 736 > +\r
 737 > +Notmuch works just fine with maildir---it's one of the things that it\r
 738 > +likes the best.\r
 739 > +\r
 740 > +Happy hacking,\r
 741 > +\r
 742 > +-Carl\r
 743 > +\r
 744 > --=20\r
 745 > 1.7.6\r
 746 >=20\r
 747 Non-text part: application/pgp-signature\r
 748 \r
 749 --=-=-=--\r
 750 \r
 751 --==-=-=\r
 752 Content-Type: application/pgp-signature\r
 753 \r
 754 -----BEGIN PGP SIGNATURE-----\r
 755 Version: GnuPG/MacGPG2 v2.0.17 (Darwin)\r
 756 \r
 757 iEYEARECAAYFAk4hkS8ACgkQMxgKfMb9y5aXbwCdHinY92kqbDUW4aFdOHoGT5r+\r
 758 7xMAn3TxfSI3Rvqov48PtSf5mIviF1XU\r
 759 =3bcH\r
 760 -----END PGP SIGNATURE-----\r
 761 --==-=-=--\r