Bug 710781

Summary: Unowned /usr/lib/ConsoleKit, /usr/lib/ConsoleKit/run-seat.d dirs
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: udevAssignee: Harald Hoyer <harald>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: unspecified    
Version: rawhideCC: harald, jonathan
Target Milestone: ---Keywords: EasyFix, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-06-20 08:46:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Ville Skyttä 2011-06-04 18:11:07 UTC
$ rpm -qf /usr/lib/ConsoleKit
file /usr/lib/ConsoleKit is not owned by any package
$ rpm -qf /usr/lib/ConsoleKit/run-seat.d
file /usr/lib/ConsoleKit/run-seat.d is not owned by any package
$ rpm -qf /usr/lib/ConsoleKit/run-seat.d/udev-acl.ck 
udev-171-1.fc16.x86_64

udev should own the  /usr/lib/ConsoleKit and /usr/lib/ConsoleKit/run-seat.d dirs.  Fix below, let me know if you'd like me to commit and build this for devel.

diff --git a/udev.spec b/udev.spec
index 4a14c08..acbe4a0 100644
--- a/udev.spec
+++ b/udev.spec
@@ -286,2 +286,4 @@ rm -rf $RPM_BUILD_ROOT
 %{udev_scriptdir}/keymaps/*
+%dir %{_prefix}/lib/ConsoleKit
+%dir %{_prefix}/lib/ConsoleKit/run-seat.d
 %attr(0644,root,root) %{_prefix}/lib/ConsoleKit/run-seat.d/udev-acl.ck

Comment 1 Harald Hoyer 2011-06-17 09:45:10 UTC
$ rpm -qf /usr/lib/ConsoleKit
ConsoleKit-0.4.5-1.fc15.x86_64

Comment 2 Harald Hoyer 2011-06-17 09:46:04 UTC
udev might require ConsoleKit then.. not sure, if that is wanted

Comment 3 Ville Skyttä 2011-06-17 20:13:48 UTC
If udev does not actually require ConsoleKit, the dependency should not be added just for dir ownership, instead the dirs should be owned in udev which is what the patch does.

http://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership

Comment 4 Harald Hoyer 2011-06-20 08:16:38 UTC
all right! thanks!