[Marble-devel] Review Request 122731: Support qt4/qt5 parallel-instalable libmarblewidget

Torsten Rahn tackat at kde.org
Fri Apr 3 09:04:08 UTC 2015



> On Feb. 28, 2015, 4:06 nachm., Dennis Nienhüser wrote:
> > Makes sense generally. I do not have a system to test with at the moment, but the problems with plugin loading sound like a must-fix before shipping. Got some details on it? Do some plugins load, or none? Is it related to the qt version?
> 
> Rex Dieter wrote:
>     Looks like I'm getting the same crashes without this patch, so probably unrelated.  I'll file a bug (separately).
> 
> Rex Dieter wrote:
>     bug filed, https://bugs.kde.org/show_bug.cgi?id=344692
> 
> Dennis Nienhüser wrote:
>     With this patch we get
>     - Qt4: libmarblewidget
>     - Qt5: libmarblewidget-qt5
>      
>     Other options would be
>     - Qt4: libmarblewidget-qt4
>     - Qt5: libmarblewidget
>     
>     and also
>     - Qt4: libmarblewidget-qt4
>     - Qt5: libmarblewidget-qt5
>     
>     and last but not least using different folders for each installation.
>     
>     I'm mereley thinking aloud to avoid that we forget a viable option. The current patch might confuse some people / build scripts when changing from qt4 to qt5 and the (possibly hardcoded) libmarblewidget does not exist anymore. Using libmarblewidget-qt4/libmarblewidget would stress that qt5 is the new default which I'd like to push; using libmarblewidget-qt4/libmarblewidget-qt5 seems more consistent.
>     
>     Opinions, anyone?
> 
> Rex Dieter wrote:
>     I'd commented onlist, but for transparency and posterity, I'll repost here.
>     
>     I'd suggest not changing api for older apps (qt4), and only require a change 
>     for qt5 (and newer).  It will be easier to get newer application developers 
>     to adapt to a change than to fix legacy applications.

Rex Dieter: Yes, let's go with that approach (I just talked to Dennis and he agrees with this as well).


- Torsten


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


On Feb. 26, 2015, 6:50 nachm., Rex Dieter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122731/
> -----------------------------------------------------------
> 
> (Updated Feb. 26, 2015, 6:50 nachm.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Support qt4/qt5 parallel-installabie libmarblewidget.  Current mplementation preserves compatibility with qt4 builds, and qt5 builds changes base library soname to be libmarblewidget-qt5 instead.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt ace9a51 
>   FindMarble.cmake 99f017a 
>   FindMarble.cmake.in PRE-CREATION 
>   MarbleMacros.cmake b86481f 
>   src/apps/marble-kde/CMakeLists.txt 384f867 
>   src/apps/marble-mobile/CMakeLists.txt b86cac1 
>   src/apps/marble-qt/CMakeLists.txt 75b465e 
>   src/apps/marble-touch/CMakeLists.txt 1d37cd4 
>   src/apps/marble-ui/CMakeLists.txt 74f1849 
>   src/bindings/python/CMakeLists.txt 57683cb 
>   src/lib/marble/CMakeLists.txt 6c65015 
>   src/plasmarunner/CMakeLists.txt b52cc35 
>   src/plasmoid/CMakeLists.txt 03e251c 
>   src/plugins/designer/latloneditwidget/CMakeLists.txt eb5d543 
>   src/plugins/runner/gpx/CMakeLists.txt aed6f83 
>   src/thumbnailer/CMakeLists.txt 175b5fc 
> 
> Diff: https://git.reviewboard.kde.org/r/122731/diff/
> 
> 
> Testing
> -------
> 
> Marble and subsurface (a qt5 app that uses libmarblewidget) builds and runs... mostly (there seems to be some problem with loading marble plugins though)
> 
> 
> Thanks,
> 
> Rex Dieter
> 
>

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


More information about the Marble-devel mailing list