Skip to content

added qt5 and win support#1

Open
DmiitriyJarosh wants to merge 38 commits intorobocup-ssl-russia:masterfrom
DmiitriyJarosh:win_mingw
Open

added qt5 and win support#1
DmiitriyJarosh wants to merge 38 commits intorobocup-ssl-russia:masterfrom
DmiitriyJarosh:win_mingw

Conversation

@DmiitriyJarosh
Copy link
Member

No description provided.

*.swp
Thumbs.db
.idea/*
cmake-* No newline at end of file

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Какие конкретно файлы перекрывает эта маска?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cmake-build-debug
cmake-build-release -- dirs which CLion creates with his cmake. Do you think it will be better to specify only them?

- cmake ..
- make
- sudo make install No newline at end of file
- sudo make install

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Таких изменений быть не должно.

sudo: required
- os: osx
osx_image: xcode8
osx_image: xcode10

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что не так с предыдущей версией было? Каких библиотек не хватало?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

qt5 wasn't supported, was it?

find_package(Qt5OpenGL REQUIRED)
find_package(Qt5Network REQUIRED)
set(QT_LIBRARIES "")
macro(qt4_wrap_cpp VARNAME)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А зачем это? Не очень понимаю :( и ниже который.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to add support for both versions with less changes

list(APPEND DLL_LIB_LIST ${QT5_DEBUG_DLL})
endif()
if (CXX_BIT STREQUAL 64)
list(APPEND DLL_LIB_LIST ${RES}/libgcc_s_seh-1.dll)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обе возможности протестированы?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

- cd ..
- set PATH=%MSYS_DIR%\mingw%bit%\bin;%QT_MinGW_DIR%\bin
- cd vartypes_dir
- mkdir build-cmake && cd build-cmake

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А почему часть билдов in-source, а часть out-of-source? Можно как-то унифицировать?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a problem with calling configure for ODE, it is a bash script and i try to find way to run it in cmd? Any ideas?

@@ -1,14 +1,47 @@
cmake_minimum_required(VERSION 2.8)

set(USE_QT5 1)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У cmake должен быть аналог pri файлов в Qt, так называемых include файлов. Туда можно и нужно выносить общие вещи между разными таргетами и просто инклюдить. Давай попробуем.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments