-
Notifications
You must be signed in to change notification settings - Fork 18.1k
[release-branch.go1.20] path/filepath: consider \\?\c: as a volume on Windows #64091
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
[release-branch.go1.20] path/filepath: consider \\?\c: as a volume on Windows #64091
Conversation
… Windows While fixing several bugs in path handling on Windows, beginning with \\?\. Prior to #540277, VolumeName considered the first path component after the \\?\ prefix to be part of the volume name. After, it considered only the \\? prefix to be the volume name. Restore the previous behavior. Fixes golang#64040 Updates golang#64028 Change-Id: I6523789e61776342800bd607fb3f29d496257e68 Reviewed-on: https://go-review.googlesource.com/c/go/+/541175 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Roland Shoemaker <[email protected]> (cherry picked from commit eda42f7)
This PR (HEAD: e10b05d) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/541520. Important tips:
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541520. |
This PR (HEAD: e10b05d) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/541836. Important tips:
|
Message from Rob Pike: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541836. |
Message from Mauri de Souza Meneguzzo: Patch Set 1: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/541836. |
This PR is being closed because golang.org/cl/541836 has been abandoned. Thanks for trying to prepare this cherry-pick. As mentioned at the bottom of https://go.dev/wiki/MinorReleases#making-cherry-pick-cls, using Pull Requests is unfortunately not a supported workflow at this time. |
While fixing several bugs in path handling on Windows, beginning with \?.
Prior to #540277, VolumeName considered the first path component after the \?\ prefix to be part of the volume name. After, it considered only the \? prefix to be the volume name.
Restore the previous behavior.
Fixes #64040
Updates #64028
Change-Id: I6523789e61776342800bd607fb3f29d496257e68 Reviewed-on: https://go-review.googlesource.com/c/go/+/541175
LUCI-TryBot-Result: Go LUCI [email protected]
Reviewed-by: Roland Shoemaker [email protected]
(cherry picked from commit eda42f7)