Page MenuHomePhabricator

Use --no-ext-diff in `arc land` call to `git diff`

Authored by epriestley on Aug 6 2016, 1:41 PM.
Referenced Files
Unknown Object (File)
Mon, Feb 27, 8:53 PM
Unknown Object (File)
Mon, Feb 27, 4:21 AM
Unknown Object (File)
Mon, Feb 27, 4:20 AM
Unknown Object (File)
Mon, Feb 27, 4:20 AM
Unknown Object (File)
Feb 17 2023, 3:50 PM
Unknown Object (File)
Feb 9 2023, 2:34 AM
Unknown Object (File)
Jan 24 2023, 6:16 PM
Unknown Object (File)
Jan 8 2023, 7:50 AM



Fixes T11435. This isn't a perfect solution since there's a little code duplication, but a perfect solution is probably a bit more involved.

See T11435 for some discussion. In particular, most git diff commands already get this flag via ArcanistGitAPI->getDiffBaseOptions().

Test Plan

Will land this change.

Diff Detail

rARC Arcanist
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Use --no-ext-diff in `arc land` call to `git diff`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 6 2016, 2:18 PM
This revision was automatically updated to reflect the committed changes.