-
Notifications
You must be signed in to change notification settings - Fork 428
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
Automatically bundle Qt plugin for AppImage updates #380
Comments
Its possible, You don't really need any code from |
Great 👍 Currently we still need to specify where the update menu is. Can you "guesstimate" it so that the configuration does not need to be changed for every application? |
So I will write an option to pre-build Qt Plugins and Qt Plugin Injector for doing this stuff. OR We could just write the program to guesstimate everything. I think the former is more work you, isn't it? Its up to you to decide because you are the maintainer. |
Guesstimate everything 👍 |
So you are going to use sub process to use the tool? like you do with appimagetool? I think that solves a lot of problem. Okay then. |
Or we could build linuxdeployqt/shared/shared.cpp Line 1057 in 11e35ce
|
It is a small C program, it only depends on standard C libraries so integrating it is not that hard but it adds more complication. Like you need to copy the OpenSSL libs too. For now we copy the build system's OpenSSL libs assuming that it works because the build system should have successfully built the program itself but this is not fail safe. It would be nice if linuxdeployqt can package all the dependencies of the plugins you package, All we need is Qt network module which in turn depends on OpenSSL
I think we can pass a command line argument for that. |
If linuxdeployqt automatically packages OpenSSL then we can save a lot of time. If you want the updater, you depend on the Qt Network module and OpenSSL so linuxdeployqt should automatically package that. |
I'm currently making everything into a single repo(in updatedeployqt), you can see that in the experimental branch. So I might need some time. |
Yes, everything in one repository sounds good, just ping me when you think it is ready. |
Okay everything is now in a single repo(but untested). I also added the option to guess the configuration. You have to execute it like this,
For now there is no AppImage available for the latest build, But the build and deploy is in progress at travis-ci Each build and deploy takes about 2:30 hours since we have to build 5 versions of Qt from source. I will not be available for the next 14 hours so if this build succeeds -> https://travis-ci.org/TheFutureShell/updatedeployqt/builds/567591279 then you can get the AppImage from the releases under the continuous tag as usual. If everything goes well I will delete all unwanted repos from the org and start refactoring the code. |
Wow, that is very exciting @antony-jr thank you. Keep me updated once there is an AppImage available. 👍 |
oh cool this seems very exciting, would this help with this issue? nextcloud/desktop#168 |
@Fuseteam Thanks for your interest but this is very alpha(i.e updatedeployqt -> https://github.com/antony-jr/updatedeployqt), But I've had it working for any Qt application but for now it's not suitable for production. But I do have the Qt Plugin and library in a very good condition but the thing it's not official, the official one is at https://github.com/AppImage/AppImageUpdate but if you want to look at the unofficial library which gives full support for the Qt framework then take a look at https://antony-jr.github.io/QAppImageUpdate (this can be built as plugin or as a library however you wish to use it) (There is also python support for it via Qt plugins) |
@antony-jr it all still a bit foreign to me, so thanks for explaining |
I would like to integrate this into
linuxdeployqt
so thatlinuxdeployqt
users must do nothing special to get the update functionality for their Qt apps automatically.antony-jr/updatedeployqt#9
cc @antony-jr
The text was updated successfully, but these errors were encountered: