After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779898 - weather: Handle GeoClue not responding
weather: Handle GeoClue not responding
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: calendar
3.23.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2017-03-11 13:30 UTC by Sebastian Keller
Modified: 2017-03-11 19:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
weather: Handle GeoClue not responding (2.71 KB, patch)
2017-03-11 13:30 UTC, Sebastian Keller
committed Details | Review

Description Sebastian Keller 2017-03-11 13:30:39 UTC
Created attachment 347703 [details] [review]
weather: Handle GeoClue not responding

Sometimes GeoClue seems to be unresponsive which causes gnome-shell to display the weather of New York City, regardless of the most recently used location in gnome-weather which is supposed to be used as a fallback. I have not yet investigated what causes the issue on the GeoClue side. What happens in gnome-shell is that the callback of Geoclue.Simple.new is never being called due to GeoClue being stuck. This means that _gclueFailed remains false, despite GeoClue not responding. The fallback code however checks for _gclueFailed and only sets the location if GeoClue has failed. Because neither code path actually sets a location in this case, the default location of gweather is being used instead, which happens to be New York City.
Comment 1 Florian Müllner 2017-03-11 15:15:57 UTC
Review of attachment 347703 [details] [review]:

Alternatively, we could all relocate to New York City ...

::: js/misc/weather.js
@@ +166,2 @@
         if (this._useAutoLocation) {
+            this._startGClueService();

Style nit: Remove braces now that both blocks have just one line
Comment 2 Florian Müllner 2017-03-11 19:53:36 UTC
Attachment 347703 [details] pushed as ce5875f - weather: Handle GeoClue not responding