Hi

thanks for the patch. I don't know if you are aware of the branch:

https://sourceforge.net/p/qlandkartegt/code/HEAD/tree/QLandkarteGT/branches/Qt5/

That's an attempt to port QLGT to QT5 a few month ago. We tried QT5.1 at this 
time and there where still 
too many peculiarities to go on. However this branch has fixes for Win32 and OS 
X. I would like to ask you 
to look at this branch and to add the fixes to your patch. Is that possible?

Additionally I noticed:

* I had to add the path to the Qt cmake files. Something like:

set(CMAKE_PREFIX_PATH /opt/Qt/5.2.0/gcc_64/lib/cmake/)

to make cmake find the findModuleXXX scripts for Qt5

* src/CDeviceGPSD.cpp  failed because of #include <QtWidgets> missing.

* add_subdirectory(3rdparty/SerialPort) and 
add_subdirectory(3rdparty/QMKToolBox) can be skipped. The first is now part of 
Qt5 (needs some fixes in 
the code) the later is obsolete.

* add_subdirectory(3rdparty/tzone) probably can be replaced by QTimeZone. If 
that works it is the biggest 
benefit from Qt5 as tzone has bugs for some timezones, e.g. in Russia.


It would be fine if you could fix these topics. I like your patch as you 
started to add ifdefs to compile for Qt4 
and 5. That makes it interesting to add it to the main branch and thus part of 
further releases.

Oliver


> Hi all, this patch will help you to build qlandkartegt with Qt5.
> All 3rd party apps/adds have ported too, except conan.
> 
> Configure like this (cmake 2.8.9 required):
> cmake -DQK_QT5_PORT=ON src_dir
> Also may be need to set QT_QMAKE_EXECUTABLE (points to Qt5 variant of
> qmake) variable, it is set to "/usr/lib64/qt5/bin/qmake" by default in
> CMakeLists.txt:5
> 
> Patch is based on current 3884 subversion revision.
> 
> Lebedev Dmitry

------------------------------------------------------------------------------
Flow-based real-time traffic analytics software. Cisco certified tool.
Monitor traffic, SLAs, QoS, Medianet, WAAS etc. with NetFlow Analyzer
Customize your own dashboards, set traffic alerts and generate reports.
Network behavioral analysis & security monitoring. All-in-one tool.
http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk
_______________________________________________
Qlandkartegt-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qlandkartegt-users

Reply via email to