pytevent improvements & Python 3 port

Andrew Bartlett abartlet at samba.org
Tue Jun 9 22:06:25 MDT 2015


On Thu, 2015-06-04 at 12:29 +0000, Jelmer Vernooij wrote:
> On Tue, Jun 02, 2015 at 03:14:39PM +0200, Petr Viktorin wrote:
> > On 05/26/2015 02:07 PM, Petr Viktorin wrote:
> > > Hello,
> > > 
> > > The first patch is a fix for a bug in the buildsystem changes for py3.
> > > (It only occurs when building for two Pythons, so Samba is safe.)
> > > 
> > > I've played a bit with the pytevent wrapper, and found that the types
> > > for timers and FDs are not defined, so, for example, trying to print
> > > them out segfaults:
> > > 
> > >     >>> import tevent
> > >     >>> ctx = tevent.Context()
> > >     >>> timer = ctx.add_timer(0, lambda t: None)
> > >     >>> timer
> > >     Segmentation fault (core dumped)
> > > 
> > > I'm sending a quick fix for FDs, and a more comprehensive one for timers.
> > > 
> > > Also, "Context.add_timer" requires passing a struct timeval cast to a
> > > long inside a Python integer, which is very hard to do portably from
> > > Python. I've added "add_timer_offset", which takes a number of seconds
> > > from now.
> > > 
> > > I've also ported pytevent to Python 3. The library doesn't deal with
> > > strings much, so the port is trivial.
> > 
> > Ping, could someone find the time to look at these patches?
> Thanks, this looks good to me. Reviewed-By: me
> 
> Can somebody else also review and push?

Sorry for the unacceptable delay in looking at this.

Reviewed-by: me and pushed to autobuild.

Thanks!

Andrew Bartlett

-- 
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team         http://samba.org
Samba Development and Support, Catalyst IT   http://catalyst.net.nz/services/samba








More information about the samba-technical mailing list