From 520b0dcd1f48e0bc3f23593c3ae1dc6f9de56c78 Mon Sep 17 00:00:00 2001 From: Shea Levy Date: Mon, 15 Aug 2016 08:43:47 +2000 Subject: [PATCH] Re: [PATCH 0/4] Allow specifying alternate names for addresses in other_email --- 1b/3b86bfdd98aae55b509e46018bef1a12707ef5 | 107 ++++++++++++++++++++++ 1 file changed, 107 insertions(+) create mode 100644 1b/3b86bfdd98aae55b509e46018bef1a12707ef5 diff --git a/1b/3b86bfdd98aae55b509e46018bef1a12707ef5 b/1b/3b86bfdd98aae55b509e46018bef1a12707ef5 new file mode 100644 index 000000000..736dbd23f --- /dev/null +++ b/1b/3b86bfdd98aae55b509e46018bef1a12707ef5 @@ -0,0 +1,107 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by arlo.cworth.org (Postfix) with ESMTP id 9C2EA6DE880B + for ; Sun, 14 Aug 2016 05:43:58 -0700 (PDT) +X-Virus-Scanned: Debian amavisd-new at cworth.org +X-Spam-Flag: NO +X-Spam-Score: 0 +X-Spam-Level: +X-Spam-Status: No, score=0 tagged_above=-999 required=5 + tests=[RCVD_IN_DNSWL_NONE=-0.0001] autolearn=disabled +Received: from arlo.cworth.org ([127.0.0.1]) + by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id uyCOae8ZdMav for ; + Sun, 14 Aug 2016 05:43:50 -0700 (PDT) +Received: from smtprelay.hostedemail.com (smtprelay0036.hostedemail.com + [216.40.44.36]) + by arlo.cworth.org (Postfix) with ESMTPS id A22A46DE8809 + for ; Sun, 14 Aug 2016 05:43:50 -0700 (PDT) +Received: from filter.hostedemail.com (unknown [216.40.38.60]) + by smtprelay03.hostedemail.com (Postfix) with ESMTP id 26D596A49C; + Sun, 14 Aug 2016 12:43:48 +0000 (UTC) +X-Session-Marker: 7368656140736865616C6576792E636F6D +X-Spam-Summary: 2, 0, 0, , d41d8cd98f00b204, shea@shealevy.com, :::, + RULES_HIT:41:355:379:599:800:960:973:981:988:989:1260:1277:1313:1314:1345:1359:1381:1437:1516:1518:1534:1541:1711:1730:1747:1777:1792:2198:2199:2393:2553:2559:2562:2693:2895:3138:3139:3140:3141:3142:3353:3622:3865:3866:3867:3868:3870:3871:3872:3873:3874:4321:5007:6117:6248:6261:7903:7974:10004:10044:10400:10450:10455:10848:11232:11658:11914:12043:12517:12519:12555:12663:13069:13071:13072:13311:13357:13846:14180:14181:14721:14777:19904:19999:21060:21080:21325:21433:21451:30003:30012:30022:30029:30030:30054:30080:30090:30091, 0, RBL:none, CacheIP:none, Bayesian:0.5, 0.5, 0.5, Netcheck:none, + DomainCache:0, MSF:not bulk, SPF:fn, MSBL:0, DNSBL:none, Custom_rules:0:0:0, + LFtime:2, LUA_SUMMARY:none +X-HE-Tag: ghost79_5b59c64055e41 +X-Filterd-Recvd-Size: 2448 +Received: from localhost (c-73-186-74-217.hsd1.nh.comcast.net [73.186.74.217]) + (Authenticated sender: shea@shealevy.com) + by omf12.hostedemail.com (Postfix) with ESMTPA; + Sun, 14 Aug 2016 12:43:47 +0000 (UTC) +From: Shea Levy +To: Jani Nikula , notmuch@notmuchmail.org +Subject: Re: [PATCH 0/4] Allow specifying alternate names for addresses in + other_email +In-Reply-To: <87k2fk98yh.fsf@nikula.org> +References: <1470776118-5070-1-git-send-email-shea@shealevy.com> + <87k2fk98yh.fsf@nikula.org> +User-Agent: Notmuch/0.22 (http://notmuchmail.org) Emacs/24.5.2 + (x86_64-unknown-linux-gnu) +Date: Sun, 14 Aug 2016 08:43:47 -0400 +Message-ID: + <87oa4vtta4.fsf@shlevy-laptop.i-did-not-set--mail-host-address--so-tickle-me> +MIME-Version: 1.0 +Content-Type: text/plain +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.20 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Sun, 14 Aug 2016 12:43:58 -0000 + +Hi Jani, + +Jani Nikula writes: + +> On Tue, 09 Aug 2016, Shea Levy wrote: +>> Currently, while notmuch-reply will recognize email addresses other than +>> the main address with user.other_email, it always sets the name part of +>> the address in the envelope-from and From headers to user.name. This +>> patchset enables specifying names on a per-address basis with a new +>> user.other_name property. Presumably other users of user.other_email +>> may want to use this as well, but those are not updated currently. +> +> I am not convinved by adding another configuration option, especially +> when it has to be in sync with another configuration option (ordering in +> user.other_name having to match user.other_email). I would much prefer +> allowing (but not requiring) "Name " style addresses +> both in user.primary_email and user.other_email. +> + +This would be fine with me. Is there already code in place to separate +out the name and email values from that kind of email address? + +> +> With a cursory glance at the implementation, I wonder if you could just +> pick the name based on the address you've picked earlier, and leave the +> address matching mostly as it is. Would save some passing of parameters +> around. Maybe. +> + +Hmm, I'm not sure what you mean by this, sorry. Can you expand? + +> +> Additionally, I'd very much like to have my series [1] merged +> first. It'll be *much* easier to rebase your series on top than the +> other way around... +> + +Happy to rebase my work on yours. + +> +> BR, +> Jani. +> +> +> [1] id:cover.1471088022.git.jani@nikula.org -- 2.26.2