Page MenuHomePhabricator

Merge the "--raw" and "--raw-command" flags to "arc diff" into "--raw <path>"
Needs ReviewPublic

Authored by epriestley on May 30 2020, 11:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 4:02 PM
Unknown Object (File)
Sun, Nov 10, 2:46 PM
Unknown Object (File)
Sun, Oct 27, 8:12 AM
Unknown Object (File)
Oct 22 2024, 8:33 PM
Unknown Object (File)
Sep 21 2024, 9:17 PM
Unknown Object (File)
Sep 6 2024, 8:05 AM
Unknown Object (File)
Aug 29 2024, 3:53 AM
Unknown Object (File)
Aug 28 2024, 11:09 AM
Subscribers
None
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Maniphest Tasks
T13544: Modernize the "arc diff" workflow
Summary

Ref T13544. These flags can reasonably be combined into a single flag, and I'm also not entirely comfortable with executing arbitrary subprocesses directly from CLI arguments in modern arc. There are other ways to accomplish this goal, but I'd like to move toward fewer sources of arbitrary command execution.

Test Plan

Created this revision with git show | arc diff --raw.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 24544
Build 33828: arc lint + arc unit

Event Timeline

epriestley created this revision.