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

Qt GUI: Update MSIX manifest #965

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Qt GUI: Update MSIX manifest #965

merged 1 commit into from
Nov 17, 2024

Conversation

cjee21
Copy link
Collaborator

@cjee21 cjee21 commented Nov 16, 2024

Lower Windows version requirement and add list of file extensions for FileExplorerContextMenus.

Lower Windows version requirement and add list of file extensions for FileExplorerContextMenus.
@cjee21
Copy link
Collaborator Author

cjee21 commented Nov 16, 2024

FYI with the current Qt MSIX build script, MSIX package is 149MiB and install size is 339MiB.

@JeromeMartinez JeromeMartinez merged commit 36ff2a8 into MediaArea:master Nov 17, 2024
3 checks passed
@JeromeMartinez
Copy link
Member

Nice to have that in the source code but:

MSIX package is 149MiB

It is definitely a blocker for general release, lot of people love to have MediaInfo small.
Maybe later ever removing the HTML engine or using the integrated one... But not a priority for the moment.

@cjee21
Copy link
Collaborator Author

cjee21 commented Nov 17, 2024

Maybe later ever removing the HTML engine or using the integrated one... But not a priority for the moment.

Also can make VCLibs a dependency rather than statically linking or bundling the DLLs to reduce size further. Most systems should have VCLibs installed already and it should auto-install for those that don't if added as dependency. Will also ensure VCLibs is kept updated without needing to update the app.

If others are interested in Qt version, maybe can consider publishing to Store as an alternate / beta version.

@cjee21 cjee21 deleted the Qt branch November 17, 2024 12:18
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