Adding version specifier for brew Qt in the docs #1324

Merged

root merged 1 commit from patch-1 into master

2021-09-22 21:18:04 +00:00

RedAuburn commented

2021-09-20 01:53:17 +00:00

Member

Latest Qt version is 6.1.3 which doesn't work with building, adding @5 ensures that Qt 5 is installed instead of the latest version.

(This is my first ever pull-request! 🥳)

Latest Qt version is 6.1.3 which doesn't work with building, adding @5 ensures that Qt 5 is installed instead of the latest version. (This is my first ever pull-request! 🥳)

RedAuburn commented

2021-09-20 14:09:22 +00:00

Author

Member

Sorry about the messy PR history, it's my first time using git

vng (Migrated from github.com) approved these changes 2021-09-21 21:13:20 +00:00

biodranik commented

2021-09-21 21:14:01 +00:00

(Migrated from github.com)

What errors did you get with Qt6? Can you please share them? It builds on my machine.

RedAuburn commented

2021-09-21 21:53:12 +00:00

Author

Member

What errors did you get with Qt6? Can you please share them? It builds on my machine.

I believe it's caused by the Qt5CoreConfig.cmake files being called Qt6CoreConfig.cmake in Qt6.

Console Log:

endi@Harrys-MacBook-Air organicmaps % tools/unix/build_omim.sh -r              
Setting PLATFORM_MAC to true
Build type: Release
CMake Warning at CMakeLists.txt:213 (find_package):
  By not providing "FindQt5Core.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Qt5Core", but
  CMake did not find one.

  Could not find a package configuration file provided by "Qt5Core" with any
  of the following names:

    Qt5CoreConfig.cmake
    qt5core-config.cmake

  Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH or set
  "Qt5Core_DIR" to a directory containing one of the above files.  If
  "Qt5Core" provides a separate development package or SDK, be sure it has
  been installed.


CMake Error at CMakeLists.txt:215 (message):
  Qt5 cmake files were not found, please set QT_PATH environment variable

> What errors did you get with Qt6? Can you please share them? It builds on my machine. I believe it's caused by the `Qt5CoreConfig.cmake` files being called `Qt6CoreConfig.cmake` in Qt6. **Console Log:** ``` endi@Harrys-MacBook-Air organicmaps % tools/unix/build_omim.sh -r Setting PLATFORM_MAC to true Build type: Release CMake Warning at CMakeLists.txt:213 (find_package): By not providing "FindQt5Core.cmake" in CMAKE_MODULE_PATH this project has asked CMake to find a package configuration file provided by "Qt5Core", but CMake did not find one. Could not find a package configuration file provided by "Qt5Core" with any of the following names: Qt5CoreConfig.cmake qt5core-config.cmake Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH or set "Qt5Core_DIR" to a directory containing one of the above files. If "Qt5Core" provides a separate development package or SDK, be sure it has been installed. CMake Error at CMakeLists.txt:215 (message): Qt5 cmake files were not found, please set QT_PATH environment variable ```

RedAuburn commented

2021-09-21 22:09:53 +00:00

Author

Member

waaiit, Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH. i may be stupid but let me test it quickly

waaiit, `Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH`. i may be stupid but let me test it quickly

RedAuburn commented

2021-09-21 23:02:30 +00:00

Author

Member

Update:
I've tried setting QT_PATH to /opt/homebrew/Cellar/Qt/6.1.3, and set CMAKE_PREFIX_PATH to Qt5Core & Qt6Core, nothing worked.

I'm pretty sure that it isn't working because all the Qt files use the Qt6 prefix instead of Qt5.

**Update:** I've tried setting `QT_PATH` to `/opt/homebrew/Cellar/Qt/6.1.3`, and set `CMAKE_PREFIX_PATH` to `Qt5Core` & `Qt6Core`, nothing worked. I'm pretty sure that it isn't working because all the Qt files use the Qt6 prefix instead of Qt5.

RedAuburn commented

2021-09-22 00:05:53 +00:00

Author

Member

It works out of the box with qt@5, I've realised you don't need to mess with the PATH variable either, so i removed the line i added that suggested it.