gdm init script should support the 'status' action

[Andres Rodriguez]

Binary package hint: gdm

GDM need to have 'status' action on init script

[Andres Rodriguez (andreserl)]

Hope it is good :)

[Dustin Kirkland  (kirkland)]

Hi Andres-

Patch looks good to me. My only comment would be that I tend to quote shell variables when referencing them, such as:
 * status_of_proc -p "$PIDFILE" "$DAEMON" gdm && exit 0 || exit $?

I suggest that you subscribe ubuntu-main-sponsors, and bug someone with core-dev privileges if it hasn't been sponsored within a week or so.

Good work, thanks!
:-Dustin

[Andres Rodriguez (andreserl)]

Ok, modified debdiff with your suggestions, and suscribing to ubuntu-main-sponsors

[Andres Rodriguez (andreserl)]

Thanks for the review btw!! :)

[Sebastien Bacher (seb128)]

the change looks good, thanks for your work, it'll be applied to the next upload

[Launchpad Janitor (janitor)]

This bug was fixed in the package gdm - 2.20.7-0ubuntu3

---------------
gdm (2.20.7-0ubuntu3) intrepid; urgency=low

  [ Martin Pitt ]
  * Add debian/patches/67_guest-account.patch: If a flexisession with user
    name ":guest" is requested (which is an impossible system user name), and
    guest session setup/teardown scripts are available in
    /usr/share/gdm/guest-session/ (shipped by separate gdm-guest-user
    package), run a guest session without requiring a password. This
    flexiserver command requires standard gdm authentication, and thus can
    only be run by users who are already logged into the machine.
    Guest sessions call /usr/share/gdm/guest-session/Xsession instead of
    /etc/gdm/Xsession, so that we can wrap AppArmor (or other MAC system)
    rules around it.
    (https://wiki.ubuntu.com/DesktopTeam/Specs/Intrepid/GuestAccount)

  [ Andres Rodriguez ]
  * debian/control.in: Depend on lsb >= 3.2-14, which has the
    status_of_proc() function.
  * debian/gdm.init: Add the 'status' action (LP: #251375).

 -- Martin Pitt <email address hidden> Tue, 29 Jul 2008 11:51:29 +0200