<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 07/24/2015 09:46 AM, Lukas Slebodnik wrote:<br>
    <blockquote cite="mid:20150724074658.GA25827@mail.corp.redhat.com"
      type="cite">
      <div class="moz-text-plain" wrap="true" graphical-quote="true"
        style="font-family: -moz-fixed; font-size: 12px;"
        lang="x-unicode">
        <pre wrap="">ehlo,

patch is attached.

LS
</pre>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"><legend
          class="mimeAttachmentHeaderName">0001-SPEC-Update-spec-file-for-krb5_local_auth_plugin.patch</legend></fieldset>
      <br>
      <div class="moz-text-plain" wrap="true" graphical-quote="true"
        style="font-family: -moz-fixed; font-size: 12px;"
        lang="x-unicode">
        <pre wrap="">From 9474dfc64cbd73ab25bf12660f6b55e5563fe14c Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:lslebodn@redhat.com">&lt;lslebodn@redhat.com&gt;</a>
Date: Fri, 24 Jul 2015 09:24:31 +0200
Subject: [PATCH] SPEC: Update spec file for krb5_local_auth_plugin

krb5_localauth_plugin could be build only with MIT kerberos &gt;= 1.12.
However, this feature was backported in downstream to older version
of kerberos. So there were packaging failures

error: Installed (but unpackaged) file(s) found:
   /usr/lib/sssd/modules/sssd_krb5_localauth_plugin.so
RPM build errors:
    Installed (but unpackaged) file(s) found:
   /usr/lib/sssd/modules/sssd_krb5_localauth_plugin.so
Child returncode was: 1
EXCEPTION: Command failed. See logs for output.
---
 contrib/sssd.spec.in | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
index 15f7c582cba1b9052e180596625be7dd5749599f..cb3aab78616c6c8f9442fbf3d0944d4d140ff549 100644
--- a/contrib/sssd.spec.in
+++ b/contrib/sssd.spec.in
@@ -1,3 +1,4 @@
+%global rhel6_minor %(%{__grep} -o "6.[0-9]*" /etc/redhat-release |%{__sed} -s 's/6.//')
 %global rhel7_minor %(%{__grep} -o "7.[0-9]*" /etc/redhat-release |%{__sed} -s 's/7.//')
 
 %if 0%{?rhel} &amp;&amp; 0%{?rhel} &lt;= 6
@@ -41,7 +42,7 @@
     %global with_cifs_utils_plugin_option --disable-cifs-idmap-plugin
 %endif
 
-%if (0%{?fedora} &gt;= 21 || (0%{?rhel} == 7 &amp;&amp;  0%{?rhel7_minor} &gt;= 1))
+%if (0%{?fedora} || (0%{?rhel} == 7 &amp;&amp;  0%{?rhel7_minor} &gt;= 1) || (0%{?rhel} == 6 &amp;&amp;  0%{?rhel6_minor} &gt;= 7))
     %global with_krb5_localauth_plugin 1
 %endif
 
@@ -112,11 +113,7 @@ BuildRequires: pcre-devel
 BuildRequires: libxslt
 BuildRequires: libxml2
 BuildRequires: docbook-style-xsl
-%if (0%{?with_krb5_localauth_plugin} == 1)
-BuildRequires: krb5-devel &gt;= 1.12
-%else
 BuildRequires: krb5-devel
-%endif
 BuildRequires: c-ares-devel
 BuildRequires: python-devel
 %if (0%{?with_python3} == 1)
<div class="moz-txt-sig">-- 
2.4.3

</div></pre>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
    </blockquote>
    <br>
    Hi,<br>
    I looked at the patch and successfully built rpm on RHEL 6.6 and 6.7
    (and on Fedora 22). Then I used CI, result is there<br>
<a class="moz-txt-link-freetext" href="http://sssd-ci.duckdns.org/logs/commit/7c/fbe0ff1743a1939c8066175f1634842c58de66/1977/summary.html">http://sssd-ci.duckdns.org/logs/commit/7c/fbe0ff1743a1939c8066175f1634842c58de66/1977/summary.html</a><br>
    It is look good for me. So -- if nobody has comments -- ACK.<br>
    Petr<br>
  </body>
</html>