<div dir="ltr">Thanks. Updated patch.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 14, 2014 at 3:49 AM, Petr Viktorin <span dir="ltr"><<a href="mailto:pviktori@redhat.com" target="_blank">pviktori@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 05/09/2014 05:09 AM, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
        Came across this issue in my environment the other day and<br>
thought I would send a quick patch for review for<br>
<a href="https://fedorahosted.org/freeipa/ticket/3733" target="_blank">https://fedorahosted.org/<u></u>freeipa/ticket/3733</a><br>
</blockquote>
<br></div></div>
Thanks for the patch!<br>
<br>
This works, but configure_nsswitch would now fail if it was called options.sssd=True, as nss_value won't be set.<br>
Please remove the if entirely, and perhaps mention in the function's docstring/name that it's for non-SSSD setups only.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Petrł<br>
<br>
</font></span></blockquote></div><br></div>