ddcutil in Plasma Powerdevil

[Jonathan Riddell]

I put ddcutils package in Neon unstable to be able to test it, using
Sanford's packaging from Debian but with the libraries turned on.  However
powerdevil doesn't compile with DDCUtils turned on, it fails on a cmake
error.

Jonathan


On Fri, 27 Sep 2019 at 07:14, Dorian Vogel <dorianvogel at gmail.com> wrote:

> Hey Jonathan,
> Unfortunately I did not test it for a long time (last time was around
> 5.15), and don't really have the possibility now because I'm away from home.
> *If I remember correctly,* back then I had a few instances when the
> startup of plasma (during the splashscreen) was quite longer due to retries
> of the DDC communication (of course this will not happen if the user does
> not have ddcutil itself installed). As I understand it, this is due to the
> architecture of Powerdevil, which assumes the monitors are a fixed
> configuration as far as brightness is concerned: backends are queried at
> startup and never refreshed. This is why enumeration has to happen during
> startup of plasma, and one would have to restart powerdevil when connecting
> a new DDC capable monitor
>
> Thinking back I wonder if a plasmoid would have been a better way,
> enabling to have the enumeration done after startup is finished, and making
> it easier to have a "refresh" button, and would still allow to have both an
> integrated screen (laptop) and external (for now the backends in powerdevil
> are exclusive).
>
> Considering this is my first contribution and I am not sure of the
> quality, I think some testing from more experienced devs would be required.
> I haven't heard much so far.
>
> Cheers,
> *Dorian*
>
> On Thu, Sep 26, 2019 at 11:13 AM Jonathan Riddell <jr at jriddell.org> wrote:
>
>> The DDCUtil features of Powerdevil in KDE Plasma are currently disabled
>> by default
>> "DDCUtil library support is disabled by default as recomended by authors,
>> add -DHAVE_DDCUTIL=On to enable"
>> This is based on a conversation from 2017 where it was decided it was
>> unstable.
>>
>> I see work has continued on DDCUtil and on support for it in Powerdevil.
>> Is it ready to be set to on by default?
>>
>> It is too late for Plasma 5.17 due for release in 2 weeks but we can tell
>> distros the message is old as appropriate.
>>
>> Jonathan
>>
>>
>
> --
> VOGEL Dorian
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190930/260b066c/attachment.html>