Red Hat Bugzilla – Bug 196229
Review Request: xkeyboard-config
Last modified: 2013-01-09 20:26:12 EST
Spec URL: <spec info here>
SRPM URL: <srpm info here>
NOTE: This is a package renaming request, to rename xorg-x11-xkbdata package
to xkeyboard-config. The spec file will be identical to the existing
xorg-x11-xkbdata which is of course already part of Core, and follows
Fedora packaging guidelines.
If it is mandatory to attach the actual spec/srpm here by the book, let
me know and I'll do that. Otherwise, yank xorg-x11-xkbdata spec file
and review that, as the only thing that'll be different, is the name of
the package, with the changelog reset, and any minor tweaks that might
be needed for compat.
Thanks in advance.
Package looks good, you'll have to add a Obsoletes: xorg-x11-xkbdata and
Provides xorg-x11-xkbdata for upgrade paths.
Can we get rid if the "super ugly hack" in %pre now?
I'm going to approve.
*** Bug 197939 has been marked as a duplicate of this bug. ***
(In reply to comment #1)
> Package looks good, you'll have to add a Obsoletes: xorg-x11-xkbdata and
> Provides xorg-x11-xkbdata for upgrade paths.
Yep to both.
> Can we get rid if the "super ugly hack" in %pre now?
My comment in there suggests so, but I'll need to review all possible
upgrade paths to be sure. I like to err on the side of caution though.
A quick cursory look appears that I should be able to remove that
ugliness though. Sometimes I put hacks of that nature in to prevent
short term developmental upgrade problems from causing unnecessary
bug reports - but later after an OS release is out, and there are unlikely
to be any (or many) people who would have gotten bitten by a development
only transient issue, it makes sense to flush the toilet so to speak. ;)
> I'm going to approve.
Cool. I'm going to create the new package now, import it and update
the report when I'm done. Thanks Jesse.
Ok, after closer review, it turns out that the %pre hack was only needed
to work around an rpm limitation which occured when updating from early
rawhide versions of xorg-x11-xkbdata which contained the X.Org xkbdata -
to the xorg-x11-xkbdata which contains xkeyboard-config. Therefore,
the issue can never occur on an official OS upgrade from any FC release
to any newer FC release, or any RHEL release to a newer one.
So, I've axed the %pre hack. Anyone hit by the issue now, deserves it.
Ok, I've imported xkeyboard-config-0.8-1 into CVS, fixed a minor issue and
tagged 0.8-2. I've built it locally and upgraded my modular X system with
it successfully. No rpm upgrade warnings/errors.
I have at least temporarily left out "Provides: xorg-x11-xkbdata" for now
for 2 reasons:
1) I believe only our modular X server package and possibly the
xorg-x11-xkbutils packages depend on xkbdata, and both are using the
virtual provides "Requires: xkbdata" which was present in both the
old and new package - so "Provides: xorg-x11-xkbdata" shouldn't be
needed in theory.
2) I'd like to find out if #1 is false right now, by putting a package
in rawhide without the Provides to see if other packages do have
incorrectly specified dependencies, so we can fix them to use the
virtual provide. Closer to the FC6 release, I can always add the
virtual dep if there are any problems or if we want to play things
safe (albeit a bit pedantically) ;)
The package is now ready, tested and waiting. Can you add it to dist-fc6
so I can build it, and then disable xorg-x11-xkbdata from dist-fc6.
Added to dist, package approved.
Built into rawhide.