database error
[notmuch-archives.git] / c0 / 9e5a43bff56e66b0f4b294eb428d1c7ef57b4c
1 Return-Path: <tomi.ollila@iki.fi>\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 F10C6431FB6\r
6         for <notmuch@notmuchmail.org>; Mon, 10 Sep 2012 01:52:01 -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 W1MW3rLfxk1A for <notmuch@notmuchmail.org>;\r
16         Mon, 10 Sep 2012 01:52:00 -0700 (PDT)\r
17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
18         by olra.theworths.org (Postfix) with ESMTP id 4920E431FAE\r
19         for <notmuch@notmuchmail.org>; Mon, 10 Sep 2012 01:52:00 -0700 (PDT)\r
20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
21         by guru.guru-group.fi (Postfix) with ESMTP id C867210036F;\r
22         Mon, 10 Sep 2012 11:52:05 +0300 (EEST)\r
23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
24 To: Michal Nazarewicz <mpn@google.com>, notmuch@notmuchmail.org\r
25 Subject: Re: [PATCHv3] notmuch-show: include Bcc header in json output\r
26 In-Reply-To:\r
27  <7b1848a5828820912e3f7b960528b0dbc6a8b361.1347258748.git.mina86@mina86.com>\r
28 References: <m2zk50oiq6.fsf@guru.guru-group.fi>\r
29         <7b1848a5828820912e3f7b960528b0dbc6a8b361.1347258748.git.mina86@mina86.com>\r
30 User-Agent: Notmuch/0.13.2+162~ged38330 (http://notmuchmail.org) Emacs/24.2.1\r
31         (x86_64-unknown-linux-gnu)\r
32 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
33         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
34         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
35 Date: Mon, 10 Sep 2012 11:52:05 +0300\r
36 Message-ID: <m2d31u6z0q.fsf@guru.guru-group.fi>\r
37 MIME-Version: 1.0\r
38 Content-Type: text/plain\r
39 X-BeenThere: notmuch@notmuchmail.org\r
40 X-Mailman-Version: 2.1.13\r
41 Precedence: list\r
42 List-Id: "Use and development of the notmuch mail system."\r
43         <notmuch.notmuchmail.org>\r
44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
47 List-Post: <mailto:notmuch@notmuchmail.org>\r
48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
51 X-List-Received-Date: Mon, 10 Sep 2012 08:52:02 -0000\r
52 \r
53 On Mon, Sep 10 2012, Michal Nazarewicz <mpn@google.com> wrote:\r
54 \r
55 > From: Michal Nazarewicz <mina86@mina86.com>\r
56 >\r
57 > With this change, emacs users can use notmuch-message-headers\r
58 > variable to configure notmuch-show display Bcc header.\r
59 > ---\r
60 \r
61 Thanks, that explains (also why my attempts to show Message-ID has failed ;)\r
62 \r
63 I applied the patch, run tests (which pass) and tested the feature --\r
64 The Bcc field is indeed shown.\r
65 \r
66 The only annoyance for me is that in the file I picked to test had:\r
67 \r
68 BCC: user1@host.tld, user2@host.tld,\r
69  user3@host.tld, user4@host.tld\r
70 \r
71 notmuch displayed this content as\r
72 \r
73  Bcc: user1@host.tld, user2@host.tld,\r
74 user3@host.tld, user4@host.tld\r
75 \r
76 I think this should be "fixed" (somehow (*)) in the future, but this\r
77 should not hold this patch.\r
78 \r
79 +1\r
80 \r
81 Tomi\r
82 \r
83 (*) How that is done is another issue, should the cli or emacs do it,\r
84 (i.e. which interface modifies spacing information) and should this be done\r
85 generally to all headers. Is the solution to change all whitespace to one\r
86 space or indent subsequent on screen...\r
87 \r
88 >  notmuch-show.c   |    7 +++++++\r
89 >  test/json        |    8 ++++----\r
90 >  test/test-lib.sh |    5 +++++\r
91 >  3 files changed, 16 insertions(+), 4 deletions(-)\r
92 >\r
93 > diff --git a/notmuch-show.c b/notmuch-show.c\r
94 > index 3556293..0b7abf1 100644\r
95 > --- a/notmuch-show.c\r
96 > +++ b/notmuch-show.c\r
97 > @@ -233,6 +233,13 @@ format_headers_json (sprinter_t *sp, GMimeMessage *message,\r
98 >       sp->string (sp, recipients_string);\r
99 >      }\r
100 >  \r
101 > +    recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_BCC);\r
102 > +    recipients_string = internet_address_list_to_string (recipients, 0);\r
103 > +    if (recipients_string) {\r
104 > +     sp->map_key (sp, "Bcc");\r
105 > +     sp->string (sp, recipients_string);\r
106 > +    }\r
107 > +\r
108 >      if (reply) {\r
109 >       sp->map_key (sp, "In-reply-to");\r
110 >       sp->string (sp, g_mime_object_get_header (GMIME_OBJECT (message), "In-reply-to"));\r
111 > diff --git a/test/json b/test/json\r
112 > index ac8fa8e..40d0ba3 100755\r
113 > --- a/test/json\r
114 > +++ b/test/json\r
115 > @@ -3,18 +3,18 @@ test_description="--format=json output"\r
116 >  . ./test-lib.sh\r
117 >  \r
118 >  test_begin_subtest "Show message: json"\r
119 > -add_message "[subject]=\"json-show-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=\"json-show-message\""\r
120 > +add_message "[subject]=\"json-show-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[bcc]=\"test_suite+bcc@notmuchmail.org\"" "[body]=\"json-show-message\""\r
121 >  output=$(notmuch show --format=json "json-show-message")\r
122 > -test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}, \"body\": [{\"id\": 1, \"content-type\": \"text/plain\", \"content\": \"json-show-message\n\"}]}, []]]]"\r
123 > +test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Bcc\": \"test_suite+bcc@notmuchmail.org\", \"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}, \"body\": [{\"id\": 1, \"content-type\": \"text/plain\", \"content\": \"json-show-message\n\"}]}, []]]]"\r
124 >  \r
125 >  # This should be the same output as above.\r
126 >  test_begin_subtest "Show message: json --body=true"\r
127 >  output=$(notmuch show --format=json --body=true "json-show-message")\r
128 > -test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}, \"body\": [{\"id\": 1, \"content-type\": \"text/plain\", \"content\": \"json-show-message\n\"}]}, []]]]"\r
129 > +test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Bcc\": \"test_suite+bcc@notmuchmail.org\",\"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}, \"body\": [{\"id\": 1, \"content-type\": \"text/plain\", \"content\": \"json-show-message\n\"}]}, []]]]"\r
130 >  \r
131 >  test_begin_subtest "Show message: json --body=false"\r
132 >  output=$(notmuch show --format=json --body=false "json-show-message")\r
133 > -test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}}, []]]]"\r
134 > +test_expect_equal_json "$output" "[[[{\"id\": \"${gen_msg_id}\", \"match\": true, \"excluded\": false, \"filename\": \"${gen_msg_filename}\", \"timestamp\": 946728000, \"date_relative\": \"2000-01-01\", \"tags\": [\"inbox\",\"unread\"], \"headers\": {\"Subject\": \"json-show-subject\", \"From\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"To\": \"Notmuch Test Suite <test_suite@notmuchmail.org>\", \"Bcc\": \"test_suite+bcc@notmuchmail.org\",\"Date\": \"Sat, 01 Jan 2000 12:00:00 +0000\"}}, []]]]"\r
135 >  \r
136 >  test_begin_subtest "Search message: json"\r
137 >  add_message "[subject]=\"json-search-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:00 -0000\"" "[body]=\"json-search-message\""\r
138 > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
139 > index f34b1fb..e14203d 100644\r
140 > --- a/test/test-lib.sh\r
141 > +++ b/test/test-lib.sh\r
142 > @@ -350,6 +350,11 @@ ${additional_headers}"\r
143 >  ${additional_headers}"\r
144 >      fi\r
145 >  \r
146 > +    if [ ! -z "${template[bcc]}" ]; then\r
147 > +     additional_headers="Bcc: ${template[bcc]}\r
148 > +${additional_headers}"\r
149 > +    fi\r
150 > +\r
151 >      if [ ! -z "${template[references]}" ]; then\r
152 >       additional_headers="References: ${template[references]}\r
153 >  ${additional_headers}"\r
154 > -- \r
155 > 1.7.7.3\r
156 >\r
157 > _______________________________________________\r
158 > notmuch mailing list\r
159 > notmuch@notmuchmail.org\r
160 > http://notmuchmail.org/mailman/listinfo/notmuch\r