[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[Launchpad Bug Tracker]

This bug was fixed in the package gnome-screensaver - 3.6.1-7ubuntu6

---
gnome-screensaver (3.6.1-7ubuntu6) artful; urgency=medium

  * Add support for Budgie Desktop (LP: #1703690)
- enable support for gnome-control-center
- style the lock-screen using the "Unity" patch-work

 -- David Mohammed   Thu, 31 Aug 2017 20:53:09
+0100

** Changed in: gnome-screensaver (Ubuntu)
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  Fix Released
Status in gnome-screensaver package in Ubuntu:
  Fix Released

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[Iain Lane]

Sponsored, thanks. For gnome-desktop3, I merged your patch into the
revert patch since I thought it made sense there. If that's too annoying
in future, someone can split it back out.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  Fix Released
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[Launchpad Bug Tracker]

This bug was fixed in the package gnome-desktop3 - 3.25.91.1-0ubuntu2

---
gnome-desktop3 (3.25.91.1-0ubuntu2) artful; urgency=medium

  * debian/patches/git_revert_draw_background.patch: Check
org.gnome.desktop.background explicitly, not via the passed-in GSettings.
We're passed different schemas, but DRAW_BACKGROUND is always there. Patch
from David Mohammed (LP: #1703690)

 -- Iain Lane   Mon, 04 Sep 2017 17:46:55
+0100

** Changed in: gnome-desktop3 (Ubuntu)
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  Fix Released
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

please find enc two revised patches that hopefully take into account
Laney and Lukasz comments.

gnome-desktop3 is very much simplified to explicitly check the
draw_background key from the background schema.  Note - I haven't
included a new changelog for this debdiff.

gnome-screensaver reworks to define the in_desktop check via its own
module "gs-common.h/c".  The in_desktop check simplifies the code as per
Laney's recommendation.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

** Patch removed: "patch 1 of 2: patch for gnome-desktop3"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+attachment/4913063/+files/gnome-desktop3_3.24.2-0ubuntu2.debdiff

** Patch removed: "patch 2 of 2: patch for gnome-screensaver"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+attachment/4913064/+files/gnome-screensaver_3.6.1-7ubuntu6.debdiff

** Patch added: "gnome-screensaver_3.6.1-7ubuntu6.debdiff"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+attachment/4942037/+files/gnome-screensaver_3.6.1-7ubuntu6.debdiff

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

** Patch added: "gnome-desktop3_3.25.91.1-0ubuntu1.debdiff"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+attachment/4942038/+files/gnome-desktop3_3.25.91.1-0ubuntu1.debdiff

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

additionally " two things on the gnome-screensaver one: in_desktop
should probably use g_strv_contains instead of iterating the list and I
think you should find (or create) a common file to contain that utility
function rather than defining it multiple times"

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

discussed on IRC with Laney - suggestion:

make the is_enabled check use a new GSettings, like GSettings
*background_settings; background_settings = g_settings_new
("org.gnome.desktop.background"); bg->is_enabled =
g_settings_get_boolean (background_settings, BG_KEY_DRAW_BACKGROUND);
g_object_unref (background_settings);

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Re: [Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

Thanks Łukasz

With regards to the gnome-desktop3 patch.  I had a look through the
gsettings API and there didnt seem to be a method to say "is this
gsettings key you are about to use valid" - in fact all the
documentation basically says it is up to the programmer to ensure that
a key being used is actually there ("programmer error").  GSettings
doesnt seem to-do any validation which is a shame.

I would prefer if the whole "draw-background" ubuntu patch  could be
dropped (and hence make this patch unnecessary) but I don't know
really what the current impact that would have on the Unity desktop.

Ack with regards to the static functions in gnome-screensaver.  I
suppose its a coding style to keep code as localised as possible.

I did ping on both #ubuntu-devel and #ubuntu-desktop recently but no
takers.  I don't really know anyone on the GNOME team other than
JBicha.

On 3 August 2017 at 10:15, Łukasz Zemczak <1703...@bugs.launchpad.net> wrote:
> Hello! Thanks for submitting the bug and patches!
>
> I looked briefly at the provided debdiffs and from the packaging POV
> they look good. I'm not sure about the contents though. I am not a GNOME
> developer so I'd like to get some opinion of someone that knows the code
> and would be able to say if it's the right way to go or not. Did you
> discuss these changes with the desktop team or anyone from the GNOME
> uploaders?
>
> Checking the patches I'm a bit worried about a few things. In the 
> gnome-screensaver part, the patch introduces the static function in_desktop() 
> in both gs-lock-plug.c and gs-window-x11.c. I know it's nice to have helper 
> functions like these 'local' but I'm always reluctant whenever I see 
> boilerplate code like this. Maybe it could be made global and somehow shared? 
> I guess it's fine if there's a lot of boilerplate there already, would have 
> to check the codebase.
> The gnome-desktop3 part seems good but a little bit hacky. Wonder if this 
> could be done in a more generic way. Maybe it's not possible though...
>
> Anyway, let's get someone from the GNOME people to take a look before we
> decide about uploading it.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1703690
>
> Title:
>   Add support for Budgie Desktop using GNOME Screensaver
>
> Status in gnome-desktop3 package in Ubuntu:
>   In Progress
> Status in gnome-screensaver package in Ubuntu:
>   In Progress
>
> Bug description:
>   Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
>   screen.
>
>   It also uses gnome-control-center to control both the desktop
>   background and the lock-screen background.
>
>   GNOME Screensaver does not support gnome-control-center lock screen
>   dialog - GNOME has changed to a gsettings path
>   org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
>   path org.gnome.desktop.background.
>
>   To resolve this requires patches in two packages - gnome-screensaver
>   and gnome-desktop3.
>
>   1. GNOME Screensaver has been patched to support Budgie Desktop and 
> gnome-control-center.
>   Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
> which is now defunct (I believe) since Unity uses an alternative locking 
> mechanism.  The lock-screen is styled as per the old "Unity" implementation 
> before Unity 16.04 moved to the new lock screen.
>
>   2. The consequence of making the requisite changes to GNOME
>   Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
>   has a Ubuntu specific patch to revert a GNOME upstream decision to
>   remove a key called draw-background.  GNOME Screensaver calls a public
>   function in gnome-desktop3 - the gsettings path causes a segmentation
>   fault since draw-background does not exist in
>   org.gnome.desktop.screensaver.  I have worked around this to check the
>   path being called before pulling the draw-background key.
>
>   All of this is explained in the dep3 headers of the two debdiff
>   patches attached
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this 

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[Łukasz Zemczak]

Hello! Thanks for submitting the bug and patches!

I looked briefly at the provided debdiffs and from the packaging POV
they look good. I'm not sure about the contents though. I am not a GNOME
developer so I'd like to get some opinion of someone that knows the code
and would be able to say if it's the right way to go or not. Did you
discuss these changes with the desktop team or anyone from the GNOME
uploaders?

Checking the patches I'm a bit worried about a few things. In the 
gnome-screensaver part, the patch introduces the static function in_desktop() 
in both gs-lock-plug.c and gs-window-x11.c. I know it's nice to have helper 
functions like these 'local' but I'm always reluctant whenever I see 
boilerplate code like this. Maybe it could be made global and somehow shared? I 
guess it's fine if there's a lot of boilerplate there already, would have to 
check the codebase.
The gnome-desktop3 part seems good but a little bit hacky. Wonder if this could 
be done in a more generic way. Maybe it's not possible though...

Anyway, let's get someone from the GNOME people to take a look before we
decide about uploading it.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

** Patch added: "patch 2 of 2: patch for gnome-screensaver"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-screensaver/+bug/1703690/+attachment/4913064/+files/gnome-screensaver_3.6.1-7ubuntu6.debdiff

** Tags added: patch

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

[Desktop-packages] [Bug 1703690] Re: Add support for Budgie Desktop using GNOME Screensaver

[fossfreedom]

** Patch added: "patch 1 of 2: patch for gnome-desktop3"
   
https://bugs.launchpad.net/ubuntu/+source/gnome-screensaver/+bug/1703690/+attachment/4913063/+files/gnome-desktop3_3.24.2-0ubuntu2.debdiff

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-desktop3 in Ubuntu.
https://bugs.launchpad.net/bugs/1703690

Title:
  Add support for Budgie Desktop using GNOME Screensaver

Status in gnome-desktop3 package in Ubuntu:
  In Progress
Status in gnome-screensaver package in Ubuntu:
  In Progress

Bug description:
  Ubuntu Budgie and Budgie Desktop uses GNOME Screensaver for its lock-
  screen.

  It also uses gnome-control-center to control both the desktop
  background and the lock-screen background.

  GNOME Screensaver does not support gnome-control-center lock screen
  dialog - GNOME has changed to a gsettings path
  org.gnome.desktop.screensaver.  GNOME Screensaver is expecting the
  path org.gnome.desktop.background.

  To resolve this requires patches in two packages - gnome-screensaver
  and gnome-desktop3.

  1. GNOME Screensaver has been patched to support Budgie Desktop and 
gnome-control-center.
  Note - I've taken the opportunity to reuse the existing "Unity" patchwork 
which is now defunct (I believe) since Unity uses an alternative locking 
mechanism.  The lock-screen is styled as per the old "Unity" implementation 
before Unity 16.04 moved to the new lock screen.

  2. The consequence of making the requisite changes to GNOME
  Screensaver has unfortunately impacted gnome-desktop3.  gnome-desktop3
  has a Ubuntu specific patch to revert a GNOME upstream decision to
  remove a key called draw-background.  GNOME Screensaver calls a public
  function in gnome-desktop3 - the gsettings path causes a segmentation
  fault since draw-background does not exist in
  org.gnome.desktop.screensaver.  I have worked around this to check the
  path being called before pulling the draw-background key.

  All of this is explained in the dep3 headers of the two debdiff
  patches attached

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-desktop3/+bug/1703690/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp