Bug 18681

Summary:	chksession is unable to determine the window manager
Product:	Mageia	Reporter:	David Walser <luigiwalser>
Component:	RPM Packages	Assignee:	Mageia Bug Squad <bugsquad>
Status:	RESOLVED DUPLICATE	QA Contact:
Severity:	normal
Priority:	release_blocker	CC:	jani.valimaa, mageia, mageia, nicolas.salguero, thierry.vignaud, westel
Version:	Cauldron
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	desktop-common-data-6.0-2.mga6.src.rpm	CVE:
Status comment:

Description David Walser 2016-06-10 17:19:58 CEST

/usr/sbin/chksession uses the files in /etc/X11/wmsession.d to determine which window managers are available, as well as, when called as chksession -F (as in from /etc/X11/Xsession), to determine which one is in use.  Since we no longer use wmsession.d, this script needs to be updated to use the new location for window managers that are installed.

One result of this bug is that in IceWM, mate-polkit is not run, since /etc/X11/xinit.d/xdg-autostart is unable to determine that it is running in icewm (the variable is still set to "default").

Comment 1 David Walser 2016-06-10 19:04:16 CEST

/usr/share/xsessions is the new directory it needs to parse.  It currently doesn't know how to parse this directory, even if you pass it with --dir=.

CC'ing wally and neoclust, who worked on the transition to /usr/share/xsessions.

CC: (none) => jani.valimaa, mageia

Comment 2 Nicolas Salguero 2016-06-13 11:03:50 CEST

Hi,

I created bug 17617 with a proposed patch to solve that problem (which also impacts bug 17616 and bug 17913).

Best regards,

Nico.

CC: (none) => nicolas.salguero

Comment 3 Thierry Vignaud 2016-06-13 14:35:21 CEST

Yeah, we need to adapt it for a worls where xsessions is the source of session files

CC: (none) => mageia, thierry.vignaud

Comment 4 David Walser 2016-06-13 14:43:00 CEST

(In reply to Nicolas Salguero from comment #2)
> I created bug 17617 with a proposed patch to solve that problem (which also
> impacts bug 17616 and bug 17913).

Thanks for taking a first crack at this.  The patch doesn't look quite correct yet, so I posted a comment on Bug 17617.  Do we really need three bugs for autologin doesn't work, though?

Comment 5 David Walser 2016-07-22 15:35:25 CEST

*** Bug 18918 has been marked as a duplicate of this bug. ***

CC: (none) => westel

David Walser 2016-08-09 23:57:45 CEST

Blocks: (none) => 15527
Priority: Normal => release_blocker

Comment 6 Thierry Vignaud 2016-08-13 00:25:27 CEST

(In reply to David Walser from comment #4)
No

*** This bug has been marked as a duplicate of bug 17617 ***

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

Samuel Verschelde 2017-01-17 10:29:39 CET

Blocks: 15527 => (none)