Re: [Freeipa-devel] [PATCH] Allow ipa-replica-conncheck and ipa-adtrust-install to read krb5 includedir

2013-02-18 Thread Petr Viktorin

On 02/10/2013 10:22 PM, Alexander Bokovoy wrote:

On Fri, 08 Feb 2013, Jakub Hrozek wrote:

The krb5 includedir uses a different delimeter (space) than the rest
of the krb5
config file (equal sign). But only the ipa-client-install and
ipa-server-install scripts were set with the correct delimeters. This
patch also adds the delimeters to ipa-adtrust-install and
ipa-replica-conncheck.

ACK. Works for me on Fedora 18 with git master.

I've also checked RHEL 6.4 beta but since both original fix and this one
are missing there, no changes were required to have ipa-adtrust-install
working with 'includedir' defined in /etc/krb5.conf.




This has been ACKed for a week, could someone with commit rights push it?

--
PetrĀ³

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] Allow ipa-replica-conncheck and ipa-adtrust-install to read krb5 includedir

2013-02-18 Thread Martin Kosek
On 02/10/2013 10:22 PM, Alexander Bokovoy wrote:
 On Fri, 08 Feb 2013, Jakub Hrozek wrote:
 The krb5 includedir uses a different delimeter (space) than the rest of the 
 krb5
 config file (equal sign). But only the ipa-client-install and
 ipa-server-install scripts were set with the correct delimeters. This
 patch also adds the delimeters to ipa-adtrust-install and
 ipa-replica-conncheck.
 ACK. Works for me on Fedora 18 with git master.
 
 I've also checked RHEL 6.4 beta but since both original fix and this one
 are missing there, no changes were required to have ipa-adtrust-install
 working with 'includedir' defined in /etc/krb5.conf.
 

Pushed to master, ipa-3-1.

Martin

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel