Bug 17617 - [mga6-dev1] autologin seems to need the files in /etc/X11/wmsession.d
Summary: [mga6-dev1] autologin seems to need the files in /etc/X11/wmsession.d
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: release_blocker critical
Target Milestone: ---
Assignee: Thierry Vignaud
QA Contact:
URL:
Whiteboard:
Keywords: PATCH
: 18681 (view as bug list)
Depends on:
Blocks: 17554 17616
  Show dependency treegraph
 
Reported: 2016-01-25 11:13 CET by Nicolas Salguero
Modified: 2017-01-17 10:29 CET (History)
4 users (show)

See Also:
Source RPM: desktop-common-data-6.0-2.mga6
CVE:
Status comment:


Attachments
A patch to modify chksession behaviour (5.23 KB, patch)
2016-02-12 13:25 CET, Nicolas Salguero
Details | Diff
patch to fix chksession (1.43 KB, patch)
2016-08-15 23:11 CEST, David Walser
Details | Diff

Description Nicolas Salguero 2016-01-25 11:13:33 CET
Hi,

In MCC, when I try to enable autologin, the list for the default desktop seems to be generated with the files present in /etc/X11/wmsession.d but, with Mga6, there is not more file in that directory.

Best regards,

Nico.
Comment 1 Marja Van Waes 2016-01-26 08:49:34 CET
Confirming, that in the "Set up autologin to automatically log in" screen, only "default" can be chosen.

CC: (none) => marja11
Assignee: bugsquad => thierry.vignaud
Source RPM: (none) => drakxtools

Marja Van Waes 2016-01-26 10:15:55 CET

Blocks: (none) => 17616

Comment 2 Nicolas Salguero 2016-02-12 13:25:22 CET
Created attachment 7451 [details]
A patch to modify chksession behaviour

Hi,

As chksession no longer needs to generate files from "/etc/X11/wmsession.d", I propose a patch to modify chksession behaviour:
- Remove all generation options.
- Modify how the list of possible sessions is created by using "/usr/share/xsessions" instead of "/etc/X11/wmsession.d" and putting IceWM as the last option.

Best regards,

Nico.
Nicolas Salguero 2016-02-12 13:25:53 CET

Keywords: (none) => PATCH

Marja Van Waes 2016-02-14 16:14:17 CET

Blocks: (none) => 17554

Comment 3 David Walser 2016-06-13 14:41:58 CEST
Putting IceWM looks incorrect.  According to the wmsession.d files in Mageia 5, it appears it should only be after the "main" DEs, but before all of the niche ones.

Does /usr/share/xsessions have any provision for specifying an ordering, or do we have to hardcode it into the chksession script?

For reference, here is the ordering from Mageia 5:

/etc/X11/wmsession.d/01KDE
/etc/X11/wmsession.d/02GNOME
/etc/X11/wmsession.d/02Plasma
/etc/X11/wmsession.d/03GNOME_CLASSIC
/etc/X11/wmsession.d/03WindowMaker
/etc/X11/wmsession.d/04LXDE
/etc/X11/wmsession.d/06Xfce
/etc/X11/wmsession.d/07IceWM
/etc/X11/wmsession.d/09fvwm2
/etc/X11/wmsession.d/10cinnamon
/etc/X11/wmsession.d/10fvwm-crystal
/etc/X11/wmsession.d/10MATE
/etc/X11/wmsession.d/11cinnamon2d
/etc/X11/wmsession.d/12swm
/etc/X11/wmsession.d/13LXQT
/etc/X11/wmsession.d/15afterstep
/etc/X11/wmsession.d/15Kodi
/etc/X11/wmsession.d/16fluxbox
/etc/X11/wmsession.d/19awesome
/etc/X11/wmsession.d/22Matchbox
/etc/X11/wmsession.d/23E18
/etc/X11/wmsession.d/26openbox
/etc/X11/wmsession.d/30pekwm
/etc/X11/wmsession.d/31i3
/etc/X11/wmsession.d/40dwm
/etc/X11/wmsession.d/41spectrwm
/etc/X11/wmsession.d/99MythFrontend

CC: (none) => luigiwalser

Comment 4 Nicolas Salguero 2016-06-13 14:51:46 CEST
Regarding IceWM, I did that to also solve bug 17554 but that bug was solved without the need of that patch so I think that the patch could be stripped from the part corresponding to IceWM.
Thierry Vignaud 2016-08-04 11:48:37 CEST

Assignee: thierry.vignaud => bugsquad
Source RPM: drakxtools => desktop-common-data-6.0-2.mga6

David Walser 2016-08-09 23:55:10 CEST

Priority: Normal => release_blocker

David Walser 2016-08-09 23:55:32 CEST

Blocks: (none) => 15527

Comment 5 Thierry Vignaud 2016-08-13 00:25:27 CEST
*** Bug 18681 has been marked as a duplicate of this bug. ***
Thierry Vignaud 2016-08-13 01:39:04 CEST

Status: NEW => ASSIGNED
CC: (none) => thierry.vignaud
Assignee: bugsquad => thierry.vignaud

Comment 6 Mageia Robot 2016-08-13 02:04:01 CEST
commit 0a47f2c60d821432a22c3c4eb0df1022bc92d32a
Author: Nicolas Salguero <nicolas.salguero@...>
Date:   Sat Aug 13 01:00:01 2016 +0200

    stop generating session files (mga#17617)
---
 Commit Link:
   http://gitweb.mageia.org/software/desktop/common-data/commit/?id=0a47f2c60d821432a22c3c4eb0df1022bc92d32a
Comment 7 Mageia Robot 2016-08-13 02:04:03 CEST
commit a8aec7a0e6f505ed750ce61991012d6d47cc7bda
Author: Nicolas Salguero <nicolas.salguero@...>
Date:   Sat Aug 13 01:02:31 2016 +0200

    use sessions from /usr/share/xsessions/ (mga#17617)
    
    instead of from /etc/X11/wmsession.d/
    v2 (Thierry Vignaud):
    - prune useless stuff (removed by earlier perl_checker commit)
    - do not reparse twice the directory, just keep the existing order logic
      (which is slowly going towards /dev/null)
---
 Commit Link:
   http://gitweb.mageia.org/software/desktop/common-data/commit/?id=a8aec7a0e6f505ed750ce61991012d6d47cc7bda
Comment 8 Thierry Vignaud 2016-08-13 02:06:48 CEST
Fixed in git.
I've:
- split your changes in 2 logical commits
- remove useless variables
- kept the previous logic
- excluded the icewm sort

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

Comment 9 David Walser 2016-08-15 19:31:12 CEST
Installing this update has broken IceWM completely.  When logging into it, it immediately sends me back to the login screen (xdm in this case).

Status: RESOLVED => REOPENED
Resolution: FIXED => (none)
Severity: normal => critical

Comment 10 David Walser 2016-08-15 19:34:59 CEST
I just built desktop-common-data 6.1 locally and confirmed that reverting to that makes IceWM runnable again.
Comment 11 David Walser 2016-08-15 23:11:04 CEST
Created attachment 8348 [details]
patch to fix chksession

I attached a patch that fixes chksession.  It restores the -x option and setting the $script{$xession} variable, which is needed as /etc/X11/Xsession still makes use of these.

Once that's applied, /etc/X11/xinit.d/xdg-autostart needs to be fixed to say IceWM instead of icewm, and it will work.

The only thing left that hasn't been handled correctly here as far as I can tell is the ordering.
Comment 12 Olav Vitters 2016-08-16 17:14:02 CEST
Various display managers also support wayland sessions; they're in a different directory.

CC: (none) => olav

Comment 13 Thierry Vignaud 2016-08-22 22:04:43 CEST
Just commit your fix and ask for a freeze exception...
Ulrich Beckmann 2016-08-22 22:23:48 CEST

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=19234

Comment 14 David Walser 2016-08-22 23:28:28 CEST
Sorry, I don't know how to commit for software.
Comment 15 Mageia Robot 2016-08-23 18:00:01 CEST
commit 46e66c480c4447e014c51e7283f24186e4978a62
Author: daviddavid <geiger.david68210@...>
Date:   Tue Aug 23 17:59:40 2016 +0200

    fix chksession from David Walser (mga#17617)
---
 Commit Link:
   http://gitweb.mageia.org/software/desktop/common-data/commit/?id=46e66c480c4447e014c51e7283f24186e4978a62
Comment 16 David Walser 2016-08-23 19:13:19 CEST
desktop-common-data and xdg-compliance updates submitted, which should fix some issues and IceWM should work correctly.

The only remaining issue here that I know of is that we no longer have a mechanism for ordering, which is used to choose a correct default when multiple desktops are installed but no preferred one is listed in /etc/sysconfig/desktop.
Comment 17 Rémi Verschelde 2016-08-29 11:36:55 CEST
(In reply to David Walser from comment #16)
> The only remaining issue here that I know of is that we no longer have a
> mechanism for ordering, which is used to choose a correct default when
> multiple desktops are installed but no preferred one is listed in
> /etc/sysconfig/desktop.

That would be bug 19234 (so maybe this one could be closed and  bug 19234 upgraded to release blocker, or the other way around, as you prefer).
Comment 18 Thierry Vignaud 2016-08-29 13:57:52 CEST
Yeah let's make each bug report target a specific issue.
This one was about chksession no more listing sessions

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

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

Blocks: 15527 => (none)


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