ehlo,

we (Jakub, me) didn't catch this as part of review.
I tested on minimal machine. Later I saw conflict when I was testing
latest sssd with freeipa-server.

I think we can ignore unowned directory /usr/share/polkit-1/rules.d
or we can add dependecy (soft dependency on fedora) to polkit.

LS
>From 79737d716fea5b1809296eb24f3276cc78959fae Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <lsleb...@redhat.com>
Date: Mon, 25 Jan 2016 15:17:33 +0100
Subject: [PATCH] SPEC: Fix conflict with polkit

We recently added /usr/share/polkit-1/rules.d to the spec file
to fix issues with unowned directories. However there is conflict
with polkit package. The owner is not root.

Running transaction test
Error: Transaction check error:
  file /usr/share/polkit-1/rules.d from install of 
sssd-common-1.13.90-0.20160125.1503.git1b8858b.master.fc23.x86_64 conflicts 
with file from package polkit-0.113-4.fc23.x86_64

[root@fe03f7aca7dc /]# ls -ld /usr/share/polkit-1/rules.d
drwx------. 1 polkitd root 0 Jul 14  2015 /usr/share/polkit-1/rules.d

The polkitd user is not part of default local users(/etc/passwd)
It's created as a part of instalation package polkit.
Therefore we cannot use hard-coded uid for this directory.
---
 contrib/sssd.spec.in | 2 --
 1 file changed, 2 deletions(-)

diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
index 
76066de9ac8b8ac1f9b14bf3bc787169d317fe2f..296f2dd5590027abfd6c70206801948b068c3269
 100644
--- a/contrib/sssd.spec.in
+++ b/contrib/sssd.spec.in
@@ -712,8 +712,6 @@ rm -rf $RPM_BUILD_ROOT
 %{_libexecdir}/%{servicename}/p11_child
 
 %if (0%{?install_pcscd_polkit_rule} == 1)
-%dir %{_datadir}/polkit-1
-%dir %{_datadir}/polkit-1/rules.d
 %{_datadir}/polkit-1/rules.d/*
 %endif
 
-- 
2.5.0

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to