On Tue, Nov 24, 2015 at 02:36:17PM -0500, Simo Sorce wrote:
> On Tue, 2015-11-24 at 17:34 +0100, Jan Cholasta wrote:
> > On 24.11.2015 17:30, Simo Sorce wrote:
> > > On Tue, 2015-11-24 at 09:14 +0100, Jan Cholasta wrote:
> > >> On 24.11.2015 09:06, Petr Spacek wrote:
> > >>> On 24.11.2015 07:32, Jan Cholasta wrote:
> > >>>> On 23.11.2015 21:18, Simo Sorce wrote:
> > >>>>> Fixes #2203 by reading the server name from /etc/ipa/default.conf if 
> > >>>>> not
> > >>>>> provided on the command line.
> > >>>>>
> > >>>>> Simo.
> > >>>>
> > >>>> Just a thought: it would be nice if we had libipaconfig and used it 
> > >>>> everywhere
> > >>>> (the framework, ipa-getkeytab, certmonger, ...). I don't like that 
> > >>>> there are
> > >>>> at least 3 ipa config parser implementations now, each with its own 
> > >>>> quirks.
> > >>>
> > >>> Yeah, define a Augeas grammar for IPA config and call it. There is C and
> > >>> Python binding in RHEL 7.2 already.
> > >>
> > >> This won't be sufficient, as it should go beyond just parsing the config
> > >> file - there are dynamic defaults and backward compatiblity support
> > >> which should be the same everywhere too.
> > >
> > > This is a much bigger task, that someone keen on big refactoring may
> > > undertake, but I do not think we should delay this useful feature for
> > > that. It will be really easy to change this code to use whatever other
> > > ipa library when it materializes.
> > >
> > > For now this patch stands IMHO.
> > 
> > Sure, I'm not suggesting otherwise.
> 
> Rebased patch that includes proper spec file buildrequires.
> 
> Simo.
> 
Tested and works as expected (great usability win IMO).

Man page should be updated accordingly.

Cheers,
Fraser

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to