Re: b4 bugs / future improvements

[Maxime Ripard <maxime@cerno.tech>]

Hi,

On Thu, Oct 20, 2022 at 10:49:30AM +0200, Maxime Ripard wrote:
> On Wed, Oct 19, 2022 at 10:01:40AM +0200, Maxime Ripard wrote:
> > On Tue, Oct 18, 2022 at 04:18:04PM -0400, Konstantin Ryabitsev wrote:
> > > > There's a bunch of things I have noticed in the recent weeks:
> > > > 
> > > >  - It looks like 87e0e464959c breaks msmtp which doesn't recognize the
> > > >    -f option, and will just error out
> > > 
> > > There's got to be something else going on, since -f is required for sendmail
> > > compatibility. The msmtp manpage specifically mentions that is supports it:
> > > https://manpages.ubuntu.com/manpages/trusty/man1/msmtp.1.html
> > > 
> > > (see under "options specific to sendmail mode")
> > 
> > It might be the auto argument that confuses it then? I'll check.
> 
> So, here's the behaviour I'm seeing:
> 
> $ printf "From: <maxime@cerno.tech>\nTo: <maxime@cerno.tech>\nSubject: Test\n\nTest Test." | msmtp -i -f auto maxime@cerno.tech
> msmtp: recipient address maxime@cerno.tech not accepted by the server
> msmtp: server message: 504 5.5.2 <auto>: Sender address rejected: need fully-qualified address
> msmtp: could not send mail (account default from /home/max/.config/msmtp/config)
> 
> If I drop the -f auto, it works:
> 
> $ printf "From: <maxime@cerno.tech>\nTo: <maxime@cerno.tech>\nSubject: Test\n\nTest Test." | msmtp -i maxime@cerno.tech
> $ echo $?
> 0
> 
> It seems to be related to
> https://github.com/marlam/msmtp-mirror/issues/51 except I don't have
> set_from_header in my config. I've tried to use allow_from_override and
> force it to off, and it didn't help.

I just discovered something else.

For various reasons, on one of my series I keep a per-commit changelog,
in the commit log after a ---

This works generally fine, until I run b4 trailers -u which seems to
drop the entire content after --- when adding the tags.

Maxime