Page MenuHomePhabricator

Use `git diff a b` for ranges, not `git diff a..b`
ClosedPublic

Authored by epriestley on Jul 11 2014, 4:57 PM.
Tags
None
Referenced Files
F19515196: D9898.id23770.diff
Wed, Jan 14, 9:05 AM
F19512022: D9898.diff
Mon, Jan 12, 10:57 AM
F19491323: D9898.id23770.diff
Thu, Jan 8, 7:18 AM
F19491322: D9898.id23763.diff
Thu, Jan 8, 7:18 AM
F19091838: D9898.id23770.diff
Dec 3 2025, 3:31 PM
F19036437: D9898.id23770.diff
Nov 25 2025, 8:08 PM
F18990333: D9898.id23763.diff
Nov 18 2025, 5:41 PM
F18749002: D9898.id.diff
Oct 4 2025, 1:53 AM
Subscribers

Details

Summary

These are documented as being identical, but git diff a b works if a is a tree (for example, 4b825d..., the empty tree hash), but git diff a..b does not.

Particularly, with the a..b form, arc diff --base arc:empty does not work. With the a b form, it does.

Test Plan

Ran arc diff --base arc:empty in a repository and got a diff.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Use `git diff a b` for ranges, not `git diff a..b`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: talshiri, btrahan.
talshiri edited edge metadata.
This revision is now accepted and ready to land.Jul 11 2014, 5:43 PM
epriestley updated this revision to Diff 23770.

Closed by commit rARCf9f2092246d2 (authored by @epriestley).