[MIR] geoclue
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| geoclue (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Binary package hint: geoclue
A new feature for natty in indicator-datetime is setting the timezone based on location. For now we are only depend on location provided by the ubuntu geoip service. ubuntu-geoip and indicator-datetime depend on geoclue. I have reviewed the packaging, which is maintained in debian and I don't see a need to make any changes now. I have also set myself to get bug notifications on the geoclue ubuntu bugs, I'll be maintaining that for the ubuntu desktop team. The bug flow is very low, both in ubuntu and debian and it meets the requirements for main inclusion.
The package is split nicely so all the providers are in different binaries, for natty we will only ship geoclue, and libgeoclue0 by default and libgeoclue-dev is a build depends for ubuntu-geoip and indicator-datetime. In future cycles we will look at including additional providers.
| Changed in geoclue (Ubuntu): | |
| assignee: | nobody → Michael Terry (mterry) |
| status: | New → In Progress |
| Michael Terry (mterry) wrote | #1 |
| Changed in geoclue (Ubuntu): | |
| assignee: | Michael Terry (mterry) → nobody |
| status: | In Progress → Incomplete |
| Ken VanDine (ken-vandine) wrote | #2 |
| Changed in geoclue (Ubuntu): | |
| status: | Incomplete → New |
| Michael Terry (mterry) wrote | #3 |
| Changed in geoclue (Ubuntu): | |
| status: | New → Fix Committed |
| Matthias Klose (doko) wrote | #4 |
| Changed in geoclue (Ubuntu): | |
| status: | Fix Committed → Incomplete |
| Artur Rona (ari-tczew) wrote | #5 |
| Ken VanDine (ken-vandine) wrote | #6 |
| Changed in geoclue (Ubuntu): | |
| status: | Incomplete → New |
| Michael Terry (mterry) wrote | #7 |
| Changed in geoclue (Ubuntu): | |
| status: | New → Fix Committed |
| Sebastien Bacher (seb128) wrote | #8 |
| Changed in geoclue (Ubuntu): | |
| status: | Fix Committed → Fix Released |
Looks good, very good packaging and good maintainership. I see that we have some Ubuntu modifications, so we should probably rename Vcs-* fields to XS-Debian-Vcs-* and update the Maintainer field. Not a blocker though.
One issue that is more serious is debian/copyright. It claims LGPL 2.1+, but the source is actually licensed as LGPL 2+. If that is fixed, I'll approve.