[SSSD] [PATCHES] Two patches related to SSSDDomain.remove_provider()

Stephen Gallagher sgallagh at redhat.com
Wed Dec 9 04:51:48 UTC 2009


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 12/08/2009 10:48 PM, Jakub Hrozek wrote:
> On 12/07/2009 11:04 PM, Stephen Gallagher wrote:
>> Patch 0001: remove_provider() really doesn't need the provider name
>> (ldap, krb5, etc.) specified in order to remove it. This patch
>> eliminates that argument to the function.
> 
> 
> ACK
> 
> Good change to the API.
> 
>> Patch 0002: remove_provider() needs to clean up any options that were
>> set for that provider. This patch does so, as well as ensuring that it
>> does not clean up options that may be needed by other providers (such as
>> krb5_realm, which may be used by the ldap provider)
> 
> 
> ACK but please fix whitespace errors before pushing.

Pushed to master. Whitespace errors were fixed by another patch.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAksfLOQACgkQeiVVYja6o6PlgACfZjuGwad3wonpCIaBdAN7KbSM
W+cAn2ZoLnC5kcRlz3hpq53bx5rlZD+S
=yOIY
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list