Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: xorg-x11-drivers https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226573 fedora@xxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review- |fedora-review+ ------- Additional Comments From fedora@xxxxxxxxxxxxx 2007-02-25 07:14 EST ------- (In reply to comment #3) > Okay, did some cleanup. thx > I did the same to all the driver packages I could, with the exception of a few > that really are arch-specific. [...] thx, looks a lot better. But I think I spotted a bug: xorg-x11-drv-i810 should be required on x86_64, too, as there are x86_64 compatible boards with intel graphic chipsets out there. > [...] > Which leaves us with: > > % rpmlint xorg-x11-drivers-7.2-2.fc7.src.rpm > W: xorg-x11-drivers no-url-tag > % rpmlint i686/xorg-x11-drivers-7.2-2.fc7.i686.rpm > W: xorg-x11-drivers no-url-tag > E: xorg-x11-drivers no-binary > W: xorg-x11-drivers no-documentation > Also I don't see the point in making a trivial doc payload, given > that it'd just be a repeat of the %description. I'd say it would be worth the trouble. Regarding the bugs: - Fixing Bug 199381 would be nice and shouldn't do any harm, even if those drivers are are not (yet) packaged in main Fedora -- but I don't consider this a blocker (sorry dgilmore -- I'm counting on ajax cooperation in this case). - Bug 198294 will be fixed in another package APPROVED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review