Skip to content

fix: windows installer can't be uninstalled#320

Open
amoeba wants to merge 2 commits intomainfrom
fix/windows-installer-cant-uninstall
Open

fix: windows installer can't be uninstalled#320
amoeba wants to merge 2 commits intomainfrom
fix/windows-installer-cant-uninstall

Conversation

@amoeba
Copy link
Member

@amoeba amoeba commented Feb 17, 2026

Updates our WiX template to fix an issue where dbc can't be uninstalled with Add/Remove programs. We were setting two values in our .wxs template incorrectly. The "Product Code" should vary between releases and the "Package Code" should as well.

See:

Closes #319

@amoeba amoeba force-pushed the fix/windows-installer-cant-uninstall branch from a7c0edc to 1db6f5a Compare February 17, 2026 21:50
@amoeba amoeba marked this pull request as ready for review February 17, 2026 21:54
@amoeba
Copy link
Member Author

amoeba commented Feb 17, 2026

I just downloaded the MSI from the CI artifacts and it installs and uninstalls (through add/remove programs) cleanly.

</Fragment>
<Product Id="5B742771-61E1-4DC1-9276-6E1287282B3F" Name="dbc"
Manufacturer="Columnar Technologies Inc." Version="{{ .Version }}"
<Product Id="*" Name="dbc" Language="1033"
Copy link
Member

Choose a reason for hiding this comment

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

I think this needs to be one line up

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh yep. Fixed in 8ed1402.

Copy link
Member Author

Choose a reason for hiding this comment

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

This isn't quite right. Will fix in a sec.

Copy link
Member Author

Choose a reason for hiding this comment

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

Never mind, I can't read. Should be good now.

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.

winget uninstall not fully uninstalling dbc and leaves system in inconsistent state

2 participants

Comments