[Bug 226573] Merge Review: xorg-x11-drivers

[bugzilla at redhat.com]

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 at leemhuis.info changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review-              |fedora-review+




------- Additional Comments From fedora at leemhuis.info  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.