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

Bug #360989 reported by Matt Zimmerman
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gdm (Ubuntu)
Invalid
Undecided
Unassigned
Jaunty
Won't Fix
Undecided
Unassigned
notify-osd (Ubuntu)
Fix Released
Medium
Loïc Minier
Jaunty
Fix Released
Medium
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
ProcEnviron:
 LC_COLLATE=C
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/zsh
SourcePackage: notify-osd
Uname: Linux 2.6.28-11-generic i686

Revision history for this message
Matt Zimmerman (mdz) wrote :
tags: added: ubuntu-unr
Loïc Minier (lool)
description: updated
Revision history for this message
Loïc Minier (lool) wrote :

GDMSESSION is "default" here

Revision history for this message
Loïc Minier (lool) wrote :

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

Revision history for this message
Loïc Minier (lool) wrote :

Please attach your gdm.conf

Revision history for this message
Matt Zimmerman (mdz) wrote :
Revision history for this message
Matt Zimmerman (mdz) wrote :

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

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?

Revision history for this message
Matt Zimmerman (mdz) wrote :
Revision history for this message
Matt Zimmerman (mdz) wrote :

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

Revision history for this message
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
Revision history for this message
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

Revision history for this message
Loïc Minier (lool) wrote :

Fix is in my ppa

Revision history for this message
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"

Revision history for this message
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
Revision history for this message
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/org.freedesktop.Notifications.service.in: 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
Revision history for this message
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
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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