----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107257/#review22678 -----------------------------------------------------------
Just FTR: The DPMS *always* exists in the config file, no matter if it is supported, compiled, or otherwise doesn't work. It is being added by the profile generator on first start along with all the other default settings. This means this message will always appear if the system does not DPMS for some reason. - Kai Uwe Broulik On Nov. 21, 2012, 7:20 p.m., Oliver Henshaw wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107257/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2012, 7:20 p.m.) > > > Review request for Solid. > > > Description > ------- > > Handle unsupported actions quietly > > Attempting to load a configured action on a machine where it is not > supported (e.g. DPMS when the display doesn't support it or it is not > compiled in) fails and brings up a notification, something that is > particularly intrusive during login. > > Provide a method for ActionPool::loadAction callers to ask why the load > failed. PowerDevil::Core::loadProfile uses this check to decide whether > to warn to stderr rather than notifying the user of a misconfiguration. > Other loadAction callers are unchanged. > > Action loading failure may be due to an error during initialisation or > simply because no such action exits. In the former case the error must > be recorded so that it is available to pass on to later loadAction > callers. > > NB: A more complete fix might involve detecting whether the action is > supportable when loading the action configuration and/or in the profile > generator. However that may not turn out to be a feasible approach. > > BUG: 302846 > > > Diffs > ----- > > powerdevil/daemon/powerdevilactionpool.h > 8a94eacc8ef2c2aead8cb075cbc80b783c1aeb4c > powerdevil/daemon/powerdevilactionpool.cpp > a9950f174fe184b8faa54c54fc00654984c65b3f > powerdevil/daemon/powerdevilcore.cpp > 2dcdbc62236d5c1fae384fdb9111825a2ebf5204 > > Diff: http://git.reviewboard.kde.org/r/107257/diff/ > > > Testing > ------- > > Tested in VM with cirrus/vnc (dpms) and qxl/spice (non-dpms) graphics. Tested > the NoAction and LoadFailed cases act as expected. Tested nothing horrible > happens when disabling and re-enabling powerdevil in kded Services Manager. > > > Thanks, > > Oliver Henshaw > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
