-
Notifications
You must be signed in to change notification settings - Fork 51
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
Fix build issues and create APK for the Android with Qt6 #65
base: master
Are you sure you want to change the base?
Conversation
Hi, I have not used cmake from either the command line or with QtCreator to build an android target. I'm able to build the exe in QtCreator and from the cmd line, but not the apk. I cloned your fork and opened CMakeList.txt in QtCreator. build output is |
You're using Qt 6.5.1 but I added Android support only for 6.6.x and later. Should I fail the Cmake (configure) if it's lower than that? |
template T QJniObject::object() was not added until Qt6.7, so your change would require 6.7 or later. This commit would break Qt 6.0 to 6.6.3. Does the QtZeroConf library not compile in Qt 6.7? I am not able to test as Qt 6.7 does not build on Debian 12; 6.6.3 does build, and QtZeroConf compiles fine with 6.6.3 as is. |
No it didn't built. Let me check. |
Fix the build issues. |
It doesn't seem right that Qt 6.7 would not compile code that Qt 6.6 does. I suspect something is still wrong. If you want to go ahead with this change, can you make the commits atomic? It is unclear to me why the changes in CMakeList are necessary to fix compiling and why openssl is needed. QtZeroConf and the example both compile with cmake and qmake as is. |
Build with Qt 6.7.0 and tested on;