On Thu, 2012-02-09 at 14:16 +0100, Jakub Hrozek wrote:
On Thu, Feb 09, 2012 at 11:55:03AM +0100, Jan Cholasta wrote:
> On 7.2.2012 17:24, Stephen Gallagher wrote:
> >On Tue, 2012-02-07 at 17:24 +0100, Jan Cholasta wrote:
> >>Dne 7.2.2012 16:35, Stephen Gallagher napsal(a):
> >>>On Tue, 2012-02-07 at 16:34 +0100, Jan Cholasta wrote:
> >>>>Hi,
> >>>>
> >>>>while working on ipa-client-install changes that reflect the
addition of
> >>>>my SSH work to SSSD, I have noticed that the new (autofs, sudo,
ssh)
> >>>>services are not listed in sssd.api.conf.
> >>>>
> >>>>As I need SSSDConfig in ipa-client-install to support the ssh
service, I
> >>>>have made a patch that fixes this issue. See attachment.
> >>>
> >>>
> >>>Nack. This list is the set of default services we're running. Since
> >>>these features are still experimental, they should not be enabled by
> >>>default.
> >>>
> >>
> >>Well, what can I say. Sorry for submitting patches based on wrong
> >>assumptions :-)
> >>
> >>Correct patch attached.
> >
> >Not a problem. New patch looks good. Ack.
>
> I forgot to update the tests. Updated patch attached.
>
> --
> Jan Cholasta
make check passes now. Ack.
Pushed to master and sssd-1-8.