On Mon, Jan 28, 2008 at 08:38:59AM -0800, Gary Winiger wrote:
> > If WORKSTATION_OWNER=Workstation Owner (or CONSOLE_USER=...) will be
> > added on upgrade then removing it manually cannot be a good way to
> > express the legacy "-" from /etc/default/sys-suspend...  The next
> > upgrade would restore the default "console-owner" sys-suspend behaviour.
> 
>       The intent is that i.policyconf act to WORKSTATION_OWNER the
>       same as it does with AUTHS_GRANTED and PROFS_GRANTED.
>       If present or commented out, don't change.  If absent add the
>       default.

> > > for "-" (none), comment out or remove the policy.conf(4)
> > > WORKSTATION_OWNER key;

The manpage doesn't say "remove" so I guess that's OK.  Eventually there
should be a case obsoleting /etc/default/sys-suspend, right?  That
manpage shouldn't say that "-" maps to "comment out _or remove_"...

> > Instead, allowing and recommending an empty value to disable the feature
> > may be the way to go.
> 
>       Just as with AUTHS_GRANTED and PROFS_GRANTED, WORKSTATION_OWNER
>       can contain an empty value.
> 
> Gary..
> P.S.  If you'd like to code review or test ;-) .....

I'll code review.

Reply via email to