Fedora Merge Review: gnome-audio http://cvs.fedora.redhat.com/viewcvs/devel/gnome-audio/ Initial Owner: alexl
Good: * Source URL is canonical * Package name conforms to the Fedora Naming Guidelines * Group Tag is from the official list * All paths begin with macros * All directories are owned by this or other packages * Builds fine in mock. Must fix: * rpmlint error: W: gnome-audio hardcoded-path-in-buildroot-tag /var/tmp/gnome-audio-%{version}-root Buildroot must be '%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)'. Minor things to look at, not blockers: * Might be nice to move the %files section before the ChangeLog. * rpmlint error: W: gnome-audio summary-ended-with-dot Sounds for GNOME events.
All comments fixed in cvs (2.0.0-4)
Changes to spec look good. +1 APPROVE.
This review is done.