c2/094e4d6269a808e24d7f3116df9001fd8c6493

   1 Return-Path: <cworth@cworth.org>\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 B7834431FD0\r
   6         for <notmuch@notmuchmail.org>; Fri,  3 Jun 2011 12:56:57 -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 mOmNjfiB63Gn for <notmuch@notmuchmail.org>;\r
  16         Fri,  3 Jun 2011 12:56:57 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 0A0B4431FB6\r
  19         for <notmuch@notmuchmail.org>; Fri,  3 Jun 2011 12:56:57 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id EA13D29A047;\r
  22         Fri,  3 Jun 2011 12:56:55 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id D96DB54C02C; Fri,  3 Jun 2011 12:56:55 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  27         Notmuch Mail <notmuch@notmuchmail.org>\r
  28 Subject: Re: [PATCH 25/25] Fix stdout stream grabbing in\r
  29         format_part_content_text\r
  30 In-Reply-To: <1306619520-25730-26-git-send-email-jrollins@finestructure.net>\r
  31 References: <1306619520-25730-1-git-send-email-jrollins@finestructure.net>\r
  32         <1306619520-25730-2-git-send-email-jrollins@finestructure.net>\r
  33         <1306619520-25730-3-git-send-email-jrollins@finestructure.net>\r
  34         <1306619520-25730-4-git-send-email-jrollins@finestructure.net>\r
  35         <1306619520-25730-5-git-send-email-jrollins@finestructure.net>\r
  36         <1306619520-25730-6-git-send-email-jrollins@finestructure.net>\r
  37         <1306619520-25730-7-git-send-email-jrollins@finestructure.net>\r
  38         <1306619520-25730-8-git-send-email-jrollins@finestructure.net>\r
  39         <1306619520-25730-9-git-send-email-jrollins@finestructure.net>\r
  40         <1306619520-25730-10-git-send-email-jrollins@finestructure.net>\r
  41         <1306619520-25730-11-git-send-email-jrollins@finestructure.net>\r
  42         <1306619520-25730-12-git-send-email-jrollins@finestructure.net>\r
  43         <1306619520-25730-13-git-send-email-jrollins@finestructure.net>\r
  44         <1306619520-25730-14-git-send-email-jrollins@finestructure.net>\r
  45         <1306619520-25730-15-git-send-email-jrollins@finestructure.net>\r
  46         <1306619520-25730-16-git-se nd-email-jrollins@finestructure.net>\r
  47         <1306619520-25730-17-git-send-email-jrollins@finestructure.net>\r
  48         <1306619520-25730-18-git-send-email-jrollins@finestructure.net>\r
  49         <1306619520-25730-19-git-send-email-jrollins@finestructure.net>\r
  50         <1306619520-25730-20-git-send-email-jrollins@finestructure.net>\r
  51         <1306619520-25730-21-git-send-email-jrollins@finestructure.net>\r
  52         <1306619520-25730-22-git-send-email-jrollins@finestructure.net>\r
  53         <1306619520-25730-23-git-send-email-jrollins@finestructure.net>\r
  54         <1306619520-25730-24-git-send-email-jrollins@finestructure.net>\r
  55         <1306619520-25730-25-git-send-email-jrollins@finestructure.net>\r
  56         <1306619520-25730-26-git-send-email-jrollins@finestructure.net>\r
  57 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  58         (i486-pc-linux-gnu)\r
  59 Date: Fri, 03 Jun 2011 12:56:50 -0700\r
  60 Message-ID: <87r57almd9.fsf@yoom.home.cworth.org>\r
  61 MIME-Version: 1.0\r
  62 Content-Type: multipart/signed; boundary="=-=-=";\r
  63         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Fri, 03 Jun 2011 19:56:57 -0000\r
  77 \r
  78 --=-=-=\r
  79 Content-Type: text/plain; charset=utf-8\r
  80 Content-Transfer-Encoding: quoted-printable\r
  81 \r
  82 On Sat, 28 May 2011 14:52:00 -0700, Jameson Graef Rollins <jrollins@finestr=\r
  83 ucture.net> wrote:\r
  84 > The declaration of the GMimeStream pointer to stdout in\r
  85 > format_part_content_text was somehow preventing subsequent printf\r
  86 > calls from outputting to stdout if the output was redirected to a\r
  87 > file.  Scoping the declaration to the actual use of the stream pointer\r
  88 > works around this problem.\r
  89 \r
  90 This commit message sounds like we don't actually understand the problem\r
  91 being fixed here.\r
  92 \r
  93 I'd like to investigate this more. Perhaps with a test case?\r
  94 \r
  95 Otherwise, the patches up to this point in the thread have all either\r
  96 been pushed or I've asked for some additional information (perhaps\r
  97 that's just this patch and the "old style fcc dirs" patch?).\r
  98 \r
  99 I'll continue to work through my patch queue as contained in email\r
 100 messages. I'm finding it easier to do that (and just piping the patches\r
 101 I like to "git am") as opposed to working through a release-candidate\r
 102 branch in git, (and having to keep rebasing/postponing it after I reject\r
 103 some patch in the series).\r
 104 \r
 105 I'm actually a bit surprised to see myself preferring patches in\r
 106 email. When Linus first wrote git, I couldn't understand why the Linux\r
 107 community kept to such a consistent culture of sending patches via\r
 108 email. It seemed so backwards to do these awkward machinations (git\r
 109 format-patch, git send-email, SMTP, MUA, git am), and risk all the\r
 110 problems of email clients corrupting patches, etc.=E2=80=94especially when =\r
 111 git\r
 112 has such clean mechanisms for reliably moving patches around (git push,\r
 113 git pull).\r
 114 \r
 115 Call me converted. Email really is the right way to do collaborative\r
 116 code development.\r
 117 \r
 118 =2DCarl\r
 119 \r
 120 --=-=-=\r
 121 Content-Type: application/pgp-signature\r
 122 \r
 123 -----BEGIN PGP SIGNATURE-----\r
 124 Version: GnuPG v1.4.11 (GNU/Linux)\r
 125 \r
 126 iEYEARECAAYFAk3pPIIACgkQ6JDdNq8qSWhPawCePe9T2KBadwP2rhfaY0aPqOVx\r
 127 OZQAn3b4zv8JccIp37uxcOOyEsPHS/xE\r
 128 =aA8X\r
 129 -----END PGP SIGNATURE-----\r
 130 --=-=-=--\r