<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body>
<div class="moz-cite-prefix">On 04/05/2016 12:07 PM, Martin Babinsky
wrote:<br>
</div>
<blockquote cite="mid:57038E52.3090900@redhat.com" type="cite">On
04/05/2016 10:24 AM, Milan Kubík wrote:
<br>
<blockquote type="cite">On 04/05/2016 10:17 AM, Milan Kubík wrote:
<br>
<blockquote type="cite">On 04/05/2016 09:31 AM, Martin Babinsky
wrote:
<br>
<blockquote type="cite">On 04/01/2016 12:02 PM, Milan Kubík
wrote:
<br>
<br>
<blockquote type="cite">Patches attached.
<br>
<br>
<br>
<br>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/5733">https://fedorahosted.org/freeipa/ticket/5733</a>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
</blockquote>
Hi Milan,
<br>
<br>
<br>
<br>
I would be more happy if you could send a separate patch for
the context
<br>
manager fix, since the issue is orthogonal to the added test
case (even
<br>
if the test suite explodes without it).
<br>
<br>
<br>
<br>
Otherwise LGTM.
<br>
<br>
<br>
<br>
<br>
<br>
</blockquote>
<br>
Done. Patch 0035 now applies to all branches, context manager
fix
<br>
needs separate patch for ipa-4-2.
<br>
<br>
</blockquote>
Updated commit message in patches 0036 to include the ticket.
<br>
<br>
</blockquote>
Thanks, ACK.
<br>
<br>
</blockquote>
Add freeipa-devel back to the loop & push request :)<br>
<br>
<pre class="moz-signature" cols="72">--
Milan Kubik</pre>
</body>
</html>