hal: Branch 'master'
david at fubar.dk
Mon Aug 27 10:07:55 PDT 2007
> > Keep in mind that you get the action name as part of the exception
> > detail so you only need to know the action name for the rather oddball
> > cases where the caller checks himself to e.g. decide to put a lock icon
> > next to the button. I doubt anyone actually does this yet?
> Yes, you get a exception. But this is not what you want in each case. For
> powermanagemant you want e.g. don't show a action as s2ram/s2disk to the
> user if it isn't allowed for this special user. For this you need to know the
> name of the policy/action. KPowersave and also powersaved e.g. check the
> policies at startup and from time to time (e.g. if the session state change)
> to disable/enablee.g. s2ram or change the brightness e.g. in the applet menu.
> And this would now work any longer.
Yeah, that's true.
> IMO this change is at least suboptimal within RC status of HAL.
It's not really optimal, I agree. But it should be a rather minimal
change (either in hal or the kpowersave / powersaved) packages;
basically just a string change I believe? (Btw, I'm going to bump the
soname of libpolkit.so too because of some other changes)
> P.S. Do you have the time to review/approve patches on the HAL ML in
> the next days?! Would help a lot!
It would and I will. I'm just slow mostly because there's a lot of RHEL4
and RHEL5 work eating up my time; it's like it's never going away :-(
More information about the hal-commit