[PATCH] more PY2/PY3 compat changes

Andrew Bartlett abartlet at samba.org
Wed Jul 4 19:04:42 UTC 2018


On Wed, 2018-07-04 at 16:22 +0200, Andreas Schneider via samba-
technical wrote:
> On Wednesday, 27 June 2018 16:38:38 CEST Noel Power via samba-technical wrote:
> > Hi,
> > 
> > More hopefully low hanging fruit patches, the exception I would see as
> > '[PATCH 05/26] lib/ldb: Implement a bytes derived object for attributes
> > py2/py3' which is the modification discussed on this list some time back
> > (see https://lists.samba.org/archive/samba-technical/2018-May/127718.html)
> > 
> > This passed CI earlier, I squashed some patches and fixed some
> > whitespace issues and resubmitted (should pass again)
> > 
> > please see attached for patch and
> > https://gitlab.com/samba-team/devel/samba/pipelines/24688131 for CI
> > 
> > Branch is at
> > https://gitlab.com/samba-team/devel/samba/commits/npower-python3-misc%232
> 
> I've reviewed them with some minor cosmetic changes. Attached is my patchset 
> with my RB+.
> 
> Attaching it here if someone else wants to comment.

Thanks Andreas,

I would like Joe to take a look over this and comment.  

My understanding is that he found some issues around the str() handling
but I distracted him onto customer work before he could report back. 

I'm just as keen to keep this moving, but please hold off for just a
day or two while we get these ducks in a row.

Thanks,

Andrew Bartlett

> 
> Please push.
> 
> 
-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list