Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: memcached - High Performance, Distributed Memory Object Cache https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238994 matthias@xxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |matthias@xxxxxxxxxxxx ------- Additional Comments From matthias@xxxxxxxxxxxx 2007-05-29 11:40 EST ------- A few comments : - You could be a little more consistent in your spec wrt "%{name}" vs. "memcached" usage. For instance, the init script is included as "memcached" but the scriplets control the service as "%{name}". - Would you consider replacing the current empty OPTIONS with "-l 127.0.0.1" in order to "secure" default installations? - As already mentioned the "Requires: initscripts" is useless since "Requires(preun): /sbin/chkconfig" already makes sure it'll be installed. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review