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

Use PerUserProgramFilesFolder for default INSTALLFOLDER in a per-user package #582

Merged
merged 3 commits into from
Dec 30, 2024

Conversation

robmen
Copy link
Member

@robmen robmen commented Dec 29, 2024

Also, adds a standard directory for PerUserProgramFilesFolder.

Fixes wixtoolset/issues#8101

@robmen robmen requested a review from barnson December 29, 2024 02:02
@robmen robmen enabled auto-merge (rebase) December 29, 2024 02:02
barnson
barnson previously approved these changes Dec 29, 2024
src/api/wix/WixToolset.Data/Symbols/WixPackageSymbol.cs Outdated Show resolved Hide resolved
@robmen robmen disabled auto-merge December 29, 2024 16:25
@robmen robmen marked this pull request as draft December 29, 2024 16:25
Windows Installer uses LocalAppDataFolder\Programs as the replacement for
ProgramFilesFolder when installing per-user in a dual-mode package. Add a
standard directory in WiX to match it for use by default INSTALLFOLDER.
@robmen robmen force-pushed the robmen/peruser-installfolder branch from 6be148d to 6ff0cd5 Compare December 29, 2024 17:52
@robmen robmen marked this pull request as ready for review December 29, 2024 17:52
@robmen robmen requested a review from barnson December 29, 2024 17:52
@robmen robmen enabled auto-merge (rebase) December 29, 2024 17:52
@robmen robmen merged commit d9bb113 into main Dec 30, 2024
2 checks passed
@robmen robmen deleted the robmen/peruser-installfolder branch December 30, 2024 00:29
@github-actions github-actions bot locked and limited conversation to collaborators Dec 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting the "Scope" property to "perUser" results in an unexpected INSTALLFOLDER path
2 participants