FS#45445 - [lightdm] Loads xprofile before dbus session is launched

Attached to Project: Community Packages
Opened by Richard Adenling (dreeze) - Wednesday, 24 June 2015, 08:29 GMT
Last edited by Maxime Gauduin (Alucryd) - Friday, 26 June 2015, 10:46 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Maxime Gauduin (Alucryd)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
/etc/lightdm/Xsession loads the profile scripts (.profile, .xprofile, ...) before loading the xinitrc scripts (found in /etc/X11/xinit/xinitrc.d). This means that anything started in xprofile won't have DBUS_SESSION_BUS_ADDRESS set since it's set by /etc/X11/xinit/xinitrc.d/30-dbus.sh.

In my particular case, this means that dbus-send calls which I run from xbindkeys don't work if I start xbindkeys in .xprofile.

I solved this by moving the xinitrc section in /etc/lightdm/Xsession to the top of the file. I see no reason for loading the xinitrc scripts last, so my suggestion is to move the loading of the scripts to the top of Xsession.

Additional info:
* lightdm 1:1:14.0-3
This task depends upon

Closed by  Maxime Gauduin (Alucryd)
Friday, 26 June 2015, 10:46 GMT
Reason for closing:  Implemented
Comment by Maxime Gauduin (Alucryd) - Thursday, 25 June 2015, 12:22 GMT
profile and xprofile contain a set of env variables, some of which are likely needed by programs launched by, eg, xinitrc. Please invoke xbindkeys in your xinitrc instead.
Comment by Richard Adenling (dreeze) - Friday, 26 June 2015, 04:30 GMT
  • Field changed: Percent Complete (100% → 0%)
lightdm does not load the users .xinitrc so that wouldn't be possible. Upstream has solved this by loading .xsession after the system xinitrc have be loaded. There is no support for xsession in the archlinux lightdm package which is why I still would consider this a bug.
Comment by Maxime Gauduin (Alucryd) - Friday, 26 June 2015, 06:49 GMT
There's still the option of using /etc/X11/xinit/xinitrc.d/ but fair enough. I just pushed 1:1.14.0-4 to [community-testing], it loads ~/.xsession, but only if it's +x (it's -x by default), so that behavior will stay the same as before for most people. Can you give it a shot?
Comment by Richard Adenling (dreeze) - Friday, 26 June 2015, 08:17 GMT
Yes, using xinitrc.d would've been possible, but I think running a thing like xbindkeys should be possible without modifying /etc. Anyways, I tested 1:1.14.0-4, renamed .xprofile to .xsession and it works perfectly. Thank you!
Comment by Maxime Gauduin (Alucryd) - Friday, 26 June 2015, 10:46 GMT
Thanks for testing! I'll move it in [community] in a few days then.

Loading...