Dear Mike Frysinger,
In message 201012010634.19596.vapier@gentoo.org you wrote:
hrm, after running this through our test bench, it seems the old code and new code are not functionality equivalent. it boils down to default values when the env var is not set. some places interpret this to mean "yes" while others expect "no". getenv_yesno() takes the "no" default which doesnt always work.
In addition to the default values, there is the difference that the documentation states that "autostart" has to be set to "yes", i. e. a plain "y" is not supposed to mean 'yes'.
i can update the API to take a 2nd arg (the default value), or we can punt this patch. it's in "next", so there's less pressure to get it fixed immediately ...
I think the easiest way to fix this is to revert the commit.
Do you agree?
Best regards,
Wolfgang Denk