I think the article is assuming the base for `arc diff` is always the parent revision, i.e. `arc diff HEAD^`, which is how the workflow works best. Strangely I don't think the open source Phabricator is set up to do this by default so you have to actually type `arc diff HEAD^` (there's probably some setting somewhere so that you can make this the default).

On the diff in Phabricator you can enter the dependencies manually. Really the tooling ought to do this for you (and at Facebook our internal tooling does do this) but for now manually specifying the dependencies is not terrible. Then Phabricator shows you the nice dependency tree in the UI, so you can see the state of all of your diffs in the stack.

Cheers

Simon

On Fri, 5 Oct 2018 at 04:30, Niklas Hambüchen <mail@nh2.me> wrote:

There are some things in these argumentations that I don't get.

When you have a stack of commits on top of master, like:

* C
|
* B
|
* A
|
* master

What do you use as base for `arc diff` for each of them?

If B depends on A (the patch expressed by B doesn't apply if A was applied first),
do you still use master as a base for B, or do you use Phabricator's feature to have diffs depend on other diffs?