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=526311 --- Comment #16 from Ruben Kerkhof <ruben@xxxxxxxxxxxxxxxx> 2009-11-19 05:35:04 EDT --- Great, nice work! Ok, another couple comments: * package is still x86_64, should be noarch * mysql-mmm-monitor still needs the DBD::MySQL Requires * I don't mind the Provides, but use %{version}-%{release} * The files section contains 2 entries for /var/run/mysql-mmm * Replace /var/log/mysql-mmm with %{_localstatedir}/log/mysql-mmm -- 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