-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Remove duplicate entries from dotnet --info #60376
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. |
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 <[email protected]>
* 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 <[email protected]> Co-authored-by: vitek-karas <[email protected]>
Fixes #59833
Result before:
After: