Page MenuHomePhabricator

Use phutil functions to copy/move files
ClosedPublic

Authored by alexmv on Jul 11 2016, 10:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 19, 7:51 PM
Unknown Object (File)
Mon, Mar 4, 7:07 PM
Unknown Object (File)
Feb 22 2024, 10:17 AM
Unknown Object (File)
Feb 8 2024, 2:17 PM
Unknown Object (File)
Feb 3 2024, 11:21 AM
Unknown Object (File)
Jan 20 2024, 4:31 PM
Unknown Object (File)
Jan 16 2024, 4:07 PM
Unknown Object (File)
Jan 12 2024, 4:44 PM
Subscribers

Details

Summary

The cp and mv commands, when run from a Windows
environment, error out. Use library functions from Filesystem
for cross-platform compatibility.

Test Plan

Ran arc lint with auto-fix lint errors on both Linux and
Windows.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alexmv retitled this revision from to Provide windows command-line equivalents for `cp` and `mv` commands.
alexmv updated this object.
alexmv edited the test plan for this revision. (Show Details)
epriestley added a reviewer: epriestley.

These should go in Filesystem in libphutil, as copyFile($src, $dst) and something like overwriteFile($src, $dst), and have unit tests so they survive the coming apocalypse in T8298 / D15675.

This revision now requires changes to proceed.Jul 11 2016, 11:23 PM
alexmv retitled this revision from Provide windows command-line equivalents for `cp` and `mv` commands to Use phutil functions to copy/move files.Jul 12 2016, 12:20 AM
alexmv updated this object.
alexmv edited edge metadata.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 12 2016, 1:26 PM

Thanks! Looks like you're already in Blessed Committers. I added you to Community for good measure.

This revision was automatically updated to reflect the committed changes.