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
F15458884: D21305.id50722.diff
Mon, Mar 31, 8:01 AM
F15457196: D21305.id.diff
Sun, Mar 30, 2:54 PM
F15455119: D21305.diff
Sat, Mar 29, 10:20 PM
F15426971: D21305.diff
Sun, Mar 23, 11:56 AM
F15408454: D21305.id.diff
Tue, Mar 18, 11:11 PM
F15395735: D21305.id50722.diff
Sun, Mar 16, 9:24 AM
F15333191: D21305.diff
Sat, Mar 8, 12:36 AM
Unknown Object (File)
Feb 9 2025, 9:25 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.