HomeSealhub

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

Description

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.

Maniphest Tasks: T13562

Differential Revision: https://secure.phabricator.com/D21643

Details

Provenance
epriestleyAuthored on Mar 22 2021, 19:35
kuba-orlikPushed on Jan 24 2022, 10:17
Parents
R118:cc23551a7de8: Update Paste help to include missing "--"
Branches
Unknown
Tags
Unknown

Event Timeline