MIME-Version: 1.0
In-Reply-To: <874nshm9yo.fsf@gmail.com>
References: <1334752753-23970-1-git-send-email-felipe.contreras@gmail.com>
	<1334752753-23970-2-git-send-email-felipe.contreras@gmail.com>
	<873981chpj.fsf@nikula.org>
	<CAMP44s2Ukz=kDaPHYG+jzR3pDfZGP+6Ojqc4Xz5J8PxEd9fKcQ@mail.gmail.com>
	<874nshm9yo.fsf@gmail.com>
Date: Wed, 18 Apr 2012 17:11:22 +0300
Message-ID:
 <CAMP44s0qHWGAiGWbKqa3pTJWbpvshoNpoc6EezJmDjE8xQ3ZYA@mail.gmail.com>
Subject: Re: [PATCH v2 1/3] Add 'compose' command
From: Felipe Contreras <felipe.contreras@gmail.com>
To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Cc: notmuch@notmuchmail.org
Precedence: list

On Wed, Apr 18, 2012 at 4:43 PM, Dmitry Kurochkin
<dmitry.kurochkin@gmail.com> wrote:
> Hi Felipe.
>
> Felipe Contreras <felipe.contreras@gmail.com> writes:
>
>> On Wed, Apr 18, 2012 at 4:06 PM, Jani Nikula <jani@nikula.org> wrote:
>>
>>> Running "notmuch compose" more than once within a second would result i=
n
>>> identical message ids for the messages, which is not a good idea. That'=
s
>>> not likely in interactive use, but the notmuch cli is highly scriptable=
,
>>> so someone is bound to hit this.
>>>
>>> Some paranoid might also be worried about "leaking" the time you run
>>> "notmuch compose"... which may be different from the actual time you
>>> send the message.
>>
>> It's still better than the current situation; nothing. In any case,
>> people that have not needed this would not be affected; their UI would
>> override the Message-ID.
>>
>
> I disagree. =C2=A0If notmuch CLI generates a Message-ID, it must be a goo=
d
> one. =C2=A0Otherwise we make users falsely believe that they do have a pr=
oper
> Message-ID while in fact they do not. =C2=A0And that would bite them soon=
er
> or later.

And then they'll report it, and we would fix it.

Anyway, everything comes from a patch, so, do you have a patch,
pseudo-code, or even a suggestion?

--=20
Felipe Contreras