mattermost / desktop

Mattermost Desktop application for Windows, Mac and Linux
Apache License 2.0
2.03k stars 830 forks source link

[MM-55341][MM-56894] Use `electron-builder` to create the MSI installer, drop support for 32-bit Windows and the NSIS installer #3028

Closed devinbinnie closed 6 months ago

devinbinnie commented 6 months ago

Summary

This PR contains changes to revamp our Windows installer building process. It does away with the custom script we had to build the MSI installer, and elects to use electron-builder with a few modifications to the flow.

Some features provided by the new MSI installer that work out of the box:

With this PR, this also marks the following changes in support for the Desktop App:

I've added a number of people for review, I will make comments on the code to help people better understand some of the changes that are happening.

Ticket Link

https://mattermost.atlassian.net/browse/MM-55341 https://mattermost.atlassian.net/browse/MM-56894

Drop support for 32-bit Windows, add support for ARM64 (in beta)
Drop support for the EXE/NSIS installer, ship only the MSI
devinbinnie commented 6 months ago

Link to built installer: https://github.com/mattermost/desktop/actions/runs/9020601613/artifacts/1488474351

github-actions[bot] commented 6 months ago

Here are the test results below:

Test Summary for Linux on commit 0a4515f9c4bc0b84d5e2d2d99369c9589c1e35ab

All stable tests passed on Linux.

Test Summary for macOS on commit 0a4515f9c4bc0b84d5e2d2d99369c9589c1e35ab

All stable tests passed on macOS.

Test Summary for Windows on commit 0a4515f9c4bc0b84d5e2d2d99369c9589c1e35ab

All stable tests passed on Windows.

devinbinnie commented 6 months ago

Screenshot 2024-05-13 025453

@yasserfaraazkhan When did you encounter this error? Was it reproducible?

Also, did you happen to try any of the admin commands?