Skip to content

cmake: fix build with Qt 6.10 #100

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

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

cmake: fix build with Qt 6.10 #100

wants to merge 1 commit into from

Conversation

Chiitoo
Copy link

@Chiitoo Chiitoo commented Jun 17, 2025

The 'Qt6FooPrivate' targets have been split into separate CMake files
in Qt 6.9, and require a 'find_package(Qt6FooPrivate)' call starting
with Qt 6.10.

See also: https://bugreports.qt.io/browse/QTBUG-87776

The 'Qt6FooPrivate' targets have been split into separate CMake files
in Qt 6.9, and require a 'find_package(Qt6FooPrivate)' call starting
with Qt 6.10.

See also: https://bugreports.qt.io/browse/QTBUG-87776
@tsujan
Copy link
Member

tsujan commented Jun 17, 2025

Thanks!

Any other library/app in addition to libfm-qt and lxqt-qtplugin?

@Chiitoo
Copy link
Author

Chiitoo commented Jun 17, 2025

These are the only ones I initially bumped into on my regular install, which was missing at least lxqt-{admin,openssh-askpass,powermanagement,sudo} and xdg-desktop-portal-lxqt, from which only the last one seems to be needing the same treatment.

For applications, qps, qterminal, and screengrab at least seem to be OK.

@Chiitoo
Copy link
Author

Chiitoo commented Jun 19, 2025

Hrm, thought I tested them out but... maybe not.

Some more:

  • lximage-qt
  • lxqt-panel
  • pcmanfm-qt
  • screengrab

(Maybe a tracker issue would have been nice for this, but o whell.)

@tsujan
Copy link
Member

tsujan commented Jun 21, 2025

Please make PRs for all components that need them.

P.S.
I may not be available for some time but will come back.

@Chiitoo
Copy link
Author

Chiitoo commented Jun 21, 2025

May take some time from now for me as well, but will do!

@tsujan
Copy link
Member

tsujan commented Jun 21, 2025

May take some time from now for me as well

There's no rush at all; take your time. Thanks again for taking care of this!

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