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

feat: installer and uninstaller #32

Merged
merged 3 commits into from
Mar 1, 2025
Merged

feat: installer and uninstaller #32

merged 3 commits into from
Mar 1, 2025

Conversation

deansheather
Copy link
Member

@deansheather deansheather commented Feb 28, 2025

Adds new package Installer which uses WixSharp to build WixToolset installers for WIndows.

We build two installers:

  • Coder Desktop (Core), which installs the app files, service files, VPN files (wintun.dll), creates a system service and an app shortcut in the start menu
  • Coder Desktop, which will netinstall the .NET runtime if it's not installed and then chain install `Coder Desktop (Core)

We will only be shipping the Coder Desktop installer, which contains the Core installer. The chained Core installation doesn't show up in Settings > Apps.

image
image
image
image

Follow-up PR will integrate to CI and add authenticode signing to the installer binaries.

Closes #11
Closes #30

Copy link
Member

@ethanndickson ethanndickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works on my machine! 🎉 🎉 🎉

Copy link
Member

@ethanndickson ethanndickson Feb 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor, but we use a flat black icon on macOS from the presskit. You very rarely see this app/executable icon during normal usage on either OS though.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Atif commented on the icon in Slack and I said it's something we can fix after we get alpha out, pretty much just involves someone designer-y redoing all of the assets in that dir + the coder.ico and banner BMP in the coder/coder repo for the old installer.

I think for these icons we want something with a background, like maybe a black square with a white icon or something. Otherwise it'll look weird on the wrong system light/dark theme.

@deansheather deansheather enabled auto-merge (squash) March 1, 2025 00:49
@deansheather deansheather disabled auto-merge March 1, 2025 00:50
@deansheather deansheather enabled auto-merge (squash) March 1, 2025 00:54
@deansheather deansheather merged commit 1c5e4d9 into main Mar 1, 2025
3 checks passed
@deansheather deansheather deleted the dean/installer branch March 1, 2025 00:58
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.

Installer/uninstaller Get the installable package to include the system service
3 participants