Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-26 Thread Jakub Hrozek
On Thu, Jan 26, 2012 at 06:39:40PM +0100, Jan Zelený wrote: > #0001: > Still Ack > > #0002: > Ack > > #0003: > I'm not sure why you moved the declarations of sysdb_svc_add() and > sysdb_svc_remove_alias() into this patch, but I guess it's just a cosmetic > issue, so I have no problem with that.

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-26 Thread Stephen Gallagher
On Thu, 2012-01-26 at 18:39 +0100, Jan Zelený wrote: > #0001: > Still Ack > > #0002: > Ack > > #0003: > I'm not sure why you moved the declarations of sysdb_svc_add() and > sysdb_svc_remove_alias() into this patch, but I guess it's just a cosmetic > issue, so I have no problem with that. > In

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-26 Thread Jan Zelený
#0001: Still Ack #0002: Ack #0003: I'm not sure why you moved the declarations of sysdb_svc_add() and sysdb_svc_remove_alias() into this patch, but I guess it's just a cosmetic issue, so I have no problem with that. Ack #0004: I've got another segfault for you ;-) You fixed the original but y

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-25 Thread Stephen Gallagher
On Wed, 2012-01-25 at 18:31 +0100, Jan Zelený wrote: > > On Tue, 2012-01-24 at 16:48 -0500, Stephen Gallagher wrote: > > > On Tue, 2012-01-24 at 11:19 -0500, Stephen Gallagher wrote: > > > > On Tue, 2012-01-24 at 16:03 +0100, Jakub Hrozek wrote: > > > > > On Mon, Jan 23, 2012 at 10:12:15PM -0500, S

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-25 Thread Jan Zelený
> On Tue, 2012-01-24 at 16:48 -0500, Stephen Gallagher wrote: > > On Tue, 2012-01-24 at 11:19 -0500, Stephen Gallagher wrote: > > > On Tue, 2012-01-24 at 16:03 +0100, Jakub Hrozek wrote: > > > > On Mon, Jan 23, 2012 at 10:12:15PM -0500, Stephen Gallagher wrote: > > > > > On Sat, 2012-01-21 at 16:54

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-24 Thread Stephen Gallagher
On Tue, 2012-01-24 at 16:48 -0500, Stephen Gallagher wrote: > On Tue, 2012-01-24 at 11:19 -0500, Stephen Gallagher wrote: > > On Tue, 2012-01-24 at 16:03 +0100, Jakub Hrozek wrote: > > > On Mon, Jan 23, 2012 at 10:12:15PM -0500, Stephen Gallagher wrote: > > > > On Sat, 2012-01-21 at 16:54 -0500, St

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-24 Thread Stephen Gallagher
On Tue, 2012-01-24 at 11:19 -0500, Stephen Gallagher wrote: > On Tue, 2012-01-24 at 16:03 +0100, Jakub Hrozek wrote: > > On Mon, Jan 23, 2012 at 10:12:15PM -0500, Stephen Gallagher wrote: > > > On Sat, 2012-01-21 at 16:54 -0500, Stephen Gallagher wrote: > > > > These patches add support for getserv

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-24 Thread Jakub Hrozek
On Mon, Jan 23, 2012 at 10:12:15PM -0500, Stephen Gallagher wrote: > On Sat, 2012-01-21 at 16:54 -0500, Stephen Gallagher wrote: > > These patches add support for getservbyname() and getservbyport() to > > the NSS responder. They do not yet support enumeration of services > > through setservent(),

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-23 Thread Stephen Gallagher
On Sun, 2012-01-22 at 02:20 -0500, Simo Sorce wrote: > On Sat, 2012-01-21 at 16:54 -0500, Stephen Gallagher wrote: > > Patch 0001: Make the sized_string structure available elsewhere in the > > source. > > ACK. > > > Patch 0002: Add support for strtouint16 (needed for managing the port > > number

Re: [SSSD] [PATCHES][PRELIMINARY] Support the 'services' NSS map in SSSD

2012-01-21 Thread Simo Sorce
On Sat, 2012-01-21 at 16:54 -0500, Stephen Gallagher wrote: > These patches add support for getservbyname() and getservbyport() to > the NSS responder. They do not yet support enumeration of services > through setservent(), getservent() and endservent(). These patches also > include a reference im