lightdm does not source /etc/X11/Xsession - missing environment for Xsession.d files

[Daniel Hahler]

lightdm does not source /etc/X11/Xsession.

This appears to provide a lot of functions ("message") and vars (USRRESOURCES) for the files in /etc/X11/Xsession.d.

This causes for example /etc/X11/Xsession.d/30x11-common_xresources to not work as expected, and the ~/.Xresources being ignored/not merged.

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: lightdm 1.0.6-0ubuntu1.2
ProcVersionSignature: Ubuntu 3.0.0-14.23-generic 3.0.9
Uname: Linux 3.0.0-14-generic x86_64
NonfreeKernelModules: nvidia
ApportVersion: 1.23-0ubuntu4
Architecture: amd64
Date: Mon Dec 5 11:25:23 2011
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 11.10 "Oneiric Ocelot" - Release amd64 (20111012)
ProcEnviron:
 LC_CTYPE=en_US.UTF-8
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/zsh
SourcePackage: lightdm
UpgradeStatus: No upgrade log present (probably fresh install)

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[OmegaPhil (omegaphil)]

Currently I am on Debian Testing, and I am investigating why a Xsession.d script that ran under Ubuntu and gdm didnt under Debian and lightdm. I think this has something to do with that (the script defines the position of the two monitors relative to each other so that this is correctly set at the greeter stage, and stopping cloned outputs).

[Paul Hebble (maceo)]

I think the fix is to add this to /etc/lightdm/lightdm.conf under [SeatDefaults]:

session-wrapper=/etc/X11/Xsession

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "lightdm.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Daniel Hahler (blueyed)]

Thanks Paul for the proposed patch.

Without having investigated I would say that there's a reason for lightdm to have /usr/sbin/lightdm-session and that this should be kept.

It seems like lightdm-session needs to be fixed instead.

[Sebastien Bacher (seb128)]

Ok, I did the change and then reverted part of it, the script already has

"# Load resources
xresourcedir="/etc/X11/Xresources"
if [ -d "$xresourcedir" ]; then
    for file in $xresourcedir/*; do
        echo "Loading resource: $file"
        xrdb -nocpp -merge "$file"
    done
fi
xresourcefile="$HOME/.Xresources"
if [ -f "$xresourcefile" ]; then
    echo "Loading resource: $xresourcefile"
    xrdb -nocpp -merge "$xresourcefile"
fi"

so the resources are loaded, the x11 script do the same without the "nocpp", we could drop those in favor of the x11 script but nocpp is an optimization so we should probably fix the x11 version before doing that

are you sure your .Xresources is not loaded?