default-session code runs before lightdm installation and has no effect

Bug #1048226 reported by Alkis Georgopoulos
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
ubuntu-defaults-builder (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

The default-session.txt settings are applied too early, i.e. in ubuntu-defaults-el-gr.postinst, and they don't have any effect at all because lightdm-set-defaults isn't yet available.
They should probably be applied when the chroot hook script is run and not from the postinst.

# cat desktop/default-session.txt
gnome-fallback

# cat recommends.txt
gnome-session-fallback

# ubuntu-defaults-image --package ubuntu-defaults-el-gr_0.1_all.deb --components main,restricted,universe --ppa ts.sch.gr/ppa
...
[output line 2877] Setting up ubuntu-defaults-el-gr (0.1) ...
...
[output line 5896] Setting up lightdm (1.2.1-0ubuntu1.1) ...
...

# cat chroot/etc/gdm/custom.conf
[daemon]
DefaultSession=gnome-fallback

# cat chroot/etc/lightdm/lightdm.conf
[SeatDefaults]
greeter-session=unity-greeter

The above file is wrong, it should also contain
user-session=gnome-fallback

Tags: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in ubuntu-defaults-builder (Ubuntu):
status: New → Confirmed
Revision history for this message
Ikuya Awashiro (ikuya-fruitsbasket) wrote :

This debdiff may fix this issue.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "fix-1048226.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
Iain Lane (laney) wrote :

Thank you, uploaded!

Changed in ubuntu-defaults-builder (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-defaults-builder - 0.54

---------------
ubuntu-defaults-builder (0.54) wily; urgency=medium

  * debhelper-scripts/postinst-ubuntu-defaults-desktop-session:
      desktop/default-session.txt works properly (LP: #1048226).

 -- Ikuya Awashiro <email address hidden> Mon, 29 Jun 2015 18:21:42 +0900

Changed in ubuntu-defaults-builder (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in ubuntu:
status: New → Confirmed
Mathew Hodson (mhodson)
affects: baltix → ubuntu
Changed in ubuntu-defaults-builder (Ubuntu):
importance: Undecided → Medium
no longer affects: ubuntu
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.