[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#57021] [PATCH] gnu: packages: Add nmail.
From: |
Christina |
Subject: |
[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@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@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
>>