gdm init script should support the 'status' action
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| gdm (Ubuntu) |
Wishlist
|
Andres Rodriguez |
Bug Description
Binary package hint: gdm
GDM need to have 'status' action on init script
Related branches
Changed in gdm: | |
assignee: | nobody → andreserl |
Dustin Kirkland (kirkland) wrote : | #2 |
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-
Good work, thanks!
:-Dustin
Andres Rodriguez (andreserl) wrote : | #3 |
Ok, modified debdiff with your suggestions, and suscribing to ubuntu-
Andres Rodriguez (andreserl) wrote : | #4 |
Thanks for the review btw!! :)
Changed in gdm: | |
status: | In Progress → Confirmed |
importance: | Undecided → Wishlist |
Changed in gdm: | |
status: | Confirmed → Triaged |
Sebastien Bacher (seb128) wrote : | #5 |
the change looks good, thanks for your work, it'll be applied to the next upload
Changed in gdm: | |
status: | Triaged → Fix Committed |
Launchpad Janitor (janitor) wrote : | #6 |
This bug was fixed in the package gdm - 2.20.7-0ubuntu3
---------------
gdm (2.20.7-0ubuntu3) intrepid; urgency=low
[ Martin Pitt ]
* Add debian/
name ":guest" is requested (which is an impossible system user name), and
guest session setup/teardown scripts are available in
/usr/
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/
/etc/
rules around it.
(https:/
[ Andres Rodriguez ]
* debian/control.in: Depend on lsb >= 3.2-14, which has the
status_
* debian/gdm.init: Add the 'status' action (LP: #251375).
-- Martin Pitt <email address hidden> Tue, 29 Jul 2008 11:51:29 +0200
Changed in gdm: | |
status: | Fix Committed → Fix Released |
Hope it is good :)