Skip to content

package: Make Windows installer respect customized path#41653

Merged
jdm merged 5 commits intoservo:mainfrom
yezhizhen:fix-windows
Jan 3, 2026
Merged

package: Make Windows installer respect customized path#41653
jdm merged 5 commits intoservo:mainfrom
yezhizhen:fix-windows

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Jan 3, 2026

Create a Burn variable and pass it to MSI, in order to really install the bundle on customized path chosen in UI.

Doc: https://docs.firegiant.com/wix/whatsnew/#default-installation-folder

Testing: Tested with try.

  • Install works
  • Uninstall works
  • Default path works

Fixes: #40995

Signed-off-by: Euclid Ye <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 3, 2026
@yezhizhen yezhizhen marked this pull request as draft January 3, 2026 13:24
Signed-off-by: Euclid Ye <[email protected]>
This reverts commit 23de37b.

Signed-off-by: Euclid Ye <[email protected]>
This reverts commit b6e6e73.

Signed-off-by: Euclid Ye <[email protected]>
Signed-off-by: Euclid Ye <[email protected]>
@yezhizhen
Copy link
Copy Markdown
Member Author

Tried a few approaches. This is not newcomer friendly..

https://github.com/servo/servo/actions/runs/20678599529/job/59369631656

I tried the artifact, and it now works with customized path.

@yezhizhen yezhizhen marked this pull request as ready for review January 3, 2026 14:50
@yezhizhen yezhizhen changed the title WIP: Fix windows installer package: Make Windows installer respect customized path Jan 3, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 3, 2026
@jdm jdm added this pull request to the merge queue Jan 3, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 3, 2026
Merged via the queue into servo:main with commit 71b74d7 Jan 3, 2026
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 3, 2026
@yezhizhen yezhizhen deleted the fix-windows branch January 4, 2026 00:41
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.

Windows Nightly installer ignores customized path

3 participants