Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add explicit version information to required packages for CMake #1884

Open
3 tasks
trisyoungs opened this issue Apr 24, 2024 · 2 comments
Open
3 tasks

Add explicit version information to required packages for CMake #1884

trisyoungs opened this issue Apr 24, 2024 · 2 comments
Assignees
Labels
Scope: Codebase Related to source code management, build etc.

Comments

@trisyoungs
Copy link
Member

trisyoungs commented Apr 24, 2024

Describe the bug
Having just tried to compile Dissolve on my laptop (OpenSuSE 15.4, Qt6 6.2) I discovered that the new usage of qt_generate_deploy_qml_app_script depends on Qt6 >=6.7.

Our current find_package calls do not have any version requirements set throughout, so we should add these.

  • Qt >= 6.7
  • ANTLR4 (Java) >= 4
  • ANTLR runtime >= 4.11
@trisyoungs trisyoungs added the Type: Bug General bug / issue label Apr 24, 2024
@trisyoungs
Copy link
Member Author

Building against Qt 6.7 on my machines (OpenSuSE) appears to be broken...

During configuration:

CMake Warning at /usr/lib64/cmake/Qt6Qml/Qt6QmlMacros.cmake:3393 (message):
  The qml plugin 'qquicklayoutsplugin' is a dependency of 'dissolve-gui', but
  the link target it defines (Qt6::qquicklayoutsplugin) does not exist in the
  current scope.  The plugin will not be linked.
Call Stack (most recent call first):
  /usr/lib64/cmake/Qt6Core/Qt6CoreMacros.cmake:678 (qt6_import_qml_plugins)
  /usr/lib64/cmake/Qt6Core/Qt6CoreMacros.cmake:678 (cmake_language)
  /usr/lib64/cmake/Qt6Core/Qt6CoreMacros.cmake:774 (_qt_internal_finalize_executable)
  /usr/lib64/cmake/Qt6Core/Qt6CoreMacros.cmake:590:EVAL:1 (qt6_finalize_target)
  CMakeLists.txt:DEFERRED

... and many other plugin libraries related to qml.

And during compilation:

/home/tris/src/dissolve/build-release/src/gui/models/moc_sortFilterProxy.cpp:131:69: error: static assertion failed: NOTIFY signal read does not exist in class (or is private in its parent)
  131 | static_assert(qxp::is_detected_v<has_nullary_read, SortFilterProxy> || qxp::is_detected_v<has_unary_read, SortFilterProxy>,

@trisyoungs trisyoungs added Scope: Codebase Related to source code management, build etc. and removed Type: Bug General bug / issue labels Jul 10, 2024
@github-project-automation github-project-automation bot moved this to Codebase in Overview Aug 30, 2024
@trisyoungs
Copy link
Member Author

But it seems building against 6.6.3 works just fine without errors. So, errr, maybe ignore all of this.

@github-project-automation github-project-automation bot moved this from Codebase to Complete in Overview Oct 21, 2024
@trisyoungs trisyoungs reopened this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Codebase Related to source code management, build etc.
Projects
Status: Complete
Development

No branches or pull requests

2 participants