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

Disable Workloads dependency on WiX nupkg during source-build to remove prebuilt #8335

Merged
merged 1 commit into from
Jan 12, 2022

Conversation

lbussell
Copy link
Contributor

The source-build team is making an effort to upstream our patches this week, see dotnet/source-build#2708.

This PR removes the dependency on WiX during source-build to prevent pulling in the WiX prebuilt binary. There was previously a PR for this change that became outdated and was closed. Additionally, there is an issue tracking the WiX prebuilt that will be resolved if this patch is merged:

Summary of the changes:

  • Rename files which directly/indirectly depend on WiX to have the .wix.cs extension
  • Exclude all files with the .wix.cs extension from the build during source-build

This removes the prebuilt WiX dependency: dotnet#8014

The patch is temporary, pending upstream fix and dependency flow.
Copy link
Member

@joeloff joeloff left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me

@lbussell lbussell merged commit 0de9a22 into dotnet:main Jan 12, 2022
@lbussell lbussell deleted the apply-source-build-patches branch January 12, 2022 02:39
@lbussell lbussell restored the apply-source-build-patches branch January 24, 2022 23:46
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.

3 participants