<div dir="ltr">I don't know if it does or not, but I'm not sure that it needs to: rename_fd fails with a permission denied error on a file marked as read-only by chmod on Windows, which is presumably sufficient for this case, no?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 5 December 2017 at 16:20, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">James Henderson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> jhenderson accepted this revision.<br>
> jhenderson added a comment.<br>
> This revision is now accepted and ready to land.<br>
><br>
> LGTM. I confirmed that it works for the case I saw with LLD too.<br>
><br>
> Is there a particular reason why you're using icacls instead of chmod (available in the GnuWin32 tools) for the test? I'd say it's more familiar to most people, but I'm not bothered either way.<br>
<br>
</span>Can chmod on windows change the D and DC ACL entries?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>