On Thu, May 05, 2011 at 12:58:24PM -0400, Stephen Gallagher wrote: > On Wed, 2011-05-04 at 17:00 +0200, Jakub Hrozek wrote: > > On 05/02/2011 08:28 PM, Stephen Gallagher wrote: > > > These patches are dependent on the patch provided in my earlier email > > > "Override config file debug_level with command-line" > > > > > > Patch 0002: Create common sss_monitor_init() > > > > > > This was implemented almost identically for both the responders > > > and the providers. It is easier to maintain as a single routine. > > > > > > This patch also adds the ability to provide a private context to > > > attach to the sbus_connection for later use. > > > > > > > > > > I think that sss_monitor_init() should either store conn into *mon_conn > > or we should remove the struct sbus_connection* from structures that > > contain them - they appear not to be used anywhere. > > > > Thanks for catching that. I forgot to assign it to the return variable. > > I want the structs to have access to the monitor context, because at > some point it may be desirable for children of the monitor to query it, > rather than have all contact originate from the monitor. > > New patches attached. (Patch 0002 is unchanged) >
Ack to both _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
