Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 622080 - gnome-extra/cinnamon-session: try to avoid upower-pm-utils dependency
Summary: gnome-extra/cinnamon-session: try to avoid upower-pm-utils dependency
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Cinnamon Team (DISABLED)
URL:
Whiteboard:
Keywords:
Depends on: 635292 639618
Blocks: 596988
  Show dependency tree
 
Reported: 2017-06-18 09:37 UTC by Pacho Ramos
Modified: 2018-06-29 22:05 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pacho Ramos gentoo-dev 2017-06-18 09:37:15 UTC
As explained in parent bug, upower-pm-utils is completely dead, most reverse deps are not relying on consolekit1 for suspend support when systemd is not used. Is upstream aware of that change? It would be nice if you could let them know the issue as sooner or later we will need to remove the old upower-pm-utils

Thanks
Comment 1 Pacho Ramos gentoo-dev 2017-06-18 09:41:06 UTC
Well, looking to reverse deps of upower-pm-utils, this seems the last one forcing to run pm-utils for suspending... after this we could finally move to upower + consolekit1/systemd :D
Comment 2 Kristian Fiskerstrand (RETIRED) gentoo-dev 2017-06-20 16:53:25 UTC
(In reply to Pacho Ramos from comment #0)
> As explained in parent bug, upower-pm-utils is completely dead, most reverse
> deps are not relying on consolekit1 for suspend support when systemd is not
> used. Is upstream aware of that change? It would be nice if you could let
> them know the issue as sooner or later we will need to remove the old
> upower-pm-utils
> 
> Thanks

I haven't been in contact with upstream on it at least (and likely won't on this issue), but there might be a possibility to replace it with sys-auth/elogind if switching the config below to true.

As for cinnamon I'm more interested in stable usage than too much fiddling around though..

$ gsettings list-recursively org.cinnamon.desktop.session| grep logind
org.cinnamon.desktop.session session-manager-uses-logind false
org.cinnamon.desktop.session settings-daemon-uses-logind false
org.cinnamon.desktop.session screensaver-uses-logind false
Comment 3 Andreas Sturmlechner gentoo-dev 2017-10-04 19:15:32 UTC
It can't use upower? Because other cinnamon* packages seem to be fine with it.
Comment 4 Larry the Git Cow gentoo-dev 2017-11-28 18:48:11 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=67b7d1e8770ba7350a088c0c154032a15f6767df

commit 67b7d1e8770ba7350a088c0c154032a15f6767df
Author:     Kristian Fiskerstrand <k_f@gentoo.org>
AuthorDate: 2017-11-28 18:45:20 +0000
Commit:     Kristian Fiskerstrand <k_f@gentoo.org>
CommitDate: 2017-11-28 18:47:56 +0000

    gnome-extra/cinnamon-session: use upower instead of upower-pm-utils
    
    With version 3.6 and after switch to elogind, I don't have concerns
    switching to upower instead of upower-pm-utils
    
    Bug: https://bugs.gentoo.org/622080
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 .../cinnamon-session-3.6.1-r3.ebuild               | 68 ++++++++++++++++++++++
 1 file changed, 68 insertions(+)}
Comment 5 Kristian Fiskerstrand (RETIRED) gentoo-dev 2017-11-28 18:49:41 UTC
Reopening, as this likely requires stabilization due to blocker.

This is consequently blocked by elogind stabilization ( https://wiki.gentoo.org/wiki/Elogind )
Comment 6 laurantino 2017-11-29 18:30:08 UTC
Hi,

The cinnamon-session-3.6.1-elogind patch breaks the compilation when using systemd.

I think the line #46 :

+  PKG_CHECK_MODULES(LOGIND, [gio-unix-2.0], [have_logind=yes], [have_logind=no])

should be replaced by

+  PKG_CHECK_MODULES(LOGIND, [libsystemd >= 183], [have_logind=yes], [have_logind=no])
Comment 7 Kristian Fiskerstrand (RETIRED) gentoo-dev 2017-11-29 19:42:10 UTC
(In reply to charlot from comment #6)
> Hi,
> 
> The cinnamon-session-3.6.1-elogind patch breaks the compilation when using
> systemd.
> 
> I think the line #46 :
> 
> +  PKG_CHECK_MODULES(LOGIND, [gio-unix-2.0], [have_logind=yes],
> [have_logind=no])
> 
> should be replaced by
> 
> +  PKG_CHECK_MODULES(LOGIND, [libsystemd >= 183], [have_logind=yes],
> [have_logind=no])

Please open a separate bug for the systemd support issue, including libsystemd in the call at that location will break elogind again
Comment 8 Kristian Fiskerstrand (RETIRED) gentoo-dev 2017-11-29 20:04:20 UTC
(In reply to Kristian Fiskerstrand from comment #7)
> (In reply to charlot from comment #6)
> > Hi,
> > 
> > The cinnamon-session-3.6.1-elogind patch breaks the compilation when using
> > systemd.
> > 
> > I think the line #46 :
> > 
> > +  PKG_CHECK_MODULES(LOGIND, [gio-unix-2.0], [have_logind=yes],
> > [have_logind=no])
> > 
> > should be replaced by
> > 
> > +  PKG_CHECK_MODULES(LOGIND, [libsystemd >= 183], [have_logind=yes],
> > [have_logind=no])
> 
> Please open a separate bug for the systemd support issue, including
> libsystemd in the call at that location will break elogind again

pushed -r4 that should solve it again
Comment 9 Pacho Ramos gentoo-dev 2018-06-29 22:05:33 UTC
this was fixed some time ago