-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Remove duplicate entries from dotnet --info #60376
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
Conversation
|
Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov Issue DetailsFixes #59833 Result before: After:
|
|
I pushed a fix to this - the trailing slashes should be removed from all paths which are considered in the MLL search. Locally this made all the tests pass. |
|
Thank you @vitek-karas. I had a hard time running these tests locally. Appreciate your help. |
vitek-karas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
|
/backport to release/6.0 |
|
Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1343949537 |
* Remove duplicate entries from dotnet --info * Remove trailing slashes from globally registered locations as well * PR feedback * Tests not ported due to missing infrastructure Co-authored-by: vitek-karas <vitek.karas@microsoft.com>
* Remove duplicate entries from dotnet --info * Remove trailing slashes from globally registered locations as well * PR feedback * Tests not ported due to missing infrastructure Co-authored-by: vitek-karas <vitek.karas@microsoft.com> Co-authored-by: vitek-karas <vitek.karas@microsoft.com>
Fixes #59833
Result before:
After: