Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=449595 David Woodhouse <dwmw2@xxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@xxxxxxxxxxxxxxxxx |dwmw2@xxxxxxxxxxxxx Flag| |fedora-review+ --- Comment #6 from David Woodhouse <dwmw2@xxxxxxxxxxxxx> 2008-11-19 19:42:31 EDT --- Aside from the zero-length file, please consider changing the description to reference Xorg instead of XFree86, although what it says at the moment _is_ technically correct. Please also consider fixing the email address in the specfile. Do you really think the spambots haven't learned to cope with [at] yet? You're only making hard for _real_ people, and not for the spammers. Other than that, it looks good. Approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review