iut-ibk / DynaMind-ToolBox

DynaMind-ToolBox
www.dance4water.org
GNU General Public License v2.0
9 stars 6 forks source link

Change to Qt 5 #252

Open abroxos opened 9 years ago

abroxos commented 9 years ago

Replace Qt 4 with Qt 5: Todo:

BASIC:

MODULES:

TESTS:

christianurich commented 9 years ago

In OS X the GUI stuff is still not fixed in Qt 4.8 and I was wondering how the transition to Qt5 is going?

abroxos commented 9 years ago

Hi Christian, currently dynamind-toolbox has a cmake option "-DUSEQT5=ON/OFF". We have tested it in linux. It would be nice if you test this in OSX. The core and gui should work. In the top CMakeLists.txt you will find lines like "if(NOT USEQT5 AND WITHPLUGIN...". representing modules which need code adaptation. Let me know if core and gui works on OSX. Anyway we will give this issue a higher priority now.

ghost commented 9 years ago

I've tested QT5 with BASICMODULES and DYNALP Modules and see no problem so far.

From Basicmodules I use:

from DynAlp I use everything :)

EDIT: I also enabled -DQGL_FROM_SOURCE=ON

full cmake command:

cmake ../ -DUSEQT5=ON -DQGL_FROM_SOURCE=ON -DWITH_PLUGIN_DYNALP=ON -DWITH_PLUGIN_BASICMODULES=ON

For that I changed the CMakeList.txt so I can use both QT5 and the plugins

attozeta commented 9 years ago

Actually the changes are pretty minor, however I can not push to DynaMind-GDALModules and CityDrain3 as I do not have write access. However I have a working patch. Furthermore QGL_FROM_SOURCE is now the default option.

christianurich commented 9 years ago

If you can send me a pull request I'll merge it

christianurich commented 9 years ago

Hi, the OS X build has issues with the qglviewer that are annoying to resolve. Since I don't need the viewer anyway I just added another option that disables the viewer. -WITH_VIEWER

christianurich commented 9 years ago

Besides the viewer everything works now for me. Thanks!!

For Testing:


export CMAKE_PREFIX_PATH=/usr/local/opt/qt5/
cmake ../../ -DUSEQT5=ON -DQGL_FROM_SOURCE=OFF -DWITH_VIEWER=OFF -DWITH_PLUGIN_DYNALP=OFF -DWITH_PLUGIN_BASICMODULES=ON -DWITH_UNIT_TESTS=True -DWITH_PLUGIN_GDALMODULE=True -DWITH_PLUGIN_PERFORMANCE_ASSESSMENT=True -DWITH_DOC=True -DWITH_PLUGIN_GDALDRAINAGE=ON -DWITH_UNIT_TESTS=ON -DWITH_PLUGIN_SEWER=OFF
abroxos commented 9 years ago

Thank you for testing. What was the problem. Do you have any build output ?

christianurich commented 9 years ago

Mainly the cmake configuration itself, the qmake in the cmake didn't work (qmake was not found properly). This can be fixed with get_target_property (QT_QMAKE_EXECUTABLE Qt5::qmake IMPORTED_LOCATION). The real problem was that the qglviewer couldn't link against the standard library which I couldn't fix.

abroxos commented 9 years ago

Finally fixed qglviewer build on OSX:

Please edit in /usr/local/Cellar/qt5/5.4.2/mkspecs/macx-clang/qmake.conf from

QMAKE_MACOSX_DEPLOYMENT_TARGET = 10.6

to

QMAKE_MACOSX_DEPLOYMENT_TARGET = 10.9

and make a clean build

This should fix the build issues.