Menu should list users if more than 1, fewer than 7 are on the system
Bug #422052 reported by
Ted Gould
This bug affects 11 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Session Menu |
Fix Released
|
High
|
Cody Russell | ||
Ubuntu |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
It should in a submenu.
Blocked on GDM support of listing users over DBus.
Relevant portion of the spec:
https:/
Related branches
lp:~bratsche/indicator-session/users-service
- Ted Gould (community): Approve
- Indicator Applet Developers: Pending requested
-
Diff: 1768 lines9 files modifiedAUTHORS (+2/-0)
src/Makefile.am (+27/-2)
src/dbus-shared-names.h (+8/-6)
src/status-service.c (+8/-8)
src/users-service-dbus.c (+1147/-0)
src/users-service-dbus.h (+81/-0)
src/users-service.c (+191/-62)
src/users-service.list (+1/-0)
src/users-service.xml (+56/-0)
lp:~indicator-applet-developers/indicator-session/ubuntu
- Ken VanDine: Pending requested
-
Diff: 201 lines (+75/-13)7 files modifiedChangeLog (+44/-0)
configure (+1/-1)
configure.ac (+1/-1)
debian/changelog (+9/-0)
src/apt-transaction.c (+0/-1)
src/apt-watcher.c (+2/-1)
src/settings-helper.c (+18/-9)
Changed in indicator-session: | |
importance: | Undecided → Medium |
milestone: | none → ubuntu-9.10-ui-freeze |
status: | New → Confirmed |
Changed in indicator-session: | |
importance: | Medium → Critical |
assignee: | nobody → Robert Ancell (robert-ancell) |
importance: | Critical → High |
importance: | High → Medium |
description: | updated |
summary: |
- Menu should list users if fewer than 6 are on the system + Menu should list users if more than 1, fewer than 7 are on the system |
Changed in indicator-session: | |
assignee: | Ted Gould (ted) → Cody Russell (bratsche) |
status: | Confirmed → In Progress |
Changed in indicator-session: | |
milestone: | ubuntu-9.10-ui-freeze → ubuntu-9.10-beta-freeze |
Changed in indicator-session: | |
status: | In Progress → Fix Committed |
Changed in indicator-session: | |
milestone: | ubuntu-9.10-beta-freeze → 0.1.5 |
Changed in indicator-session: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Does this really need to come from gdm? We can certainly add a d-bus call there, but why not just iterate over getpwent() and filter out UIDs < 500 (the system users) and >= 65534 (nobody)? That's more or less what gdm does as well.
In shell, you can do that with
getent passwd | FS=: awk -F: '($3 >= 500 && $3 < 65534) { print }'
but in C, using getpwent() is more efficient.