--===============5239089138164955740== Content-Type: multipart/alternative; boundary="===============8858774338169088286==" --===============8858774338169088286== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit > On April 23, 2013, 6:11 p.m., Lukáš Tinkl wrote: > > Thanks for the patch but you should use the official FDO interface for that (org.freedesktop.PowerManagement) > > Hrvoje Senjan wrote: > Lukáš, i just adjusted the interface from old to new one (which was also solid's, not FDO's). > To me it's ok, as it is in Workspace and can be easily kept in sync. - Dario ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110135/#review31459 ----------------------------------------------------------- On April 23, 2013, 5:28 p.m., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110135/ > ----------------------------------------------------------- > > (Updated April 23, 2013, 5:28 p.m.) > > > Review request for Plasma, Solid, Dario Freddi, and Sebastian Kügler. > > > Description > ------- > > Suspend/hibernate are now longer are part of > org.kde.Solid.PowerManagement interface, so adjusting to new one. > > > This addresses bug 318737. > http://bugs.kde.org/show_bug.cgi?id=318737 > > > Diffs > ----- > > plasma/desktop/applets/kickoff/core/itemhandlers.cpp 4e83c37 > > Diff: http://git.reviewboard.kde.org/r/110135/diff/ > > > Testing > ------- > > Compiled, actions work now. > > > Thanks, > > Hrvoje Senjan > > --===============8858774338169088286== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit

This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110135/

On April 23rd, 2013, 6:11 p.m. UTC, Lukáš Tinkl wrote:
Thanks for the patch but you should use the official FDO interface for that (org.freedesktop.PowerManagement)
On April 23rd, 2013, 6:21 p.m. UTC, Hrvoje Senjan wrote:
Lukáš, i just adjusted the interface from old to new one (which was also solid's, not FDO's).

To me it's ok, as it is in Workspace and can be easily kept in sync.

- Dario

On April 23rd, 2013, 5:28 p.m. UTC, Hrvoje Senjan wrote:

Review request for Plasma, Solid, Dario Freddi, and Sebastian Kügler.

By Hrvoje Senjan.

Updated April 23, 2013, 5:28 p.m.

Description

Suspend/hibernate are now longer are part of
org.kde.Solid.PowerManagement interface, so adjusting to new one.

Testing

Compiled, actions work now.

Bugs: 318737

Diffs

plasma/desktop/applets/kickoff/core/itemhandlers.cpp (4e83c37)

View Diff

--===============8858774338169088286==-- --===============5239089138164955740== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============5239089138164955740==--