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

os: RemoveAll no longer returns *os.PathError [1.12 backport] #30859

Closed
gopherbot opened this issue Mar 15, 2019 · 2 comments
Closed

os: RemoveAll no longer returns *os.PathError [1.12 backport] #30859

gopherbot opened this issue Mar 15, 2019 · 2 comments

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Mar 15, 2019

@bradfitz requested issue #30491 to be considered for backport to the next 1.12 minor release.

@gopherbot, please backport to Go 1.12.

@gopherbot

This comment has been minimized.

Copy link
Author

@gopherbot gopherbot commented Mar 15, 2019

Change https://golang.org/cl/167739 mentions this issue: [release-branch.go1.12] os: consistently return PathError from RemoveAll

@gopherbot

This comment has been minimized.

Copy link
Author

@gopherbot gopherbot commented Mar 15, 2019

Closed by merging ff04803 to release-branch.go1.12.

@gopherbot gopherbot closed this Mar 15, 2019
gopherbot pushed a commit that referenced this issue Mar 15, 2019
Fixes #30859
Updates #30491

Change-Id: If4070e5d39d8649643d7e90f6f3eb499642e25ab
Reviewed-on: https://go-review.googlesource.com/c/go/+/164720
Run-TryBot: Baokun Lee <nototon@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
(cherry picked from commit d039e12)
Reviewed-on: https://go-review.googlesource.com/c/go/+/167739
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Baokun Lee <nototon@gmail.com>
@golang golang locked and limited conversation to collaborators Mar 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.