-
Notifications
You must be signed in to change notification settings - Fork 14k
Canonicalize path when displaying a FileName::Real
#68654
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
Canonicalize path when displaying a FileName::Real
#68654
Conversation
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
What effect does this PR have on windows systems? |
|
@estebank I couldn’t test this, as I haven’t got a Windows device myself |
87d3774 to
2c67a66
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
Could you please canonicalize when creating the |
b0947bb to
b13b60f
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
Ping from triage: @LeSeulArtichaut can you please address the check failures? Thank you. |
c016e98 to
e208404
Compare
|
@JohnCSimon Done! |
e208404 to
ffcb348
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
ffcb348 to
b7d992b
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
I think you'd need to modify |
|
ping from triage: |
|
☔ The latest upstream changes (presumably #66364) made this pull request unmergeable. Please resolve the merge conflicts. |
|
@LeSeulArtichaut closing this as inactive because it hasn't been moved in a month. Please reopen when you're ready to continue with this. |
Addresses #51349 (?)
I'm also not sure if failing to canonicalize the path should result in failing to display the
FileName, or if the original path should be used as a fallback. My current implementation uses the latter, as it seemed the "wisest" solution to me.r? @estebank