07/6497d46697938575dbbbfc9c11d1b39b8a6d46

   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 086BC431FC2;\r
   6         Wed, 18 Nov 2009 08:29:37 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id DsOqqwEWzYp2; Wed, 18 Nov 2009 08:29:36 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id A961F431FBF;\r
  13         Wed, 18 Nov 2009 08:29:35 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Lars Kellogg-Stedman <lars@seas.harvard.edu>, notmuch\r
  16         <notmuch@notmuchmail.org>\r
  17 In-Reply-To: <20091118010116.GC25380@dottiness.seas.harvard.edu>\r
  18 References: <20091118005829.GB25380@dottiness.seas.harvard.edu>\r
  19         <20091118010116.GC25380@dottiness.seas.harvard.edu>\r
  20 Date: Wed, 18 Nov 2009 08:29:20 -0800\r
  21 Message-ID: <874ooraja7.fsf@yoom.home.cworth.org>\r
  22 MIME-Version: 1.0\r
  23 Content-Type: text/plain; charset=us-ascii\r
  24 Subject: Re: [notmuch] "notmuch help" outputs to stderr?\r
  25 X-BeenThere: notmuch@notmuchmail.org\r
  26 X-Mailman-Version: 2.1.12\r
  27 Precedence: list\r
  28 List-Id: "Use and development of the notmuch mail system."\r
  29         <notmuch.notmuchmail.org>\r
  30 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  31         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  32 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  33 List-Post: <mailto:notmuch@notmuchmail.org>\r
  34 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  35 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  37 X-List-Received-Date: Wed, 18 Nov 2009 16:29:37 -0000\r
  38 \r
  39 On Tue, 17 Nov 2009 20:01:16 -0500, Lars Kellogg-Stedman <lars@seas.harvard.edu> wrote:\r
  40 > > I've attached a patch that lets usage() take a FILE * argument so that\r
  41 > > you can output to stderr in response to usage errors, and stdout in\r
  42 > > response to an explicit request.\r
  43 > \r
  44 > Whoops, missed a couple of stderr's in that last patch.  New one\r
  45 > attached.\r
  46 > \r
  47 > -- \r
  48 > Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
  49 > Senior Technologist, Computing and Information Technology\r
  50 > Harvard University School of Engineering and Applied Sciences\r
  51 \r
  52 Thanks Lars!\r
  53 \r
  54 That was obviously a bug, so I really appreciate both your report and\r
  55 your fix.\r
  56 \r
  57 BTW, for future patches, it would be easier for me, (and hopefully not\r
  58 too much harder for you), if you could format your mails so that "git\r
  59 am" will find a good commit message in them. There are a few different\r
  60 ways you can do that:\r
  61 \r
  62 Option (1) takes more work upfront to get comfortable with it and to\r
  63 make sure it works, but it's probably less work (particuarly 1.b with\r
  64 "git send-email) once you get into the flow of it. Option (2) is\r
  65 probably the least different from what you're currently doing.\r
  66 \r
  67 1. Make a commit locally with the commit message. Then run "git\r
  68    format-patch", (with "git format-patch HEAD~N" where N is the number\r
  69    of commits you want to generate), to generate formatted email\r
  70    messages, (or even an entire mbox).\r
  71 \r
  72    a. Suck those messages up into an email program. Add any commentary\r
  73       (to not be part of the commit message) after the --- separator and\r
  74       send.\r
  75 \r
  76    b. Use "git send-email" to fire the messages off.\r
  77 \r
  78 2. Just attach the "git diff" output like you did, but then carefully\r
  79    word the subject and body of the email message to look like a commit\r
  80    message. You can again use a --- separator for commentary that's not\r
  81    part of the patch.\r
  82 \r
  83 For example, option (2) would have looked like the below to generate the\r
  84 commit message I ended up using for your commit.\r
  85 \r
  86 Anyway, thanks again,\r
  87 \r
  88 -Carl\r
  89 \r
  90 Subject: [Patch] notmuch help: Print to stdout, not to stderr.\r
  91 \r
  92 Let usage() take a FILE * argument so that you can output to stderr in\r
  93 response to usage errors, and stdout in response to an explicit request.\r
  94 \r
  95 ---\r
  96 \r
  97 Whoops, missed a couple of stderr's in that last patch.  New one\r
  98 attached.\r
  99 \r
 100 -- \r
 101 Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
 102 Senior Technologist, Computing and Information Technology\r
 103 Harvard University School of Engineering and Applied Sciences\r