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=477570 --- Comment #4 from Peter Lemenkov <lemenkov@xxxxxxxxx> 2009-02-16 10:34:29 EDT --- * I also think that the mochiweb should be excluded and packaged separately. How different version, shipped with couchdb, from the upstream one? * rpmlint warnings (and one error) should be safely ignored. * "Requires: %{_bindir}/icu-config" looks ugly. First, is it really need, to have libicu-devel, to couchdb to work? Second, why you decided to require %{_bindir}/icu-config instead of libicu-devel? Other (still remaining) issues from Hubert's notes: * No need to explicitly require shadow-utils (as Hubert already mentioned). * Use %{_localstatedir} in sed string and in couchdb_home var instead of /var * remove 'exit 0' in %pre * use %{_initrddir} instead of %{_sysconfdir} -- 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