tree 3c626be6a3ac1faa2592e55929862ec1c4a8c7e0
parent bf1f4ec7fa3938e1ce1297b367c16aea30280697
author Bryan C. Mills <bcmills@google.com> 1559681459 -0400
committer Bryan C. Mills <bcmills@google.com> 1559750293 +0000

cmd/go: replace uses of ioutil.ReadFile with renameio.ReadFile

Windows does not have atomic renames; instead, it produces one of a
handful of errors in case a read races with a rename.

CL 180219 added a utility function that retries those errors in most
cases; this change updates the locations that use renameio for writes
to also use the new renameio.ReadFile function for reads.

It remains possible for a renameio.ReadFile to fail with a spurious
ERROR_FILE_NOT_FOUND, but with retries in place for the other errors
(and practical limits on write concurrency) such failures are unlikely
in practice.

Fixes #32188

Change-Id: I78c81051cc871325c1e3229e696b921b0fcd865a
Reviewed-on: https://go-review.googlesource.com/c/go/+/180517
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Jay Conrod <jayconrod@google.com>