QtLocation OSM based routing does not work any more

Bug #1568439 reported by Michael Zanetti
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Undecided
Pat McGowan
qtlocation-opensource-src (Ubuntu)
Fix Released
Undecided
Timo Jyrinki
qtlocation-opensource-src (Ubuntu RTM)
Fix Released
Undecided
Unassigned

Bug Description

Apparently OSM changed a status code in their server, so all routing operations fail.

This has been fixed in Qt 5.5. Please consider backporting this patch to our Qt 5.4 version:

http://code.qt.io/cgit/qt/qtlocation.git/patch/?id=9969179a06c30b56d3d057aba127686b64ccd659

I have tested it locally and it seems to work fine.

affects: tagger → qtlocation-opensource-src (Ubuntu)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Reportedly something else would be wanted to be backported from Qt 5.6 too, so please elaborate in this bug report.

Changed in qtlocation-opensource-src (Ubuntu):
assignee: nobody → Timo Jyrinki (timo-jyrinki)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Well, while we're at it, the OSM backend in Qt 5.5 is much more complete in general than in 5.4. It supports the Places api and detailed routing. I have an app in the works for that. If there are no blocking issues and we're still "far" away from upgrading Qt in our phone baseline, it'd be great to backport the complete OSM backend. That should be doable quite easily I think.

In Qt 5.6 the map implementation gained some performance boost and also had some API changes. So if we're going to backport more than just the OSM backend, we'd probably require to keep the version bump to 5.6 too in order to not mix up a 5.6 api into a 5.4 version.

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

We have a need to go to 5.5 fairly soon so perhaps we gain this directly rather than backport.

Changed in canonical-devices-system-image:
assignee: nobody → Pat McGowan (pat-mcgowan)
status: New → Incomplete
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Silo 67 now has a backport for xenial (Qt 5.5) and vivid (Qt 5.4) available for testing. The 5.4 needed some rebasing and backporting additional features besides the OSM plugin changes:

qtlocation-opensource-src (5.4.1-1ubuntu2~overlay1~vivid1~test6) vivid; urgency=medium

  * debian/patches/Add-center-function-to-QGeoShape.patch
    debian/patches/Allow-HTML-based-copyright-notices.patch
    debian/patches/backport_qt_56_osm_backend.patch
    (the last one include CycleMap support adding)
    - Backport OpenStreetMap backend changes from Qt 5.6. (LP: #1568439)
  * Update symbols for the copyrightsChanged symbols.

Changed in canonical-devices-system-image:
milestone: none → 11
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

This bug was fixed in the package qtlocation-opensource-src 5.4.1-1ubuntu2~overlay1 in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay

---------------

qtlocation-opensource-src (5.4.1-1ubuntu2~overlay1) vivid; urgency=medium

  * debian/patches/Add-center-function-to-QGeoShape.patch
    debian/patches/Allow-HTML-based-copyright-notices.patch
    debian/patches/backport_qt_56_osm_backend.patch
    (the last one include CycleMap support adding)
    - Backport OpenStreetMap backend changes from Qt 5.6. (LP: #1568439)
  * Update symbols for the copyrightsChanged symbols.

 -- Timo Jyrinki <email address hidden> Mon, 25 Apr 2016 11:32:12 +0000

Changed in qtlocation-opensource-src (Ubuntu RTM):
status: New → Fix Released
Changed in canonical-devices-system-image:
status: Incomplete → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtlocation-opensource-src - 5.5.1-3ubuntu2

---------------
qtlocation-opensource-src (5.5.1-3ubuntu2) xenial; urgency=medium

  * debian/patches/backport_qt_56_osm_backend.patch:
    - Backport OpenStreetMap backend changes from Qt 5.6. (LP: #1568439)

 -- Timo Jyrinki <email address hidden> Mon, 25 Apr 2016 11:33:49 +0000

Changed in qtlocation-opensource-src (Ubuntu):
status: New → Fix Released
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.