[Merge] ~enr0n/ubuntu-release-upgrader:lp.1959458 into ubuntu-release-upgrader:ubuntu/main

Nick Rosbrook mp+415396 at code.launchpad.net
Fri Feb 11 22:29:22 UTC 2022


> This looks great, however I think we should check with other flavors (xubuntu,
> ubuntu-mate, etc...) as to whether or not they use xdg-screensaver (on Focal
> or Impish) before yanking out that bit of code.
> 
> Otherwise, just some minor in-line comments.

After looking at the xdg-screensaver source [1], I decided to leave that code alone. While in some cases xdg-screensaver will also use the org.freedesktop.ScreenSaver interface, it has alternate methods that work where that interface is not available (e.g. xscreensaver). I would rather not accidentally break something.
-- 
https://code.launchpad.net/~enr0n/ubuntu-release-upgrader/+git/ubuntu-release-upgrader/+merge/415396
Your team Ubuntu Core Development Team is subscribed to branch ubuntu-release-upgrader:ubuntu/main.




More information about the Ubuntu-reviews mailing list