From: Takuya Noguchi <takninnovationresearch@xxxxxxxxx> The notation <commit> can be misunderstandable only for commit SHA1, but merge-base accepts any commit references. Like rev-parse, the name of arguments should be <rev> instead of <commit>. Signed-off-by: Takuya Noguchi <takninnovationresearch@xxxxxxxxx> --- doc: use 'rev' instead of 'commit' for merge-base arguments The notation can be misunderstandable only for commit SHA1, but merge-base accepts any commit references. Like rev-parse, the name of arguments should be instead of . Changes since v1: * Use rev instead of ref, not commit-ish which is less used than rev through the document. Signed-off-by: Takuya Noguchi takninnovationresearch@xxxxxxxxx [takninnovationresearch@xxxxxxxxx] Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-719%2Ftnir%2Fmerge-base-supporting-refs-v2 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-719/tnir/merge-base-supporting-refs-v2 Pull-Request: https://github.com/git/git/pull/719 Range-diff vs v1: 1: 9c83cf1c620 ! 1: 6e2157da4b1 doc: use 'ref' instead of 'commit' for merge-base arguments @@ -1,10 +1,10 @@ Author: Takuya Noguchi <takninnovationresearch@xxxxxxxxx> - doc: use 'ref' instead of 'commit' for merge-base arguments + doc: use 'rev' instead of 'commit' for merge-base arguments The notation <commit> can be misunderstandable only for commit SHA1, - but merge-base accepts any commit references. Like reflog, the name of - arguments should be <ref> instead of <commit>. + but merge-base accepts any commit references. Like rev-parse, the name + of arguments should be <rev> instead of <commit>. Signed-off-by: Takuya Noguchi <takninnovationresearch@xxxxxxxxx> @@ -20,11 +20,11 @@ -'git merge-base' --is-ancestor <commit> <commit> -'git merge-base' --independent <commit>... -'git merge-base' --fork-point <ref> [<commit>] -+'git merge-base' [-a|--all] <ref> <ref>... -+'git merge-base' [-a|--all] --octopus <ref>... -+'git merge-base' --is-ancestor <ref> <ref> -+'git merge-base' --independent <ref>... -+'git merge-base' --fork-point <ref> [<ref>] ++'git merge-base' [-a|--all] <rev> <rev>... ++'git merge-base' [-a|--all] --octopus <rev>... ++'git merge-base' --is-ancestor <rev> <rev> ++'git merge-base' --independent <rev>... ++'git merge-base' --fork-point <ref> [<rev>] DESCRIPTION ----------- @@ -41,11 +41,11 @@ - N_("git merge-base --independent <commit>..."), - N_("git merge-base --is-ancestor <commit> <commit>"), - N_("git merge-base --fork-point <ref> [<commit>]"), -+ N_("git merge-base [-a | --all] <ref> <ref>..."), -+ N_("git merge-base [-a | --all] --octopus <ref>..."), -+ N_("git merge-base --independent <ref>..."), -+ N_("git merge-base --is-ancestor <ref> <ref>"), -+ N_("git merge-base --fork-point <ref1> [<ref2>]"), ++ N_("git merge-base [-a | --all] <rev> <rev>..."), ++ N_("git merge-base [-a | --all] --octopus <rev>..."), ++ N_("git merge-base --independent <rev>..."), ++ N_("git merge-base --is-ancestor <rev> <rev>"), ++ N_("git merge-base --fork-point <ref> [<rev>]"), NULL }; @@ -54,7 +54,7 @@ N_("is the first one ancestor of the other?"), 'a'), OPT_CMDMODE(0, "fork-point", &cmdmode, - N_("find where <commit> forked from reflog of <ref>"), 'f'), -+ N_("find where <ref2> forked from reflog of <ref1>"), 'f'), ++ N_("find where <rev> forked from reflog of <ref>"), 'f'), OPT_END() }; Documentation/git-merge-base.txt | 10 +++++----- builtin/merge-base.c | 12 ++++++------ 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/Documentation/git-merge-base.txt b/Documentation/git-merge-base.txt index 2d944e0851f..60438a00871 100644 --- a/Documentation/git-merge-base.txt +++ b/Documentation/git-merge-base.txt @@ -9,11 +9,11 @@ git-merge-base - Find as good common ancestors as possible for a merge SYNOPSIS -------- [verse] -'git merge-base' [-a|--all] <commit> <commit>... -'git merge-base' [-a|--all] --octopus <commit>... -'git merge-base' --is-ancestor <commit> <commit> -'git merge-base' --independent <commit>... -'git merge-base' --fork-point <ref> [<commit>] +'git merge-base' [-a|--all] <rev> <rev>... +'git merge-base' [-a|--all] --octopus <rev>... +'git merge-base' --is-ancestor <rev> <rev> +'git merge-base' --independent <rev>... +'git merge-base' --fork-point <ref> [<rev>] DESCRIPTION ----------- diff --git a/builtin/merge-base.c b/builtin/merge-base.c index e3f8da13b69..e03f2269e88 100644 --- a/builtin/merge-base.c +++ b/builtin/merge-base.c @@ -29,11 +29,11 @@ static int show_merge_base(struct commit **rev, int rev_nr, int show_all) } static const char * const merge_base_usage[] = { - N_("git merge-base [-a | --all] <commit> <commit>..."), - N_("git merge-base [-a | --all] --octopus <commit>..."), - N_("git merge-base --independent <commit>..."), - N_("git merge-base --is-ancestor <commit> <commit>"), - N_("git merge-base --fork-point <ref> [<commit>]"), + N_("git merge-base [-a | --all] <rev> <rev>..."), + N_("git merge-base [-a | --all] --octopus <rev>..."), + N_("git merge-base --independent <rev>..."), + N_("git merge-base --is-ancestor <rev> <rev>"), + N_("git merge-base --fork-point <ref> [<rev>]"), NULL }; @@ -158,7 +158,7 @@ int cmd_merge_base(int argc, const char **argv, const char *prefix) OPT_CMDMODE(0, "is-ancestor", &cmdmode, N_("is the first one ancestor of the other?"), 'a'), OPT_CMDMODE(0, "fork-point", &cmdmode, - N_("find where <commit> forked from reflog of <ref>"), 'f'), + N_("find where <rev> forked from reflog of <ref>"), 'f'), OPT_END() }; base-commit: 2f268890c2cd2f115424936bbee27f8269080e5e -- gitgitgadget