[Marble-devel] Review Request: Fix OSRM routing API integration

Dennis Nienhüser earthwings at gentoo.org
Thu Sep 13 18:41:02 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106439/#review18934
-----------------------------------------------------------

Ship it!


Thanks for taking care of your clients! :-)

Do you have commit access or shall I commit on your behalf? It also needs backporting to the KDE/4.9 branch.

- Dennis Nienhüser


On Sept. 13, 2012, 1:11 p.m., Dennis Luxen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106439/
> -----------------------------------------------------------
> 
> (Updated Sept. 13, 2012, 1:11 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> The OSRM API has changed over time and upcoming changes will be much more strict about implementation. I.e. current Marble integration will break soon. The attached patch fixes this problem and is also compatible to the current available API.
> 
> 
> Diffs
> -----
> 
>   src/plugins/runner/open-source-routing-machine/OSRMRunner.cpp af9d594 
> 
> Diff: http://git.reviewboard.kde.org/r/106439/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dennis Luxen
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20120913/b0e50f20/attachment.html>


More information about the Marble-devel mailing list