-----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----- _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel