GNU bug report logs - #57021
[PATCH] gnu: packages: Add nmail.

Previous Next

Full log

Message #57 received at 57021 <at> debbugs.gnu.org (full text, mbox):

From: Christina <cdo <at> mutix.org>
To: Mathieu <mlaparie <at> disr.it>
Cc: 57021 <at> debbugs.gnu.org
Subject: Re: bug#57021: [PATCH] gnu: packages: Add nmail.
Date: Mon, 22 Apr 2024 11:19:25 +0100

Hi Mathieu,

I'm testing it right now connecting to my server on IMAP STARTTLS.

It's working fine for me over IMAP and works over on SMTP over SSL if you receive this message.

Is --enable-login required for OAUTH?

I did notice it failed to build a second time due to missing dependencies which is odd. I'll send an update patch to fix that shortly.

Kind regards,
Christina  

On 2024-04-22 10:21 Mathieu <mlaparie <at> disr.it> wrote:

> Awesome news, thanks a ton Christina!
>  
> Have you tried sending emails without the `cyrus-sasl` patch? Unless
> the package changed since I submitted this patch, sending emails may
> fail because `cyrus-sasl` needs to be compiled with the
> `--enable-login` flag, which I think was not default. I haven't
> tested lately though because I was running `nmail` and `cyrus-sasl`
> from my private channel since a long time.
>  
> Mathieu
>  
> On 2024-04-21 16:03 Christina O'Donnell <cdo <at> mutix.org> wrote:
>  
>> Hi Mathieu Laparie,
>>  
>> Thank you for your patch. Sorry it has taken so long to get to it.
>>  
>> This patch looks good. It applies and runs and lints with few
>> issues.
>>  
>> I've re-rolled a version 9 patch on the latest master branch,
>> updating the version and changing the formatting slightly.
>>  
>> Regarding the second diff, I'm not sure what the justification is
>> for
>> changing cyrus-sasl in the same patch, so I've removed that part.
>> If
>> that change needs to happen then I would think it would make more
>> sense as a separate patch.
>>  
>> Kind regards, Christina
>>

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.