Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: TastyMenu - KMenu replacement aiming to provide the maximum usability https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238235 chris.stone@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris.stone@xxxxxxxxx ------- Additional Comments From chris.stone@xxxxxxxxx 2007-04-28 14:32 EST ------- Kelly, I can sponser you, however I really prefer it if you can log onto EFNet IRC #fedora-devel as communication this way is much faster and easier. My nick there is XulChris. Some initial comments on your spec file: - Release tag needs a dist tag, for example: Release: 2%{?dist} Do not use 0.nixa. stuff, also remove dist tag stuff from your changelog comments, it should just be: * Fri Apr 27 2007 Kelly Miller <lightsolphoenix@xxxxxxxxx> 0.7.1-2 - The URL tag should just be: http://www.notmart.org - Your BR lists redundant packages, change it to: BuildRequires: qt-devel >= 3.0.0, kdelibs-devel >= 3.0.0 - I would recommend just using the first paragraph for summary - Change %files to %{_docdir}/HTML/en/tastymenu so that you own the directory, do not bother listing each file under this directory, the directory is all you need to list, anything under it gets picked up automatically. -- 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