use NtSetInformationFile if os.Rename fails#220
Conversation
Signed-off-by: Manju Karikatti <manju.karikatti@docker.com>
477163b to
f141cb9
Compare
|
This obviously needs some windows-specific tests. |
kolyshkin
left a comment
There was a problem hiding this comment.
nit: you've introduced atomicwriterRenameAt in the first commit, and then rename it to atomicwriterRename in the second commit. Why not use the right name from the start?
nit: stuttering at the name (atomicwriter.atomicwriterRename).
sorry I thought in your initial review comment, you suggested to remove 'At' ? |
Yes, but here I'm not saying |
This PR adds
atomicwriterRenameAtwhich usesNtSetInformationFileto rename a file on Windows.