[PATCH 2 of 2 STABLE] py3: use native strings when forming email headers in patchbomb

Yuya Nishihara yuya at tcha.org
Mon Oct 28 10:57:38 EDT 2019


On Mon, 28 Oct 2019 10:09:22 +0100, Denis Laxalde wrote:
> Yuya Nishihara a écrit :
> > On Fri, 25 Oct 2019 15:02:11 +0200, Denis Laxalde wrote:
> > > # HG changeset patch
> > > # User Denis Laxalde <denis at laxalde.org>
> > > # Date 1572008488 -7200
> > > #      Fri Oct 25 15:01:28 2019 +0200
> > > # Branch stable
> > > # Node ID 6058175493d062dbaec2cb0e897e49da4596d147
> > > # Parent  17fad3f1b4742d75db23e17a04c85223bcc239a2
> > > py3: use native strings when forming email headers in patchbomb
> > 
> > I feel this change is too big to be in stable.
> > 
> > > -        # Fix up all headers to be native strings.
> > > -        # TODO(durin42): this should probably be cleaned up above in the future.
> > > -        if pycompat.ispy3:
> > > -            for hdr, val in list(m.items()):
> > 
> > Can't we somehow get around the issue here?
> 
> Well, I'm not sure I'd be interested in implementing a workaround,
> especially here since the change aims at removing this TODO while making
> the code Python 3 compatible. Fixing the issue is a side effect somehow.

Okay, then let's move the series to default branch if you aren't interested
in minimizing the stable changes.

There are too much py3 changes after the freeze.


More information about the Mercurial-devel mailing list