Review Request 129651: [dragon] Inhibit powersave

Anthony Fieroni bvbfan at abv.bg
Fri Dec 16 06:30:18 GMT 2016



> On Дек. 14, 2016, 12:15 след обяд, Harald Sitter wrote:
> > Inhibiting is so very bad... 
> > 
> > - org.freedesktop.PowerManagement is deprecated https://www.freedesktop.org/wiki/Specifications/power-management-spec/ there seems to be no information as to what it was deprecated in favor of though
> > - we could use org.kde.Solid.PowerManagement but that would not work on !plasma as I understand it
> > - we could use org.freedesktop.login1 https://www.freedesktop.org/wiki/Software/systemd/inhibit/ which would work everywhere but on plasma, as apparently powerdevil tells logind to shut up
> > - there's this unlanded RR https://git.reviewboard.kde.org/r/126650/ which would kinda solve our problem, albeit also using org.freedesktop.PowerManagement
> > 
> > Where to go from here I do not actually know. I'd rather not use org.freedesktop.PowerManagement given its depreated state, at the same time I am somewhat inclined to not care given every source code I checked is using it anyway. that being said the RR currently doesn't inhibit screensavers. 
> > 
> > ---
> > 
> > As for the merge problem. When you commit to maintenance branches (i.e. Applications/) you need to manually merge them into master or any other relevant branches as well. There is no automatic merge and I am also only merging into master on-demand, when I make a change myself. To fix this it appears you can simply merge Applications/16.04 into Applications/16.12 and then Applications/16.12 into master

QDBusInterface iface(QLatin1String("org.freedesktop.login1"),
                     QLatin1String("/org/freedesktop/login1"),
                     QLatin1String("org.freedesktop.login1.Manager"),
                     QDBusConnection::systemBus());
 reply = iface.call(QLatin1String("Inhibit"),
                    QLatin1String("sleep:shutdown"),
                    i18n("Dragon Player"),
                    i18n("playing a video"),
                    QLatin1String("block"));
does not work


> On Дек. 14, 2016, 12:15 след обяд, Harald Sitter wrote:
> > src/app/mainWindow.cpp, line 820
> > <https://git.reviewboard.kde.org/r/129651/diff/1/?file=487471#file487471line820>
> >
> >     I am pretty sure the inhibition port is lackluster. Notably KnotificationRestrictions inhibited screensaving which I am assuming doesn't fall into the PowerManagement domain so the new code while inhibiting sleeping it seems ot miss screensaver stuff.
> >     
> >     Fortunately knotificationrestrictions is ported and you can just keep on using it.

Can we use only knotificationrestrictions, but sleep/suspend to not handle at all?


- Anthony


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129651/#review101431
-----------------------------------------------------------


On Дек. 14, 2016, 6:49 преди обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129651/
> -----------------------------------------------------------
> 
> (Updated Дек. 14, 2016, 6:49 преди обяд)
> 
> 
> Review request for KDE Multimedia and Harald Sitter.
> 
> 
> Repository: dragon
> 
> 
> Description
> -------
> 
> Since solid power api is not released, i use freedesktop dbus to fit other desktop power managers
> 
> 
> Diffs
> -----
> 
>   src/app/mainWindow.cpp 4f0b3e0 
> 
> Diff: https://git.reviewboard.kde.org/r/129651/diff/
> 
> 
> Testing
> -------
> 
> *Harald master was not merged (16.04 / 16.08 to master) -> it has fresh bugreports for fixed issues.
> Last merge to 16.08
> 2016-04-18	Merge branch 'Applications/16.04'	Harald Sitter
> after that in brach 16.04
> 2016-04-26	Hide toolbar on multiscreen setup	Anthony Fieroni
> 2016-04-20	Don't try to resize maximized window	Anthony Fieroni
> 2016-04-19	MPRIS2 property 'Identity' to match desktop file entry	Anthony Fieroni
> 
> Do you want to merge them or you will ship ?
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20161216/8c859e72/attachment.htm>


More information about the kde-multimedia mailing list