Bug 23899 - Cinnamon session cannot start if cinnamon-screensaver is not installed
Summary: Cinnamon session cannot start if cinnamon-screensaver is not installed
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Joseph Wang
QA Contact:
URL:
Whiteboard:
Keywords: 7beta1
Depends on:
Blocks:
 
Reported: 2018-11-25 23:12 CET by Martin Whitaker
Modified: 2020-06-10 17:48 CEST (History)
4 users (show)

See Also:
Source RPM: cinnamon
CVE:
Status comment:


Attachments

Description Martin Whitaker 2018-11-25 23:12:31 CET
From a fresh install from the Mageia-7-beta1-x86_64 ISO, selecting the Cinnamon DE. On graphical login a pop-up dialogue reports

  Failed to load session "cinnamon"

with the only option being to log out. In .xsession-errors there is the message:

  cinnamon-session[2588]: WARNING: t+0.01587s: Unable to find required component 'cinnamon-screensaver'

Installing the cinnamon-screensaver packages fixes the problem. So it seems there is a missing requires in the cinnamon package.
Comment 1 Marja Van Waes 2018-11-26 09:03:22 CET
Assigning to the registered maintainer.

Assignee: bugsquad => joequant
Source RPM: (none) => cinnamon
Keywords: (none) => 7beta1
CC: (none) => isobuild, marja11

Comment 2 Stig-Ørjan Smelror 2018-12-14 09:56:40 CET
I've pushed an update to task-cinnamon that now requires cinnamon-screensaver to be installed instead of recommended.

Hopefully this will solve this issue.

Cheers,
Stig

CC: (none) => smelror

Comment 3 Martin Whitaker 2018-12-14 10:02:50 CET
I don't think that will help Stig. rpmsrate only pulls in task-cinnamon-minimal. 

joequant deliberately removed the requires on cinnamon-screensaver in r1328343. But unless he responds to this bug report we won't know why.
Comment 4 Stig-Ørjan Smelror 2018-12-14 10:05:29 CET
The cinnamon-screensaver package is only mentioned in task-cinnamon-minimal, which is required in task-cinnamon.
This means that it should pull in cinnamon-screensaver.
Comment 5 Martin Whitaker 2018-12-14 10:13:58 CET
Ah, I only read your words, rather than looking at what you had actually done ;-)
Comment 6 Stig-Ørjan Smelror 2018-12-14 20:13:02 CET
I've just installed Cauldron and Cinnamon and can confirm that I am able to log in to a fully working desktop.

Would appreciate more testers.

Cheers,
Stig
Comment 7 Ben McMonagle 2018-12-14 22:22:06 CET
x86_64 net-install with LXDE as a backup desktop
(mirror => mirror.math.princeton.edu...)

install and reboot to working desktop without issue.

good to go for me, 

thanks Stig.

(now to test the 32 bit)
Comment 8 Ben McMonagle 2018-12-14 23:49:49 CET
i586 net-install single DE install

reboot to working desktop.

good to go for me
Comment 9 Stig-Ørjan Smelror 2018-12-14 23:52:24 CET
Thanks for the testing Ben. Appreciate it.

Don't know if I should close this now or wait for input from Joseph.

Cheers,
Stig
Comment 10 Ben McMonagle 2018-12-15 03:33:10 CET
(In reply to Stig-Ørjan Smelror from comment #9)
> 
> 
> Don't know if I should close this now or wait for input from Joseph.
> 
> Cheers,
> Stig

MartinW as bug creator would normally be the one to do so, or he would defer to Joseph (sorry Martin).

as an aside, I will also be testing upgrade from Mga6 cinnamon, as it has been failing for a similar (task-cinnamon-minimal) reason.

cheers
Comment 11 Martin Whitaker 2018-12-15 18:51:26 CET
Shouldn't the requires really be in the cinnamon package, rather than in task-cinnamon-minimal? Looking back at mga6 it was in both (along with several other requires), which also seems wrong to me.
Comment 12 Stig-Ørjan Smelror 2018-12-15 19:28:05 CET
(In reply to Martin Whitaker from comment #11)
> Shouldn't the requires really be in the cinnamon package, rather than in
> task-cinnamon-minimal? Looking back at mga6 it was in both (along with
> several other requires), which also seems wrong to me.

I don't think Cinnamon works without the screensaver, hence it belongs in task-cinnamon-minimal.
This is the reason we have this bug report.

Unless there is a bug in either Cinnamon or Cinnamon-Screensaver that causes this.

Cheers,
Stig
Comment 13 Martin Whitaker 2018-12-15 19:35:19 CET
(In reply to Stig-Ørjan Smelror from comment #12)
> (In reply to Martin Whitaker from comment #11)
> > Shouldn't the requires really be in the cinnamon package, rather than in
> > task-cinnamon-minimal? Looking back at mga6 it was in both (along with
> > several other requires), which also seems wrong to me.
> 
> I don't think Cinnamon works without the screensaver

Yes, that's my point. If a user installs cinnamon, rather than task-cinnamon-minimal, they'll get something that doesn't work.
Comment 14 Stig-Ørjan Smelror 2018-12-15 19:43:20 CET
The cinnamon package, in task-cinnamon, Requires task-cinnamon-minimal which means that cinnamon-screensaver will always get installed.

Unless you want cinnamon-screensaver enabled directly in the cinnamon package instead of task-cinnamon.

Cheers,
Stig
Comment 15 Martin Whitaker 2018-12-15 20:01:51 CET
Try this:

Do a clean install of another DE, say Xfce. Reboot and login. As root

  urpmi cinnamon

Log out, select Cinnamon, log back in. This bug still occurs. Reselect Xfce, log in, and

  urpmi cinnamon-screensaver

Now cinnamon will work.

The installer selects task-cinnamon-minimal, but the user might not.
Comment 16 Martin Whitaker 2018-12-20 15:26:48 CET
Reducing priority as install is no longer broken (thanks Stig). I still think the packaging is wrong though.

Priority: release_blocker => Normal
Severity: major => normal

Comment 17 Stig-Ørjan Smelror 2018-12-24 22:27:35 CET
(In reply to Martin Whitaker from comment #16)
> Reducing priority as install is no longer broken (thanks Stig). I still
> think the packaging is wrong though.

I agree, but I don't want to step on the maintainers toes by changing his changes. I guess he did that for a good reason, but the is no mention of why in the changelog.

It's easy to change, though.

Besides, we're now quite a few versions behind upstream. I already made a blunder with Mate and don't want to repeat that feat :-)
Comment 18 Martin Whitaker 2020-06-10 17:48:26 CEST
Well, we've had no more complaints, so I think the fix is good enough. Thanks Stig.

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.