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 not look in root for .config\dotnet-tools.json manifest for windows #33282

Merged
merged 3 commits into from
Jun 22, 2023

Conversation

JL03-Yue
Copy link
Member

Targeting branch 8.0.0

When use this command in Windows system. The process will search for tool information in the .config\dotnet-tools.json file from the bottom up and download it.

For Windows, the program does not check the root folder for the tool manifest if the Environment Variable `CHECK_MANIFEST_IN_ROOT" is not set.

For Linux/MacOS, the program checks the root folder for the tool manifest.

JL03-Yue added 2 commits June 15, 2023 08:01
When use this command in Windows system, The process will search for tool information in the .config\dotnet-tools.json file from the bottom up and download it.

For Windows, the program does not check the root folder for the tool manifest if the Environment Variable `CHECK_MANIFEST_IN_ROOT" is not set.

For Linux/MacOS, the program checks the root folder for the tool manifest.
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Jun 15, 2023
Copy link
Member

@GrabYourPitchforks GrabYourPitchforks left a comment

Choose a reason for hiding this comment

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

Looks like a straightforward port of the approved 7.0 change.

@GrabYourPitchforks
Copy link
Member

Marc told me offline that this change already came in via an automated PR #33290. That would explain the conflicts you're seeing.

@marcpopMSFT
Copy link
Member

Post-merge resolution looks like the only change left in this PR is white space so I think we're good to close it as that confirms the codeflow in 33290 brought in the fix.

@JL03-Yue JL03-Yue merged commit 0da4075 into dotnet:main Jun 22, 2023
@JL03-Yue JL03-Yue deleted the liannie/msrc-8-0-0 branch June 22, 2023 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants