<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 01/21/2016 04:23 PM, Martin Kosek
wrote:<br>
</div>
<blockquote cite="mid:56A0F7EB.5050202@redhat.com" type="cite">
<pre wrap="">On 01/21/2016 04:22 PM, thierry bordaz wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On 01/21/2016 03:46 PM, Martin Kosek wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On 01/21/2016 01:37 PM, thierry bordaz wrote:
Thanks! Couple comments:
I miss ticket number of description.
</pre>
</blockquote>
<pre wrap="">
Thanks Martin for looking at it.
Ouch... the ticket number is <a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/4026">https://fedorahosted.org/freeipa/ticket/4026</a>
</pre>
<blockquote type="cite">
<pre wrap="">
Does this patch mean that all blocker on DS side preventing remote DNA were
fixed? If yes, it may be worth updating Requires in the spec file in that case
and making sure the backport is in Fedora 23+ or at least FreeIPA 4.3 COPR repo.
</pre>
</blockquote>
<pre wrap="">It required following 389-ds tickets that are now both fixed in 1.3.4.6
* <a class="moz-txt-link-freetext" href="https://fedorahosted.org/389/ticket/47779">https://fedorahosted.org/389/ticket/47779</a>
* <a class="moz-txt-link-freetext" href="https://fedorahosted.org/389/ticket/48362">https://fedorahosted.org/389/ticket/48362</a>
Is it the change to do in the spec file ?
diff --git a/freeipa.spec.in b/freeipa.spec.in
index 899af6c..87e80c9 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -130,7 +130,7 @@ Requires: %{name}-client = %{version}-%{release}
Requires: %{name}-admintools = %{version}-%{release}
Requires: %{name}-common = %{version}-%{release}
Requires: python2-ipaserver = %{version}-%{release}
-Requires: 389-ds-base >= 1.3.4.4
+Requires: 389-ds-base >= 1.3.4.6
Requires: openldap-clients > 2.4.35-4
Requires: nss >= 3.14.3-12.0
Requires: nss-tools >= 3.14.3-12.0
@@ -162,7 +162,7 @@ Requires: zip
Requires: policycoreutils >= 2.1.12-5
Requires: tar
Requires(pre): certmonger >= 0.78
-Requires(pre): 389-ds-base >= 1.3.4.4
+Requires(pre): 389-ds-base >= 1.3.4.6
Requires: fontawesome-fonts
Requires: open-sans-fonts
Requires: openssl
</pre>
</blockquote>
<pre wrap="">
That spec file change should be OK, thanks. I did not do other testing, I would
leave that up to other developers.
</pre>
</blockquote>
<font face="Times New Roman, Times, serif">Thanks for the review.
Here the second fix<br>
<br>
<br>
thierry<br>
</font>
</body>
</html>