Ubuntu Location Provider Here

Merge lp:~lool/ubuntu-location-provider-here/lp-1415029 into lp:ubuntu-location-provider-here

  1. lp-1415029
  2. Merge into trunk
Proposed by Loïc Minier
Status: Merged
Approved by: Thomas Voß
Approved revision: 22
Merged at revision: 22
Proposed branch: lp:~lool/ubuntu-location-provider-here/lp-1415029
Merge into: lp:ubuntu-location-provider-here
Diff against target: 14 lines (+5/-0)
1 file modified
etc/init/ubuntu-location-provider-here-after-wizard.conf (+5/-0)
To merge this branch: bzr merge lp:~lool/ubuntu-location-provider-here/lp-1415029
Reviewer Review Type Date Requested Status
Thomas Voß (community) Approve
Review via email: mp+247745@code.launchpad.net

Commit message

Sleep 5 and restart ubuntu-location-service after starting HERE stack when wizard completes; lets location-service pick up the new provider; LP: #1415029.

Description of the change

Sleep 5 and restart ubuntu-location-service after starting HERE stack when wizard completes; lets location-service pick up the new provider; LP: #1415029.

To post a comment you must log in.
Revision history for this message
Thomas Voß (thomas-voss) wrote :

Ugly, LGTM nevertheless.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'etc/init/ubuntu-location-provider-here-after-wizard.conf'
2--- etc/init/ubuntu-location-provider-here-after-wizard.conf 2014-09-17 13:07:44 +0000
3+++ etc/init/ubuntu-location-provider-here-after-wizard.conf 2015-01-27 17:10:58 +0000
4@@ -17,6 +17,11 @@
5 start ubuntu-location-provider-here-slpgwd
6 #start ubuntu-location-provider-here-posclientd
7 #start ubuntu-espoo-service
8+
9+ # XXX ugly, we need to restart location-service to pickup the now available provider :-/
10+ sleep 5
11+ restart ubuntu-location-service
12+
13 exit 0
14 end script
15

Subscribers

People subscribed via source and target branches