[Bug 477570] Review Request: couchdb - A document database server, accessible via a RESTful JSON API

[bugzilla at redhat.com]

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 at gmail.com>  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.