<div dir="ltr"><div>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.</div><div><br></div><div>Jonathan</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 27 Sep 2019 at 07:14, Dorian Vogel <<a href="mailto:dorianvogel@gmail.com">dorianvogel@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hey Jonathan,</div><div>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.</div><div><b>If I remember correctly,</b> 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</div><div><br></div><div>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).<br></div><div><br></div><div>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.</div><div><br></div><div>Cheers,</div><div><b>Dorian</b><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 26, 2019 at 11:13 AM Jonathan Riddell <<a href="mailto:jr@jriddell.org" target="_blank">jr@jriddell.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>The DDCUtil features of Powerdevil in KDE Plasma are currently disabled by default</div><div>"DDCUtil library support is disabled by default as recomended by authors, add -DHAVE_DDCUTIL=On to enable"</div><div>This is based on a conversation from 2017 where it was decided it was unstable.</div><div><br></div><div>I see work has continued on DDCUtil and on support for it in Powerdevil.  Is it ready to be set to on by default?  <br></div><div><br></div><div>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.</div><div><br></div><div>Jonathan</div><div><br></div></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr">VOGEL Dorian<br></div>
</blockquote></div>