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

qt-build-utils: Add an option to use CMake instead of QMake #1156

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

redstrate
Copy link
Contributor

This is needed for scenarios where the CMake packaging for Qt is better than what QMake can handle, e.g. split Qt installations where the modules live in different directories.

A new optional feature is added to cxx-qt-build to prefer CMake for most operations related to linking/including Qt modules. QMake is still the default.

Fixes #1153.

@redstrate redstrate self-assigned this Jan 14, 2025
@redstrate redstrate added 🔨 build system Issues related to integrating CXX-Qt into CMake/Cargo ⬆️ upstream blocked on an upstream issue labels Jan 14, 2025
@redstrate redstrate force-pushed the work/josh/cmake-package branch from 0fb6d0b to bd7a8bd Compare January 15, 2025 14:19
@redstrate redstrate marked this pull request as ready for review January 15, 2025 14:19
@redstrate redstrate removed the ⬆️ upstream blocked on an upstream issue label Jan 15, 2025
@redstrate redstrate force-pushed the work/josh/cmake-package branch from bd7a8bd to 9f7238c Compare January 15, 2025 14:21
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (e84662a) to head (1d29d33).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1156   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           71        71           
  Lines        11967     11967           
=========================================
  Hits         11967     11967           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@redstrate redstrate force-pushed the work/josh/cmake-package branch from 9f7238c to 66c1972 Compare January 15, 2025 15:47
This is needed for scenarios where the CMake packaging for Qt is better
than what QMake can handle, e.g. split Qt installations where the
modules live in different directories.

A new optional feature is added to cxx-qt-build to prefer CMake for most
operations related to linking/including Qt modules. QMake is still the
default.

Fixes KDAB#1153.
@redstrate redstrate force-pushed the work/josh/cmake-package branch from 66c1972 to 1d29d33 Compare January 15, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 build system Issues related to integrating CXX-Qt into CMake/Cargo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CXX-Qt does not work in split-Qt installations
1 participant