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=501573 --- Comment #15 from Xavier Bachelot <xavier@xxxxxxxxxxxx> 2009-07-21 05:04:16 EDT --- A couple more comments : - According to the doc, ndoutils should work with nagios 2 too. You could add conditionals to the spec to be able to build both for Fedora and RHEL/CentOS 5 (Nagios 2 is shipped in EPEL). I don't know if you wish to maintain ndoutils in EPEL too, but it looks like a good idea to me and probably others. - (pedantic) There are small typos in some comments : "# Remove executables bits from the example sripts in the documentation." executable_ and s_c_ripts. "# Upsteam does not provide a install target." Upst_r_eam and a_n_ install target "# MySQL lib locations must be explicitly specifed to be found on " location_ and specif_i_ed - You should package the latest version. -- 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