FS#34295 - [lightdm] allow local ~/.xsession

Attached to Project: Community Packages
Opened by Damjan Georgievski (damjan) - Wednesday, 13 March 2013, 13:17 GMT
Last edited by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 19:48 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Maxime Gauduin (Alucryd)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

/etc/lightdm/Xsession at the end just execs the arguments it was provided with.

I propose the following patch (attached), to exec a user provided ~/.xsession if it exists and is executable. This is also what XDM does (and kdm did at one time afaik).

A user provided .xsession to run some programs that need to create an environment for the whole DE. /usr/bin/gpg-agent is one example.


The user provided .xsession can end with exec $@ to run the selected environment.
   xsession.patch (0.3 KiB)
This task depends upon

Closed by  Maxime Gauduin (Alucryd)
Wednesday, 13 March 2013, 19:48 GMT
Reason for closing:  Implemented
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 18:29 GMT
I have modified your idea a bit to better follow the layout of the existing script. Also I named the file ~/.Xsession with a capital X :P The new rev should be up in a moment. Tell me if that does it for you.
Comment by Damjan Georgievski (damjan) - Wednesday, 13 March 2013, 18:33 GMT
.Xsession will be inconsistent with legacy X traditions and xdm.
it would be good for general documentation to be consistent throughout login managers.
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 18:39 GMT
Oh you're right (wanted it to have the same case as the wrapper). I might as well rename the wrapper while I'm at it.
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 18:52 GMT
Okay, r6 is up, with lower case x for xsession.

Loading...