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 client library version for Qt6 #373

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Gliese852
Copy link
Contributor

@Gliese852 Gliese852 commented Aug 6, 2024

I would like to port lightdm-kde-greeter to qt6, and this requires the corresponding lightdm library.

I did not delete the version for qt5, the versions do not interfere with each other. The source code has not changed much, just updated the outdated QString::toAscii method.

Added an m4 macro to search for the moc executable file to check the version. It seems that this will be more reliable if there are several of them in the system. For reference, I used macros from the freeciv project. (qt6.m4)

Copy link

github-actions bot commented Aug 6, 2024

Everyone contributing to this PR have now signed the CLA. Thanks!

@Gliese852
Copy link
Contributor Author

Well, now CI is building a version for Qt6. Tests fail just like in main; tests added for Qt6 also fail.

@Gliese852 Gliese852 mentioned this pull request Aug 30, 2024
@Gliese852
Copy link
Contributor Author

@robert-ancell,
Do I need to sign a Canonical CLA, and if so, can I use your name as a contact?

@robert-ancell
Copy link
Collaborator

@Gliese852 yes, you will need to sign the CLA and you can use my name. Thanks!

@Gliese852
Copy link
Contributor Author

Squashed the fixup commits to at the same time provoke the launch of CI, at least the cla-check is now successful.

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