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

VPN-6796: Update Qt/Wasm to 6.6.3 #10233

Merged
merged 14 commits into from
Feb 7, 2025
Merged

VPN-6796: Update Qt/Wasm to 6.6.3 #10233

merged 14 commits into from
Feb 7, 2025

Conversation

oskirby
Copy link
Collaborator

@oskirby oskirby commented Jan 29, 2025

Description

This attempts to update WASM's version of Qt to 6.6.3. This keeps it in sync with the rest of our target platforms, and fixes an issue in Thai font rendering that would lead to a crash.

This update wasn't quite as straightforward as we would like, since the WASM instantiation process has changed in Qt 6.5 to use a new API, which required a hefty change to the HTML pages.

Reference

JIRA Issue: VPN-6796
JIRA Issue: VPN-6795

Checklist

  • My code follows the style guidelines for this project
  • I have not added any packages that contain high risk or unknown licenses (GPL, LGPL, MPL, etc. consult with DevOps if in question)
  • I have performed a self review of my own code
  • I have commented my code PARTICULARLY in hard to understand areas
  • I have added thorough tests where needed

@oskirby oskirby force-pushed the vpn-6796-qt-wasm-update branch from 9308640 to de5a026 Compare January 29, 2025 20:24
@oskirby oskirby force-pushed the vpn-6796-qt-wasm-update branch 2 times, most recently from c739a7e to ce5c298 Compare February 4, 2025 00:02
@oskirby oskirby force-pushed the vpn-6796-qt-wasm-update branch from ce5c298 to 214eef3 Compare February 4, 2025 00:29
@oskirby oskirby force-pushed the vpn-6796-qt-wasm-update branch from 2324046 to c478db3 Compare February 6, 2025 21:01
@oskirby oskirby changed the title WIP: Update Qt/Wasm to 6.6.3 VPN-6796: Update Qt/Wasm to 6.6.3 Feb 6, 2025
@oskirby oskirby marked this pull request as ready for review February 6, 2025 22:39
@oskirby oskirby requested a review from a team as a code owner February 6, 2025 22:39
@oskirby oskirby requested review from gabrielBusta, mcleinman and strseb and removed request for a team February 6, 2025 22:39
@oskirby oskirby merged commit be529f3 into main Feb 7, 2025
120 checks passed
@oskirby oskirby deleted the vpn-6796-qt-wasm-update branch February 7, 2025 20:30
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