[PATCH] Port of samba.messaging to Python 3

Andrew Bartlett abartlet at samba.org
Fri Sep 15 03:58:29 UTC 2017


On Thu, 2017-09-14 at 20:02 +1200, Andrew Bartlett wrote:
> On Thu, 2017-09-14 at 09:50 +0200, Lumir Balhar via samba-technical
> wrote:
> > Hello.
> > 
> > I've made another module - samba.messaging - Python 3 compatible. The 
> > messaging module already has good test suite so I just ported it as well.
> 
> I agree, it has a good testsuite.  It is lucky for you that you waited,
> it was much less complete (or even functional!) just a few months ago. 
> 
> Reviewed-by: Andrew Bartlett <abartlet at samba.org>
> 
> Collect a second team reviewer and I'm sure this can join our autobuild
> queue!

Sadly it failed with:

[2149/4224] Compiling source4/lib/messaging/pymessaging.c
../source4/lib/messaging/pymessaging.c:496:2: error: missing braces
around initializer [-Werror=missing-braces]
  PyObject_HEAD_INIT(NULL) 0,
  ^
../source4/lib/messaging/pymessaging.c:496:2: error: (near
initialization for imessaging_Type.ob_base.ob_base) [-Werror=missing-
braces]
cc1: all warnings being treated as errors
Waf: Leaving directory `/memdisk/abartlet/a/b521293/samba/bin'
Build failed:  -> task failed (err #1): 
        {task: cc pymessaging.c -> pymessaging_9.o}
make: *** [all] Error 1

Andrew Bartlett
-- 
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