[webstack-discuss] webrev needed for upgrading memcached 1.2.5 to 1.4.1

[Matt Ingenthron]

Trond Norbye wrote:
> See:
>
> http://cr.opensolaris.org/~trond/webrev/

 From the peanut gallery....

It generally looks good.  I like the change to the default user of noaccess.

Having said that, I'd highly recommend adding some default options, say 
"-L -m 64".  Why?  On popular Linux distros they ship with default 64MB 
configurations via init scripts.  I've seen references from Ruby on 
Rails people which seem to indicate they think that's the default as a 
result (even though memcached has no default).

Or looking at it from another angle, just being able to do an "svcadm 
enable memcached" after installing the package should yield something 
useful, even if it's not perfect for anyones usage.

Might it be better for the -u flag and the -m flag to just be in the 
default service manifest?  As it is now, one would have to edit the 
script the service is execing to change the user and there isn't a sane 
default.

- Matt