Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Sep 3, 2025

This PR was automatically opened by a GitHub action. Review the changes included in this PR and determine if they should be included in the release branch. If yes, merge the PR. Otherwise revert changes that should not be included on this branch.

…st (#1485)

We have observed `ERROR_ACCESS_DENIED` in CI on
`SetRenameInformationFile`. Try to make the path more robust by first
performing a kernel based rename with POSIX semantics. This requires
Windows 10 1809+. If that is unsuccessful, verify that attributes are
not to blame.

A failure may still occur if the file is on a different volume. In such
a case, fallback to the `MoveFileExW` operation to perform a copy +
delete operation.

Hopefully this should make the implementation more robust to failures.
@jmschonfeld jmschonfeld marked this pull request as ready for review September 3, 2025 16:05
@jmschonfeld
Copy link
Contributor

@swift-ci please test

@jmschonfeld jmschonfeld requested a review from compnerd September 3, 2025 16:05
@jmschonfeld
Copy link
Contributor

@compnerd I know we're still investigating the full fix here, but this change as it stands is still good to go ahead and forward from 6.2 to main right since it's still the right direction even if it's not complete, right?

@parkera parkera merged commit 23013ca into main Sep 3, 2025
3 checks passed
@parkera parkera deleted the automerge/merge-main-2025-09-03_09-03 branch September 3, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants