Bug 196229
Summary: | Review Request: xkeyboard-config | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Mike A. Harris <mharris> |
Component: | Package Review | Assignee: | David Cantrell <dcantrell> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, mclasen, xgl-maint |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-07-17 19:25: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: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 188268 |
Description
Mike A. Harris
2006-06-22 05:13:01 UTC
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. TIA Added to dist, package approved. Built into rawhide. |