[Pm-utils] unify boolean config variables

Michael Biebl mbiebl at gmail.com
Wed Apr 29 22:09:52 PDT 2009


2009/4/30 Michael Biebl <mbiebl at gmail.com>:
> Hi,
>
> currently our man pages and config variables are not very consistent
> with regard to boolean variables.
>
> The majority uses true/false,
> there is HIBERNATE_RESUME_POST_VIDEO which is yes/no
> and NEED_CLOCK_SYNC, where anything else then "" is treated as true
> (see 90clock).
>
> I'd like to use true/false for all boolean values and update the
> manpages for HIBERNATE_RESUME_POST_VIDEO and NEED_CLOCK_SYNC and also
> the check in 90clock, to test explictly [ "$NEED_CLOCK_SYNC" = "true"
> ] || ...
>
> Any objections? Do you think this would pose upgrade problems for
> users who already set
> HIBERNATE_RESUME_POST_VIDEO to yes or no ( I could check both values
> (yes/no,true/false))

This time with patch. I keep the backward compatible check for
HIBERNATE_RESUME_POST_VIDEO = yes

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pm-utils-boolean-config-vars.patch
Type: application/octet-stream
Size: 3717 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/pm-utils/attachments/20090430/7e29f0bb/attachment.obj 


More information about the Pm-utils mailing list