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

Copy v8_context_snapshot.bin to resources when deploying Qt WebEngine. #186

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

camilasan
Copy link
Contributor

This file was added in Qt6.

See https://doc.qt.io/qt-6/qtwebengine-deploying.html

@TheAssassin
Copy link
Member

Given the opportunistic approach (i.e., we copy if the file if it exists and skip it otherwise), this shouldn't break compatibility with Qt 5. Therefore, I'm willing to merge it. Let's see what the CI has to say.

@TheAssassin TheAssassin enabled auto-merge (rebase) September 19, 2024 18:24
@TheAssassin TheAssassin merged commit 70b4665 into linuxdeploy:master Sep 19, 2024
5 of 6 checks passed
@TheAssassin
Copy link
Member

Thanks!

@mgallien
Copy link

@TheAssassin I was planning to use the fix made by @camilasan
is there something I can help with to get it published to continuous ?

@TheAssassin
Copy link
Member

I'm preparing a PR at the moment.

camilasan added a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
mgallien pushed a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
camilasan added a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
camilasan added a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
camilasan added a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
mgallien pushed a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
backportbot bot pushed a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
mgallien pushed a commit to nextcloud/desktop that referenced this pull request Sep 25, 2024
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.

3 participants