notification-daemon started instead of notify-osd in Netbook Remix

Bug #360989 reported by Matt Zimmerman on 2009-04-14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gdm (Ubuntu)
notify-osd (Ubuntu)
Loïc Minier
Loïc Minier

Bug Description

Binary package hint: notify-osd

On an Acer Aspire One, installed with UNR beta-ish and upgraded to current, I noticed that notify-osd was not running. At Loic's suggestion, I checked /usr/share/dbus-1/services/org.freedesktop.Notifications.service and noticed that $GDMSESSION in my session was set to "default.desktop" while the test is only for "default":

Exec=/bin/sh -c "if [ ! -x /usr/lib/notification-daemon/notification-daemon ] || [ $GDMSESSION = gnome ] || [ $GDMSESSION = default -a $(basename `readlink /etc/alternatives/x-session-manager`) = gnome-session ]; then exec /usr/lib/notify-osd/notify-osd; else exec /usr/lib/notification-daemon/notification-daemon; fi"

I think that test is failing, and so notification-daemon is being run.

ProblemType: Bug
Architecture: i386
DistroRelease: Ubuntu 9.04
Package: notify-osd 0.9.11-0ubuntu1
 PATH=(custom, user)
SourcePackage: notify-osd
Uname: Linux 2.6.28-11-generic i686

Matt Zimmerman (mdz) wrote :
tags: added: ubuntu-unr
Loïc Minier (lool) on 2009-04-14
description: updated
Loïc Minier (lool) wrote :

GDMSESSION is "default" here

Loïc Minier (lool) wrote :

My /etc/gdm/gdm.conf says DefaultSession=default.desktop

Loïc Minier (lool) wrote :

Please attach your gdm.conf

Matt Zimmerman (mdz) wrote :
Matt Zimmerman (mdz) wrote :

# This is the default .desktop session. One of the ones in SessionDesktopDir

I'm not sure how that gets set, but can see that it's not the default.

I think I've used desktop-switcher on this system; could that be related?

Matt Zimmerman (mdz) wrote :
Matt Zimmerman (mdz) wrote :

<njpatel> mdz: desktop-switcher doesn't touch session files (just /gnome/session/required_components gconf list)

Loïc Minier (lool) wrote :

So we don't know how the .dmrc was created / put in this state, but clearly:
- gdm shouldn't use default.desktop in the default config (but default)
- gdm should strip .desktop even for DefaultSession, not only for user defined sessions
- notify-osd should workaround this issue by stripping .desktop (what I uploaded)

Changed in notify-osd (Ubuntu):
assignee: nobody → lool
importance: Undecided → Medium
milestone: none → ubuntu-9.04
status: New → Fix Committed
Loïc Minier (lool) wrote :

I tested the notify-osd fix on a working system (didn't regress), and then I moved the working .dmrc out of the way to be in a situation like mdz' and it would still use notify-osd; so I think the fix is correct

Loïc Minier (lool) wrote :

Fix is in my ppa

Loïc Minier (lool) wrote :

What remains to be done on gdm:
- make sure .desktop is stripped from configured user and system configs so that a stripped Session is written to .dmrc
- fix the gdm.conf system config to set DefaultSession to "default" instead of "default.desktop"

Mirco Müller (macslow) wrote :

This is a distro-side issue. The needed patch is carried within the UNR distro and not included within notify-osd trunk.

Changed in notify-osd:
assignee: nobody → Mirco Müller (macslow)
status: New → Invalid
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package notify-osd - 0.9.11-0ubuntu2

notify-osd (0.9.11-0ubuntu2) jaunty; urgency=low

  * data/ strip .desktop from
    GDMSESSION before checking it; LP: #360989.

 -- Loic Minier <email address hidden> Tue, 14 Apr 2009 13:44:56 +0200

Changed in notify-osd (Ubuntu Jaunty):
status: Fix Committed → Fix Released
Steve Langasek (vorlon) wrote :

AIUI, this bug is fixed in notify-osd and no further changes are needed to gdm.

Changed in gdm (Ubuntu Jaunty):
status: New → Invalid
Loïc Minier (lool) wrote :

wontfixing the jaunty task, but should be fixed in karmic

Changed in gdm (Ubuntu Jaunty):
status: Invalid → Won't Fix
no longer affects: notify-osd
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers