Differential D21643

On Windows, implement "Filesystem::copyFile()" with "copy()"
ClosedPublic
Actions

Authored by epriestley on Mar 22 2021, 6:43 PM.

Tags

None

Referenced Files

	F15424368: D21643.id51509.diff
	Sat, Mar 22, 9:49 PM

	F15422623: D21643.diff
	Sat, Mar 22, 9:51 AM

	F15414327: D21643.diff
	Thu, Mar 20, 12:10 AM

	F15409977: D21643.id.diff
	Wed, Mar 19, 6:21 AM

	F15384773: D21643.id51510.diff
	Fri, Mar 14, 8:46 PM

	F15344514: D21643.diff
	Mon, Mar 10, 4:59 AM

	F15286132: D21643.diff
	Tue, Mar 4, 6:30 PM

	Unknown Object (File)
	Sat, Feb 22, 4:46 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13562: Windows may have irregular behaviors in unusual cases when copying a file to preserve attributes/permissions
Commits: rARC7af9846f994a: (stable) Promote 2021 Week 13
rARCf0f95e5b2612: On Windows, implement "Filesystem::copyFile()" with "copy()"

Summary

Ref T13562. Currently, "Filesystem::copyFile()" uses "copy", which doesn't work now that we no longer invoke "cmd.exe" by default.

Use "copy()" instead.

Note that this whole function is probably nonsense, but I'll follow up on T13562.

Test Plan

Created a standalone script which runs "Filesystem::copyFile()".
- Before: failed to copy any file.
- After: succesfully copied normal files.
- After: failed to copy a file over an existing directory with a reasonable error.
- After: failed to copy a file over itself with a reasonable error.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 22 2021, 6:43 PM

Harbormaster completed remote builds in B25317: Diff 51509.Mar 22 2021, 6:43 PM

epriestley requested review of this revision.Mar 22 2021, 6:43 PM

epriestley mentioned this in T13562: Windows may have irregular behaviors in unusual cases when copying a file to preserve attributes/permissions.Mar 22 2021, 6:59 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 22 2021, 7:00 PM

Closed by commit rARCf0f95e5b2612: On Windows, implement "Filesystem::copyFile()" with "copy()" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCf0f95e5b2612: On Windows, implement "Filesystem::copyFile()" with "copy()".

epriestley added a commit: rARC7af9846f994a: (stable) Promote 2021 Week 13.Mar 26 2021, 6:05 PM

Revision Contents
Changeset List

Path

Size

src/

filesystem/

24 lines

Diff 51510

src/filesystem/Filesystem.php

Loading...