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

Uninstall text not showing on uninstall for MSI package #1858

Closed
heaths opened this issue Apr 3, 2023 · 3 comments · Fixed by #1881
Closed

Uninstall text not showing on uninstall for MSI package #1858

heaths opened this issue Apr 3, 2023 · 3 comments · Fixed by #1881
Assignees
Labels
bug Something isn't working engsys installer

Comments

@heaths
Copy link
Member

heaths commented Apr 3, 2023

Despite both the Installed and REMOVE properties being set correct e.g., REMOVE=ALL, the custom uninstall text is not showing in the ExitDlg when invoked with msiexec /x. If you run msiexec /i and run the MSI in maintenance mode, then choose Uninstall, it does show as expected. The condition should fire appropriately.

Need to investigate more.

Originally posted by @heaths in #1834 (comment)

@ghost ghost added the needs-triage For new issues label Apr 3, 2023
@heaths heaths self-assigned this Apr 3, 2023
@heaths heaths added bug Something isn't working installer and removed needs-triage For new issues labels Apr 3, 2023
@barnson
Copy link

barnson commented Apr 3, 2023

🤷 Can't say I remember ever using /x in full-UI mode. Maybe it's karma for your hard-coded REINSTALLMODE?

@heaths
Copy link
Member Author

heaths commented Apr 3, 2023

"omus" is too strict when we want to allow "channel hopping" of sorts between betas and releases. That said, REMOVE is still "ALL" by the end. Maybe I should dump properties right after CostFinalize and make sure. Do you know, off hand, if dialog conditions are evaluated early at script time, though? I wouldn't think so, but done much with standard Windows Installer UI beyond simple, static customizations in a very long time since we always ended up using chainers for everything.

It'll be an interesting test, though. I can remove REINSTALLMODE and see if that helps, but would likely be just as confusing since the condition still should work. But, hey, if the shoe fits...

@barnson
Copy link

barnson commented Apr 4, 2023

Sadly, I think you can continue to abuse REINSTALLMODE. The only thing that /x might do is set Preselected, which is a bigger deal on initial dialogs, not the end. I imagine the conditions are evaluated first when the dialog is created. Then there are weird rules for what has to happen to make MSI update the UI (the whole trick about showing a second dialog just to get an update of a control tied to a property).

heaths added a commit to heaths/azure-dev that referenced this issue Apr 4, 2023
heaths added a commit to heaths/azure-dev that referenced this issue Apr 5, 2023
heaths added a commit to heaths/azure-dev that referenced this issue Jun 9, 2023
danieljurek pushed a commit that referenced this issue Aug 14, 2023
* Show uninstall text even with /x /qf

Fixes #1858

* Resolve PR feedback

* Show full UI during uninstall
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working engsys installer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants