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

[release/8.0] Prepend wix tooling to the path. #14772

Merged
merged 1 commit into from
May 16, 2024

Conversation

mmitche
Copy link
Member

@mmitche mmitche commented May 16, 2024

In some newer images, WiX tooling is on the path, so we should prepend rather than append to use the desired version. Note that we put the tools on the path because the "wix tool" we run may actually be a script that invokes tools.

To double check:

In some newer images, WiX tooling is on the path, so we should prepend rather than append to use the desired version.
Note that we put the tools on the path because the "wix tool" we run may actually be a script that invokes tools.
@mmitche mmitche changed the title Prepend wix tooling to the path. [release/8.0] Prepend wix tooling to the path. May 16, 2024
@mmitche mmitche enabled auto-merge (squash) May 16, 2024 21:55
@mmitche mmitche merged commit e6f70c7 into dotnet:release/8.0 May 16, 2024
11 checks passed
Cosifne added a commit to Cosifne/roslyn that referenced this pull request Jul 10, 2024
…se branch

As offline discussed, the goal is to flow dotnet#73507 which carries dotnet/arcade#14772 into servicing branch
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.

2 participants