[Freeipa-devel] [PATCH] SPEC: Drop sssd from BuildRequires

[Alexander Bokovoy]

On Thu, 05 Nov 2015, Lukas Slebodnik wrote:
>On (04/11/15 18:12), Alexander Bokovoy wrote:
>>On Tue, 03 Nov 2015, Lukas Slebodnik wrote:
>>>>From c466be0e769a80fe204fc70675b8fa0ea3efb7b2 Mon Sep 17 00:00:00 2001
>>>From: Lukas Slebodnik <lslebodn at redhat.com>
>>>Date: Mon, 2 Nov 2015 19:52:57 +0000
>>>Subject: [PATCH] SPEC: Drop sssd from BuildRequires
>>>
>>>Packaging of sssd was changed and more sub-packages are build
>>>from sssd.src.rpm. Especially python bindings and development packages
>>>are already in sub-packages. As a result of this change the meta package
>>>sssd can be removed from BuildRequires without any problem.
>>>
>>>FreeIPA spec file contained build requirement for latest version of
>>>sssd even though the latest sssd was not required for building
>>>FreeIPA rpms. In many cases, it was sufficient just to change requirements
>>>for FreeIPA packages instead of build requirements.
>>ACK -- we have explicit requirements to separate SSSD components
>>(libsss_idmap, libsss_nss_idmap, python-libipa_hbac, to name a few).
>>
>Thank you very much for review.
>
>>Perhaps, telling that your goal is to build without SSSD is somewhat
>>misleading -- SSSD subpackages will be installed anyway and at least
>>1.12.1 will be required.
>>
>Did we test the same patch?
>
>I cannot see installed sssd in build root on rawhide
>http://paste.fedoraproject.org/287115/46707060
>
>[build at 526468c6ceac rpms]$ grep sssd root.log
>[build at 526468c6ceac rpms]$ grep 1.13.1-4.fc24 root.log
>DEBUG util.py:393:   libipa_hbac   x86_64 1.13.1-4.fc24
>fedora  73 k
>DEBUG util.py:393:   libsss_idmap  x86_64 1.13.1-4.fc24
>fedora  77 k
>DEBUG util.py:393:   libsss_idmap-devel   x86_64 1.13.1-4.fc24
>fedora 140 k
>DEBUG util.py:393:   libsss_nss_idmap  x86_64 1.13.1-4.fc24
>fedora  77 k
>DEBUG util.py:393:   libsss_nss_idmap-devel   x86_64 1.13.1-4.fc24
>fedora 125 k
>DEBUG util.py:393:   python-libipa_hbac   x86_64 1.13.1-4.fc24
>fedora  68 k
>DEBUG util.py:393:  [SKIPPED] python-libipa_hbac-1.13.1-4.fc24.x86_64.rpm:
>Already downloaded
>DEBUG util.py:393:  [SKIPPED] libipa_hbac-1.13.1-4.fc24.x86_64.rpm: Already
>downloaded
>DEBUG util.py:393:  [SKIPPED] libsss_idmap-devel-1.13.1-4.fc24.x86_64.rpm:
>Already downloaded
>DEBUG util.py:393:  [SKIPPED] libsss_idmap-1.13.1-4.fc24.x86_64.rpm: Already
>downloaded
>DEBUG util.py:393:  [SKIPPED] libsss_nss_idmap-devel-1.13.1-4.fc24.x86_64.rpm:
>Already downloaded
>DEBUG util.py:393:  [SKIPPED] libsss_nss_idmap-1.13.1-4.fc24.x86_64.rpm:
>Already downloaded
>DEBUG util.py:393: Installing  : libsss_nss_idmap-1.13.1-4.fc24.x86_64
>91/241
>DEBUG util.py:393: Installing  : libsss_idmap-1.13.1-4.fc24.x86_64
>92/241
>DEBUG util.py:393: Installing  : libipa_hbac-1.13.1-4.fc24.x86_64
>195/241
>DEBUG util.py:393: Installing  : python-libipa_hbac-1.13.1-4.fc24.x86_64
>212/241
>DEBUG util.py:393: Installing  : libsss_idmap-devel-1.13.1-4.fc24.x86_64
>224/241
>DEBUG util.py:393: Installing  : libsss_nss_idmap-devel-1.13.1-4.fc24.x86_64
>225/241
>DEBUG util.py:393: Verifying   : python-libipa_hbac-1.13.1-4.fc24.x86_64
>19/241
>DEBUG util.py:393: Verifying   : libipa_hbac-1.13.1-4.fc24.x86_64
>77/241
>DEBUG util.py:393: Verifying   : libsss_idmap-devel-1.13.1-4.fc24.x86_64
>187/241
>DEBUG util.py:393: Verifying   : libsss_idmap-1.13.1-4.fc24.x86_64
>188/241
>DEBUG util.py:393: Verifying   : libsss_nss_idmap-devel-1.13.1-4.fc24.x86_64
>189/241
>DEBUG util.py:393: Verifying   : libsss_nss_idmap-1.13.1-4.fc24.x86_64
>190/241
>DEBUG util.py:393: libipa_hbac.x86_64 1.13.1-4.fc24
>DEBUG util.py:393: libsss_idmap.x86_64 1.13.1-4.fc24
>DEBUG util.py:393: libsss_idmap-devel.x86_64 1.13.1-4.fc24
>DEBUG util.py:393: libsss_nss_idmap.x86_64 1.13.1-4.fc24
>DEBUG util.py:393: libsss_nss_idmap-devel.x86_64 1.13.1-4.fc24
>DEBUG util.py:393: python-libipa_hbac.x86_64 1.13.1-4.fc24
>
>and freeipa packages were successfully built.
>http://paste.fedoraproject.org/287118/07222144
Right and as I said, SSSD subpackages are installed. You listed them
above. That's all we needed.
-- 
/ Alexander Bokovoy