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

Do a few tricks to shorten the intermediate output paths for the runtime packs #59479

Merged
merged 3 commits into from
Dec 14, 2024

Conversation

jkoritzinsky
Copy link
Member

The Wix tooling can't handle long paths, which is blocking VMR integration. This change gets our paths to be short enough that even a long-named analyzer's resource DLL won't go past MAX_PATH in a VMR build.

Will unblock dotnet/sdk#45430

…ime packs

The Wix tooling can't handle long paths, which is blocking VMR integration. This change gets our paths to be short enough that even a long-named analyzer's resource DLL won't go past MAX_PATH in a VMR build.
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Dec 13, 2024
@eerhardt
Copy link
Member

It's depressing that we are dealing with max path issues at the end of 2024. 😢

Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the fix!

@jkoritzinsky jkoritzinsky enabled auto-merge (squash) December 14, 2024 01:00
@jkoritzinsky jkoritzinsky merged commit eaf6162 into dotnet:main Dec 14, 2024
27 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview1 milestone Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants