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
