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

Bump Qt to 6.7.1 and update ci #248

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

Bump Qt to 6.7.1 and update ci #248

wants to merge 12 commits into from

Conversation

jrozner
Copy link
Member

@jrozner jrozner commented Mar 6, 2024

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@jrozner jrozner requested a review from jkennedyvz as a code owner March 6, 2024 20:57
jkennedyvz
jkennedyvz previously approved these changes Mar 6, 2024
@jrozner jrozner changed the title Bump Qt to 6.5.2 Bump Qt to 6.6.2 Mar 6, 2024
@jrozner
Copy link
Member Author

jrozner commented Mar 6, 2024

@dependabot rebase

@jrozner jrozner changed the title Bump Qt to 6.6.2 Bump Qt to 6.7.0 Apr 2, 2024
@jrozner
Copy link
Member Author

jrozner commented Apr 2, 2024

The install qt action doesn't support qt 6.7.0 yet. Hopefully that will change within a couple days and we can move forward with moving to the newest version. At this point it's not clear how to address the sqlmimer issue that we ran into with 6.6.2 but maybe it was fixed upstream.

@jrozner
Copy link
Member Author

jrozner commented Apr 2, 2024

@jrozner
Copy link
Member Author

jrozner commented May 20, 2024

@dependabot rebase

@jrozner
Copy link
Member Author

jrozner commented May 20, 2024

linuxdeploy/linuxdeploy-plugin-qt#153 the issue with sqlmimer isn't a qt issue but an issue with linux-deploy-qt. There are some workarounds listed here. We can probably make this work but it seems like a non-ideal solution because we need to manually specify each library we don't want to include. There are likely a lot of libraries being bundled that we don't need because the deploy tool isn't smart enough.

@jrozner jrozner changed the title Bump Qt to 6.7.0 Bump Qt to 6.7.1 May 21, 2024
@jrozner jrozner changed the title Bump Qt to 6.7.1 Bump Qt to 6.7.1 and update ci May 21, 2024
@jrozner
Copy link
Member Author

jrozner commented May 21, 2024

We need to do the work for #251 as well before we can land this. It might be worth just rolling the work into this or fixing that and rebasing this.

@jkennedyvz
Copy link
Contributor

jkennedyvz commented Jun 4, 2024

This should be ready when linuxdeploy/linuxdeploy-plugin-qt#173 is merged.

For now pointing to my artifact works #257

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