ab/7c2276bedd0e7bbf5ede0d7dcd52ff58b255e2

   1 Return-Path: <awg@xvx.ca>\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 F3B88431E82\r
   6         for <notmuch@notmuchmail.org>; Sat, 31 Mar 2012 16:36:29 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 0JsRnXSaad2x for <notmuch@notmuchmail.org>;\r
  16         Sat, 31 Mar 2012 16:36:29 -0700 (PDT)\r
  17 Received: from mail-lpp01m010-f53.google.com (mail-lpp01m010-f53.google.com\r
  18         [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 2E262431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat, 31 Mar 2012 16:36:29 -0700 (PDT)\r
  22 Received: by lahc1 with SMTP id c1so2030783lah.26\r
  23         for <notmuch@notmuchmail.org>; Sat, 31 Mar 2012 16:36:27 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=mime-version:sender:x-originating-ip:in-reply-to:references:date\r
  27         :x-google-sender-auth:message-id:subject:from:to:cc:content-type\r
  28         :x-gm-message-state;\r
  29         bh=6l1ZePoMfJXAxwz01Sv2AMEMb1au8wY70tLmPkiDP4E=;\r
  30         b=niiFS0X0FLmTwZkyh5fmiOaufaQ5/2ZVobQMDgB9ryFlvZg+1oD0UAd3e8ENsgNm2S\r
  31         e4qqx9MDU7GHYQ08CNsCCDB/Ggpii/ceIv9zV+gu5SI5v8g4lixeQaPMv5lSlhyxbFHR\r
  32         EERpdA79/GsakS2JipPqHb/1kLvZvV1DGkIQWG19lEbvn5QyWpaOOGQPSI8TeAlmWm9W\r
  33         n5z8O+vAzsm4P6V2zsZaHBNaxm6NQVYiw5qurUT5DMbQwzNy3Av3mIElk34eHCVN1c8j\r
  34         0iWlWVi637ysTSJ1RvAxjZBS8ZziNvQRWX1pxKE97B/mqQCGIiG4W1B3RkVYdj8M+VhX\r
  35         IoCg==\r
  36 MIME-Version: 1.0\r
  37 Received: by 10.152.148.234 with SMTP id tv10mr3782185lab.41.1333236987446;\r
  38         Sat, 31 Mar 2012 16:36:27 -0700 (PDT)\r
  39 Sender: awg@xvx.ca\r
  40 Received: by 10.112.97.14 with HTTP; Sat, 31 Mar 2012 16:36:27 -0700 (PDT)\r
  41 X-Originating-IP: [96.52.216.56]\r
  42 In-Reply-To: <87k42116b2.fsf@qmul.ac.uk>\r
  43 References: <1333038410-17927-1-git-send-email-awg+notmuch@xvx.ca>\r
  44         <1333038410-17927-2-git-send-email-awg+notmuch@xvx.ca>\r
  45         <87k42116b2.fsf@qmul.ac.uk>\r
  46 Date: Sat, 31 Mar 2012 17:36:27 -0600\r
  47 X-Google-Sender-Auth: WSAoc8Fee72nCGPtNChxiF-S0u0\r
  48 Message-ID:\r
  49  <CAMoJFUvMW1SCj843N5yj8LJJF=r-kSEe=Rvue=7x5Rwgm+A1pw@mail.gmail.com>\r
  50 Subject: Re: [BUG/PATCH v2 1/2] emacs: Fix header problem in reply for emacs\r
  51         23.2\r
  52 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
  53 To: Mark Walters <markwalters1009@gmail.com>\r
  54 Content-Type: text/plain; charset=ISO-8859-1\r
  55 X-Gm-Message-State:\r
  56  ALoCoQmgh88oNhdnhZ0OdYuG/IE/dPQ0sVdFR786TlxZVSxAeiwd1TQayJOUeT8yiCbpmju12E/6\r
  57 Cc: notmuch@notmuchmail.org\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Sat, 31 Mar 2012 23:36:30 -0000\r
  71 \r
  72 Hi Mark,\r
  73 \r
  74 On Sat, Mar 31, 2012 at 03:13, Mark Walters <markwalters1009@gmail.com> wrote:\r
  75 > Secondly it seems a little odd that the second patch above changes the\r
  76 > header stuff from 'From to "From" and then this patch changes it back\r
  77 > again. This is probably only a matter of a tidy history but I have to\r
  78 > admit I am confused about why the reply from alternate address still\r
  79 > works (but it does seem to)\r
  80 \r
  81 Good question. These three bugs (alternate address, References header,\r
  82 double headers in 23.2) basically stem from two problems: poor\r
  83 interactions between notmuch and message.el, and inconsistency in\r
  84 using strings and symbols for header names.\r
  85 \r
  86 All the message.el functions usually assume symbols for header names,\r
  87 but when I was writing the new reply stuff I found it easier to use\r
  88 strings. In emacs 23.3, message.el is happy to accept strings. The\r
  89 alternate address bug was caused by the fact that we were using\r
  90 strings in one place and symbols in another, and 'From isn't equal to\r
  91 "From". Rather than changing everything to symbols, I just changed the\r
  92 one place we were using symbols to use strings.\r
  93 \r
  94 But, it turns out that emacs 23.2 isn't happy with strings, which\r
  95 caused the double headers bug. So I ended up converting everything to\r
  96 use symbols. Because we're now consistent in using symbols, and never\r
  97 pass strings to message.el, the alternate address bug is still fixed.\r
  98 \r
  99 All that's a long way of saying: the fix for emacs 23.2 should\r
 100 actually fix the alternate reply bug as well. This probably means I\r
 101 could drop the alternate reply bug patch, and rebase the other two\r
 102 onto master, and we'd still have all the bugs fixed. Is that\r
 103 preferable, to reduce the number of code changes in the history?\r
 104 \r
 105 -- Adam\r