Software Updater

Merge lp://qastaging/~nskaggs/update-manager/fix-for-702418 into lp://qastaging/update-manager

fix-for-702418
Merge into main

Proposed by Nicholas Skaggs on 2011-11-21

Status:	Merged
Merged at revision:	2278
Proposed branch:	lp://qastaging/~nskaggs/update-manager/fix-for-702418
Merge into:	lp://qastaging/update-manager
Diff against target:	60 lines (+10/-34) 1 file modified UpdateManager/Core/utils.py (+10/-34)
To merge this branch:	bzr merge lp://qastaging/~nskaggs/update-manager/fix-for-702418
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Michael Vogt		2011-11-21	Pending
Review via email: mp+82946@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2011-11-04.

Description of the change

This is a fix for bug 702418. I added a check to prevent the calling the old dbus interface check for gnome-power-manager under any desktop environment that is not gnome. This will instead simply call the FreeDesktop powermanager without attempting the old check.

Revision history for this message

Michael Vogt (mvo) wrote on 2011-11-07: Posted in a previous version of this proposal

Thanks for your branch!

Testing for gnome in the env is not quite sufficient as the default session will use
"XDG_CURRENT_DESKTOP=Unity" and DESKTOP_SESSION=ubuntu these days. Actually this makes
me wonder if calling the old interface can be dropped entirely? Could you maybe check
if gnome now supports the freedesktop one?

review: Needs Information

Revision history for this message

Nicholas Skaggs (nskaggs) wrote on 2011-11-07: Posted in a previous version of this proposal

Michael, my idea in only checking for "gnome" was that only gnome 2.x
would need the old gnome dbus check. Unity and gnome shell both
support the the new freedesktop interace, as does kde and xfce and
lxde.

I'll go ahead and drop the old interface call completely and update
the branch. I will also attempt to confirm with the gnome developers
that we are free to drop legacy support. Thanks,

Nicholas

On Mon, Nov 7, 2011 at 4:16 AM, Michael Vogt <email address hidden> wrote:
> Review: Needs Information
>
> Thanks for your branch!
>
> Testing for gnome in the env is not quite sufficient as the default session will use
> "XDG_CURRENT_DESKTOP=Unity" and DESKTOP_SESSION=ubuntu these days. Actually this makes
> me wonder if calling the old interface can be dropped entirely? Could you maybe check
> if gnome now supports the freedesktop one?
> --
> https://code.launchpad.net/~nskaggs/update-manager/fix-for-702418/+merge/81298
> You are the owner of lp:~nskaggs/update-manager/fix-for-702418.
>

Revision history for this message

Michael Vogt (mvo) wrote on 2011-11-08: Posted in a previous version of this proposal

Thanks Nocholas! I think dropping the old interface is indeed the best option. We just need to make sure that that works on lucid, but I bet it does :)

Revision history for this message

Nicholas Skaggs (nskaggs) wrote on 2011-11-09: Posted in a previous version of this proposal

Pushed a commit that removed the old interface completely. Are we
wanting backport the fix to lucid or ?

Nicholas

On Tue, Nov 8, 2011 at 5:54 AM, Michael Vogt <email address hidden> wrote:
> Thanks Nocholas! I think dropping the old interface is indeed the best option. We just need to make sure that that works on lucid, but I bet it does :)
> --
> https://code.launchpad.net/~nskaggs/update-manager/fix-for-702418/+merge/81298
> You are the owner of lp:~nskaggs/update-manager/fix-for-702418.
>