[PATCH] tevent glib event loop glue

Simo simo at samba.org
Sat Feb 13 12:51:54 UTC 2016


On Sat, 2016-02-13 at 09:38 +0100, Stefan Metzmacher wrote:
> Hi Ralph,
> 
> > Attached is a patchset that adds support for polling a glib
> > g_main_context from tevent. Later commits make use of it in the
> > mdsvc
> > RPC server.
> > 
> > The second patch adds a test binary that can be used to run query
> > Gnome Tracker either using the native glib event loop or tevent.
> > 
> > I'm still unsure whether we want this added directly to tevent at
> > this
> > early stage, or instead better put it to source3/lib/. It makes no
> > difference for Samba internal consumers.
> 
> I'd prefer to keep in Samba only for now (as
> lib/util/tevent_glib_glue.[ch])
> and adding a 'samba_' prefix to the public function,so that we won't
> conflict,
> if we later also add this to tevent.
> 
> This also means that you need to remove #include "tevent_internal.h"
> and use <tevent.h> as it might come from the system.
> and use DEBUG() instead of tevent_debug(), but still use
> #undef DBGC_CLASS
> #define DBGC_CLASS DBGC_TEVENT

Is there any chance we can build glib support into a loadable module
rather than having to compile it into samba or not ?

I would rather not increase the dependency tree for a normal install if
possible ...

Simo.



More information about the samba-technical mailing list