Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.

Bug 174533

Summary: Redundant ownership of base system dirs in all libX* and xorg-x11* packages
Product: [Fedora] Fedora Reporter: Ville Skyttä <scop>
Component: xorg-x11Assignee: X/OpenGL Maintenance List <xgl-maint>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: blizzard, tcallawa
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-06-14 14:06:27 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 150221    

Description Ville Skyttä 2005-11-29 19:57:29 UTC
It seems that all (well, _many_) libX* and xorg-x11* packages in Rawhide
redundantly own %{_mandir}, which is already owned by the filesystem package:

    $ grep -l '^%dir %{_mandir}$' libX*/*.spec xorg*/*.spec | wc -l


    sed -i -e '/^%dir %{_mandir}$/d' libX*/*.spec xorg*/*.spec


Comment 1 Mike A. Harris 2005-11-30 00:19:23 UTC
This is intentional.

Comment 2 Ville Skyttä 2005-11-30 01:15:14 UTC
Could you elaborate?  Oh... I see it's also done for /etc in some cases and
/usr/share and /usr/lib (but not for /usr or /).

All these dirs are already owned by the filesystem package which cannot be
removed, so what value does owning them add?

Comment 3 Mike A. Harris 2005-11-30 03:03:43 UTC
What problem does specifying them cause?  Nothing.

Comment 4 Tom "spot" Callaway 2005-11-30 04:11:52 UTC
Reopening, until mharris can provide a good reason why he is dual-owning
anything that filesystem owns besides *X11* (which xorg should own somehow). ;)

Comment 5 Ville Skyttä 2005-12-20 16:42:46 UTC
No changes in 1.0.0-*, it seems.  (See also comment 4; I never received an email
notification about that being added.)

Comment 6 Tom "spot" Callaway 2005-12-20 20:59:56 UTC
Apparently, the owner of these packages sees no reason to be reasonable and use
best practices when making clean rpm specs.

I can't force him to do it, however, Fedora Extras packagers should not do what
this packager has done in Fedora Core.

Comment 7 Mike A. Harris 2006-02-02 06:18:40 UTC

I've discussed this request with Tom via email just prior to Christmas,
intending to clarify any confusion over the matter, and to try and ensure
we're seeing eye to eye.

I promised that I would re-evaluate the problem in the future, but also
indicated that there were a number of higher priorities to be considered
beforehand.  About a month or so has elapsed now, and it's time to
address this issue now and move forward.

For the benefit of everyone reading this, I'd like to clarify a few things.

Purely on technical grounds, it is my opinion that all packages should
technically own every directory that they place files in, and all directories
leading up to that.  I do have a good technical rationale for this opinion,
of which I have expressed in IRC, and have discussed with Tom, Jeff Johnson,
and various others.

While I am not the only person who shares this technical opinion rather
strongly, I think it is much more important to have a final resolution
to the issue, and move forward, than to risk getting caught up unnecessarily
in a heated technical debate over a trivial issue, with people whom I
respect.  Also, while comment #6 did rather offend me initially, I
understand that it was likely said in the heat of the moment at the
time, and I harbour no ill feelings about it currently.  I'd much
rather share beers at OLS while we joke about it and razz each other.  ;o)

On technical grounds, while I do still disagree about the issue, and there
do not appear to be any strict policies for Fedora Core which require
such, I have decided to put my technical opinion aside, and go ahead and
implement the requested changes, as I think we can all easily agree that
getting closure on trivial issues of this nature is more important, even
if there is technical disagreement or conflicts of opinion.

Therefore, I have started removing directory ownership from X packaging,
where the directories are common system directories such as %{_mandir}
and similar.  Since this is being done manually as I make updates to
the packages and fix bugs, etc. it will likely take a while to completely
resolve the issue as you've requested.  I'm currently updating all of
the driver packages, which should be done tomorrow or the next day.

After that, if anyone feels like providing a complete list here of remaining
packages that need to be updated yet, I'll cross them off the list as I
go along, until it's complete.  Otherwise, I'll get to them all eventually,
as I spot them when going through the packages.

Overall, I think this decision is rather reasonable, if not overly so. ;o)

Anyhow, we can all now concentrate on bigger and better things, without
having curled feathers.  ;o)


Comment 8 Mike A. Harris 2006-02-02 09:10:03 UTC
Modulo human error, all of the input driver packages have been updated for
this change.  More to come...

Comment 9 Mike A. Harris 2006-02-03 07:21:55 UTC
All video driver packages updated.

Comment 10 Mike A. Harris 2006-02-06 08:49:56 UTC
xorg-x11-server 1.0.1-3 and later should be clean when built

Comment 11 Mike A. Harris 2006-02-16 23:05:02 UTC
Fixed xorg-x11-proto-devel-7.0-3

Comment 12 Mike A. Harris 2006-03-01 07:20:34 UTC
The remainder of the X11R7 spec files have been updated in CVS to remove
ownership of system dirs.  A few have been rebuilt with other fixes for
rawhide already.  The others will pick up the changes the next time there
is good reason to rebuild the package.

Closing as RAWHIDE