[Marble-devel] Review Request 124193: QT_NO_WEBKIT option

[Torsten Rahn]

> On Juni 27, 2015, 7:36 vorm., Torsten Rahn wrote:
> > src/lib/marble/CMakeLists.txt, line 64
> > <https://git.reviewboard.kde.org/r/124193/diff/1/?file=381521#file381521line64>
> >
> >     That's not exactly great to maintain if we have to keep two cpp file lists. - I'd even claim it will break one week after GSoC is finished.

Oh I only just realized that Dennis had pointed out the same issue already.


- Torsten


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124193/#review81803
-----------------------------------------------------------


On Juni 27, 2015, 7:22 vorm., Gábor Péterffy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124193/
> -----------------------------------------------------------
> 
> (Updated Juni 27, 2015, 7:22 vorm.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> This patch includes some little bugfixes and some bulk class to compile without QtWebKit.
> 
> Previous review request:
> https://git.reviewboard.kde.org/r/124064/
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt 78b83a39ac26b17b82b072d6309bb30d88bf6534 
>   src/apps/CMakeLists.txt 2834e13025bf70a202163992e849a92e2801e1cf 
>   src/apps/marble_android/CMakeLists.txt PRE-CREATION 
>   src/apps/marble_android/main.cpp PRE-CREATION 
>   src/lib/marble/CMakeLists.txt 8b36d0843f585c09c62bf044d0cfb0651a13d4b0 
>   src/lib/marble/HttpJob.cpp 249b6800a4c0bfbc944020eb25019400bd4eff34 
>   src/lib/marble/LegendWidget.cpp b74373c92830856f29d3759f682dbf89c3dab6ae 
>   src/lib/marble/LegendWidgetQtNoWebkit.ui PRE-CREATION 
>   src/lib/marble/MarbleLegendBrowser.h 7e80c389f9e8f0fe3876dc4fe23d917e70a078d9 
>   src/lib/marble/MarbleLegendBrowser.cpp f628b835534063aa459d67ddff10f33466ac59cd 
>   src/lib/marble/MarbleModel.cpp 04605c6f7261d03e720a3c071a63f459f982d909 
>   src/lib/marble/MarbleWebViewQtNoWebkit.h PRE-CREATION 
>   src/lib/marble/MarbleWebViewQtNoWebkit.cpp PRE-CREATION 
>   src/lib/marble/PopupItem.h 4c65004d1386305c3394f5b343bc67cff9127f39 
>   src/lib/marble/PopupItem.cpp 8dd885c6efa4ed1c0981f835339b031720a24d01 
>   src/lib/marble/TinyWebBrowserQtNoWebkit.h PRE-CREATION 
>   src/lib/marble/TinyWebBrowserQtNoWebkit.cpp PRE-CREATION 
>   src/lib/marble/WebPopupWidgetQtNoWebkit.ui PRE-CREATION 
>   src/lib/marble/cloudsync/RouteItemDelegate.cpp 9670f1e3ccb643bab5400e84e01129b9cc6826a4 
>   src/lib/marble/routing/RoutingInputWidget.cpp 40c4aad9a93d3255a66fbfb98487401ee597321e 
> 
> Diff: https://git.reviewboard.kde.org/r/124193/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gábor Péterffy
> 
>

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