From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: git@vger.kernel.org
Cc: "Junio C Hamano" <gitster@pobox.com>,
dstolee@microsoft.com, git@jeffhostetler.com, peff@peff.net,
johannes.schindelin@gmx.de, jrnieder@gmail.com,
"Linus Torvalds" <torvalds@linux-foundation.org>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Subject: [PATCH 19/20] abbrev: support relative abbrev values
Date: Fri, 8 Jun 2018 22:41:35 +0000 [thread overview]
Message-ID: <20180608224136.20220-20-avarab@gmail.com> (raw)
In-Reply-To: <20180608224136.20220-1-avarab@gmail.com>
Change the core.abbrev config variable and the corresponding --abbrev
command-line option to support relative values such as +1 or -1.
Before Linus's e6c587c733 ("abbrev: auto size the default
abbreviation", 2016-09-30) git would default to abbreviating object
names to 7-hexdigits, and only picking longer SHA-1s as needed if that
was ambiguous.
That change instead set the default length as a function of the
estimated current count of objects:
Based on the expectation that we would see collision in a
repository with 2^(2N) objects when using object names shortened
to first N bits, use sufficient number of hexdigits to cover the
number of objects in the repository. Each hexdigit (4-bits) we
add to the shortened name allows us to have four times (2-bits) as
many objects in the repository.
By supporting relative values for core.abbrev we can allow users to
consistently opt-in for either a higher or lower probability of
collision, without needing to hardcode a given numeric value like
"10", which would be overkill on some repositories, and far to small
on others.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
Documentation/config.txt | 6 ++
Documentation/diff-options.txt | 3 +
Documentation/git-blame.txt | 3 +
Documentation/git-branch.txt | 3 +
Documentation/git-describe.txt | 3 +
Documentation/git-ls-files.txt | 3 +
Documentation/git-ls-tree.txt | 3 +
Documentation/git-show-ref.txt | 3 +
cache.h | 1 +
config.c | 11 +++
diff.c | 18 +++-
environment.c | 1 +
parse-options-cb.c | 12 ++-
revision.c | 18 +++-
sha1-name.c | 11 +++
t/t0014-abbrev.sh | 170 ++++++++++++++++++++++-----------
16 files changed, 204 insertions(+), 65 deletions(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index ab641bf5a9..abf07be7b6 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -919,6 +919,12 @@ core.abbrev::
in your repository, which hopefully is enough for
abbreviated object names to stay unique for some time.
The minimum length is 4.
++
+This can also be set to relative values such as `+2` or `-2`, which
+means to add or subtract N characters from the SHA-1 that Git would
+otherwise print, this allows for producing more future-proof SHA-1s
+for use within a given project, while adjusting the value for the
+current approximate number of objects.
add.ignoreErrors::
add.ignore-errors (deprecated)::
diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt
index f466600972..f1114a7b8d 100644
--- a/Documentation/diff-options.txt
+++ b/Documentation/diff-options.txt
@@ -384,6 +384,9 @@ endif::git-format-patch[]
independent of the `--full-index` option above, which controls
the diff-patch output format. Non default number of
digits can be specified with `--abbrev=<n>`.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
-B[<n>][/<m>]::
--break-rewrites[=[<n>][/<m>]]::
diff --git a/Documentation/git-blame.txt b/Documentation/git-blame.txt
index d6cddbcb2e..8559d3b0c7 100644
--- a/Documentation/git-blame.txt
+++ b/Documentation/git-blame.txt
@@ -99,6 +99,9 @@ requested, resulting in unaligned output.
Before 2.19, setting `core.abbrev=40` wouldn't apply the above rule
and would cause blame to emit output that was unaligned. This bug has
since been fixed.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
THE PORCELAIN FORMAT
diff --git a/Documentation/git-branch.txt b/Documentation/git-branch.txt
index 02eccbb931..0f8032cec6 100644
--- a/Documentation/git-branch.txt
+++ b/Documentation/git-branch.txt
@@ -182,6 +182,9 @@ See `--create-reflog` above for details.
Alter the sha1's minimum display length in the output listing.
The default value is 7 and can be overridden by the `core.abbrev`
config option.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
--no-abbrev::
Display the full sha1s in the output listing rather than abbreviating them.
diff --git a/Documentation/git-describe.txt b/Documentation/git-describe.txt
index e027fb8c4b..a3d5c7e817 100644
--- a/Documentation/git-describe.txt
+++ b/Documentation/git-describe.txt
@@ -65,6 +65,9 @@ OPTIONS
abbreviated object name, use <n> digits, or as many digits
as needed to form a unique object name. An <n> of 0
will suppress long format, only showing the closest tag.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
--candidates=<n>::
Instead of considering only the 10 most recent tags as
diff --git a/Documentation/git-ls-files.txt b/Documentation/git-ls-files.txt
index 5298f1bc30..f9af2b23bf 100644
--- a/Documentation/git-ls-files.txt
+++ b/Documentation/git-ls-files.txt
@@ -153,6 +153,9 @@ a space) at the start of each line:
Instead of showing the full 40-byte hexadecimal object
lines, show only a partial prefix.
Non default number of digits can be specified with --abbrev=<n>.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
--debug::
After each line that describes a file, add more data about its
diff --git a/Documentation/git-ls-tree.txt b/Documentation/git-ls-tree.txt
index 9dee7bef35..bcba39eed8 100644
--- a/Documentation/git-ls-tree.txt
+++ b/Documentation/git-ls-tree.txt
@@ -64,6 +64,9 @@ OPTIONS
Instead of showing the full 40-byte hexadecimal object
lines, show only a partial prefix.
Non default number of digits can be specified with --abbrev=<n>.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
--full-name::
Instead of showing the path names relative to the current working
diff --git a/Documentation/git-show-ref.txt b/Documentation/git-show-ref.txt
index d28e6154c6..05c5579075 100644
--- a/Documentation/git-show-ref.txt
+++ b/Documentation/git-show-ref.txt
@@ -66,6 +66,9 @@ OPTIONS
Abbreviate the object name. When using `--hash`, you do
not have to say `--hash --abbrev`; `--hash=n` would do.
++
+Can also be set to a relative value, see `core.abbrev` in
+linkgit:git-diff[1].
-q::
--quiet::
diff --git a/cache.h b/cache.h
index 89a107a7f7..0fb4211804 100644
--- a/cache.h
+++ b/cache.h
@@ -772,6 +772,7 @@ extern int check_stat;
extern int quote_path_fully;
extern int has_symlinks;
extern int minimum_abbrev, default_abbrev;
+extern int default_abbrev_relative;
extern int ignore_case;
extern int assume_unchanged;
extern int prefer_symlink_refs;
diff --git a/config.c b/config.c
index 12f762ad92..cd95c6bdfb 100644
--- a/config.c
+++ b/config.c
@@ -1151,6 +1151,17 @@ static int git_default_core_config(const char *var, const char *value)
return config_error_nonbool(var);
if (!strcasecmp(value, "auto")) {
default_abbrev = -1;
+ } else if (*value == '+' || *value == '-') {
+ int relative = git_config_int(var, value);
+ if (relative == 0)
+ die(_("bad core.abbrev value %s. "
+ "relative values must be non-zero"),
+ value);
+ if (abs(relative) > GIT_SHA1_HEXSZ)
+ die(_("bad core.abbrev value %s. "
+ "impossibly out of range"),
+ value);
+ default_abbrev_relative = relative;
} else {
int abbrev = git_config_int(var, value);
if (abbrev < minimum_abbrev || abbrev > 40)
diff --git a/diff.c b/diff.c
index e0141cfbc0..f7861b8472 100644
--- a/diff.c
+++ b/diff.c
@@ -4801,16 +4801,28 @@ int diff_opt_parse(struct diff_options *options,
else if (!strcmp(arg, "--abbrev"))
options->abbrev = DEFAULT_ABBREV;
else if (skip_prefix(arg, "--abbrev=", &arg)) {
+ int v;
char *end;
if (!strcmp(arg, ""))
die("--abbrev expects a value, got '%s'", arg);
- options->abbrev = strtoul(arg, &end, 10);
+ v = strtoul(arg, &end, 10);
if (*end)
die("--abbrev expects a numerical value, got '%s'", arg);
- if (options->abbrev < MINIMUM_ABBREV) {
+ if (*arg == '+' || *arg == '-') {
+ if (v == 0) {
+ die("relative abbrev must be non-zero");
+ } else if (abs(v) > the_hash_algo->hexsz) {
+ die("relative abbrev impossibly out of range");
+ } else {
+ default_abbrev_relative = v;
+ options->abbrev = DEFAULT_ABBREV;
+ }
+ } else if (v < MINIMUM_ABBREV) {
options->abbrev = MINIMUM_ABBREV;
- } else if (the_hash_algo->hexsz < options->abbrev) {
+ } else if (the_hash_algo->hexsz < v) {
options->abbrev = the_hash_algo->hexsz;
+ } else {
+ options->abbrev = v;
}
}
else if ((argcount = parse_long_opt("src-prefix", av, &optarg))) {
diff --git a/environment.c b/environment.c
index 2a6de2330b..0d48d52fba 100644
--- a/environment.c
+++ b/environment.c
@@ -22,6 +22,7 @@ int trust_ctime = 1;
int check_stat = 1;
int has_symlinks = 1;
int minimum_abbrev = 4, default_abbrev = -1;
+int default_abbrev_relative = 0;
int ignore_case;
int assume_unchanged;
int prefer_symlink_refs;
diff --git a/parse-options-cb.c b/parse-options-cb.c
index aa9984f164..2a1ab449bf 100644
--- a/parse-options-cb.c
+++ b/parse-options-cb.c
@@ -16,12 +16,22 @@ int parse_opt_abbrev_cb(const struct option *opt, const char *arg, int unset)
if (!arg) {
v = unset ? 0 : DEFAULT_ABBREV;
} else {
+ const char *origarg = arg;
if (!strcmp(arg, ""))
return opterror(opt, "expects a value", 0);
v = strtol(arg, (char **)&arg, 10);
if (*arg)
return opterror(opt, "expects a numerical value", 0);
- if (v && v < MINIMUM_ABBREV) {
+ if (*origarg == '+' || *origarg == '-') {
+ if (v == 0) {
+ return opterror(opt, "relative abbrev must be non-zero", 0);
+ } else if (abs(v) > GIT_SHA1_HEXSZ) {
+ return opterror(opt, "relative abbrev impossibly out of range", 0);
+ } else {
+ default_abbrev_relative = v;
+ v = -1;
+ }
+ } else if (v && v < MINIMUM_ABBREV) {
v = MINIMUM_ABBREV;
} else if (v > GIT_SHA1_HEXSZ) {
v = GIT_SHA1_HEXSZ;
diff --git a/revision.c b/revision.c
index 2a75fef22d..c858d32da7 100644
--- a/revision.c
+++ b/revision.c
@@ -2047,16 +2047,28 @@ static int handle_revision_opt(struct rev_info *revs, int argc, const char **arg
} else if (!strcmp(arg, "--abbrev")) {
revs->abbrev = DEFAULT_ABBREV;
} else if (skip_prefix(arg, "--abbrev=", &optarg)) {
+ int v;
char *end;
if (!strcmp(optarg, ""))
die("--abbrev expects a value, got '%s'", optarg);
- revs->abbrev = strtoul(optarg, &end, 10);
+ v = strtoul(optarg, &end, 10);
if (*end)
die("--abbrev expects a numerical value, got '%s'", optarg);
- if (revs->abbrev < MINIMUM_ABBREV) {
+ if (*optarg == '+' || *optarg == '-') {
+ if (v == 0) {
+ die("relative abbrev must be non-zero");
+ } else if (abs(v) > hexsz) {
+ die("relative abbrev impossibly out of range");
+ } else {
+ default_abbrev_relative = v;
+ revs->abbrev = DEFAULT_ABBREV;
+ }
+ } else if (v < MINIMUM_ABBREV) {
revs->abbrev = MINIMUM_ABBREV;
- } else if (revs->abbrev > hexsz) {
+ } else if (v > hexsz) {
revs->abbrev = hexsz;
+ } else {
+ revs->abbrev = v;
}
} else if (!strcmp(arg, "--abbrev-commit")) {
revs->abbrev_commit = 1;
diff --git a/sha1-name.c b/sha1-name.c
index 60d9ef3c7e..75f1bef7d1 100644
--- a/sha1-name.c
+++ b/sha1-name.c
@@ -602,6 +602,17 @@ int find_unique_abbrev_r(char *hex, const struct object_id *oid, int len)
if (len == GIT_SHA1_HEXSZ || !len)
return GIT_SHA1_HEXSZ;
+ if (default_abbrev_relative) {
+ int dar = default_abbrev_relative;
+ if (len + dar > GIT_SHA1_HEXSZ) {
+ return GIT_SHA1_HEXSZ;
+ } else if (len + dar < MINIMUM_ABBREV) {
+ len = MINIMUM_ABBREV;
+ dar = 0;
+ }
+ len += dar;
+ }
+
mad.init_len = len;
mad.cur_len = len;
mad.hex = hex;
diff --git a/t/t0014-abbrev.sh b/t/t0014-abbrev.sh
index 8448f78560..73d990ecc1 100755
--- a/t/t0014-abbrev.sh
+++ b/t/t0014-abbrev.sh
@@ -68,23 +68,33 @@ test_expect_success 'abbrev non-integer value handling differs ' '
test_i18ngrep "expects a numerical value" stderr
'
-for i in -41 -20 -10 -1 -0 +0 0 1 2 3 41
+for i in -41 +41
do
test_expect_success "core.abbrev value $i out of range errors out" "
test_must_fail git -c core.abbrev=$i log -1 --pretty=format:%h 2>stderr &&
- test_i18ngrep 'abbrev length out of range' stderr
+ test_i18ngrep 'impossibly out of range' stderr
"
done
-for i in -41 -20 -10 -1
+for i in -39 -20 -10 -3
do
- test_expect_success "negative --abbrev=$i value out of range means --abbrev=40" "
+ test_expect_success "negative -31..-3 --abbrev=$i mean --abbrev=4" "
git log --abbrev=$i -1 --pretty=format:%h >log &&
- test_byte_count = 40 log
+ test_byte_count = 4 log
+ "
+done
+
+for i in -9001 -41 +41 +9001
+do
+ test_expect_success "core.abbrev=$i and --abbrev=$i values out of range error out" "
+ test_must_fail git -c core.abbrev=$i branch -v 2>stderr &&
+ test_i18ngrep 'impossibly out of range' stderr &&
+ test_must_fail git branch -v --abbrev=$i 2>stderr &&
+ test_i18ngrep 'impossibly out of range' stderr
"
done
-for i in 0 1 2 3 4 -0 +0 +1 +2 +3 +4
+for i in 0 1 2 3 4
do
test_expect_success "non-negative --abbrev=$i value <MINIMUM_ABBREV falls back on MINIMUM_ABBREV" "
git log --abbrev=$i -1 --pretty=format:%h >log &&
@@ -92,7 +102,7 @@ do
"
done
-for i in 41 9001 +41 +9001
+for i in 41 9001
do
test_expect_success "non-negative --abbrev=$i value >MINIMUM_ABBREV falls back on 40" "
git log --abbrev=$i -1 --pretty=format:%h >log &&
@@ -110,9 +120,23 @@ do
git log --abbrev=$i -1 --pretty=format:%h >log &&
test_byte_count = $i log &&
- # core.abbrev=+N is the same as core.abbrev=N
+ # core.abbrev=+N is the same as core.abbrev=7+N
git -c core.abbrev=+$i log -1 --pretty=format:%h >log &&
- test_byte_count = $i log &&
+ if test \$((7 + $i)) -gt 40
+ then
+ test_byte_count = 40 log
+ else
+ test_byte_count = \$((7 + $i)) log
+ fi &&
+
+ # --abbrev=+N is the same as --abbrev=7+N
+ git log --abbrev=+$i -1 --pretty=format:%h >log &&
+ if test \$((7 + $i)) -gt 40
+ then
+ test_byte_count = 40 log
+ else
+ test_byte_count = \$((7 + $i)) log
+ fi &&
# The --abbrev option should take priority over
# core.abbrev
@@ -171,14 +195,17 @@ do
done
test_expect_success 'blame core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 blame A.t | cut_tr_d_n_field_n 1 >blame &&
- test_must_fail git -c core.abbrev=-1 blame A.t | cut_tr_d_n_field_n 1 >blame &&
+ git -c core.abbrev=+1 blame A.t | cut_tr_d_n_field_n 1 >blame &&
+ test_byte_count = 9 blame &&
- git blame --abbrev=-1 A.t | cut_tr_d_n_field_n 1 >blame &&
- test_byte_count = 5 blame &&
+ git -c core.abbrev=-1 blame A.t | cut_tr_d_n_field_n 1 >blame &&
+ test_byte_count = 7 blame &&
git blame --abbrev=+1 A.t | cut_tr_d_n_field_n 1 >blame &&
- test_byte_count = 5 blame
+ test_byte_count = 9 blame &&
+
+ git blame --abbrev=-1 A.t | cut_tr_d_n_field_n 1 >blame &&
+ test_byte_count = 7 blame
'
for i in $(test_seq 4 40)
@@ -193,14 +220,17 @@ do
done
test_expect_success 'branch core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 branch -v | cut_tr_d_n_field_n 3 >branch &&
- test_must_fail git -c core.abbrev=-1 branch -v | cut_tr_d_n_field_n 3 >branch &&
+ git -c core.abbrev=+1 branch -v | cut_tr_d_n_field_n 3 >branch &&
+ test_byte_count = 8 branch &&
- git branch --abbrev=-1 -v | cut_tr_d_n_field_n 3 >branch &&
- test_byte_count = 4 branch &&
+ git -c core.abbrev=-1 branch -v | cut_tr_d_n_field_n 3 >branch &&
+ test_byte_count = 6 branch &&
- git branch --abbrev=+1 -v | cut_tr_d_n_field_n 3 >branch &&
- test_byte_count = 4 branch
+ git branch -v --abbrev=+1 | cut_tr_d_n_field_n 3 >branch &&
+ test_byte_count = 8 branch &&
+
+ git branch -v --abbrev=-1 | cut_tr_d_n_field_n 3 >branch &&
+ test_byte_count = 6 branch
'
test_expect_success 'describe core.abbrev and --abbrev special cases' '
@@ -225,14 +255,17 @@ do
done
test_expect_success 'describe core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 describe | sed_g_tr_d_n >describe &&
- test_must_fail git -c core.abbrev=-1 describe | sed_g_tr_d_n >describe &&
+ git -c core.abbrev=-1 describe | sed_g_tr_d_n >describe &&
+ test_byte_count = 6 describe &&
+
+ git -c core.abbrev=+1 describe | sed_g_tr_d_n >describe &&
+ test_byte_count = 8 describe &&
git describe --abbrev=-1 | sed_g_tr_d_n >describe &&
- test_byte_count = 4 describe &&
+ test_byte_count = 6 describe &&
git describe --abbrev=+1 | sed_g_tr_d_n >describe &&
- test_byte_count = 4 describe
+ test_byte_count = 8 describe
'
for i in $(test_seq 4 40)
@@ -246,17 +279,17 @@ do
done
test_expect_success 'log core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 log --pretty=format:%h -1 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ git -c core.abbrev=+1 log --pretty=format:%h -1 >log &&
+ test_byte_count = 8 log &&
- test_must_fail git -c core.abbrev=-1 log --pretty=format:%h -1 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ git -c core.abbrev=-1 log --pretty=format:%h -1 >log &&
+ test_byte_count = 6 log &&
git log --abbrev=+1 --pretty=format:%h -1 | tr_d_n >log &&
- test_byte_count = 4 log &&
+ test_byte_count = 8 log &&
git log --abbrev=-1 --pretty=format:%h -1 | tr_d_n >log &&
- test_byte_count = 40 log
+ test_byte_count = 6 log
'
for i in $(test_seq 4 40)
@@ -291,43 +324,55 @@ do
done
test_expect_success 'diff --no-index --raw core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 diff --no-index --raw X Y 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ test_must_fail git -c core.abbrev=+1 diff --no-index --raw X Y >diff &&
+ cut_tr_d_n_field_n 3 <diff >diff.3 &&
+ test_byte_count = 8 diff.3 &&
+ cut_tr_d_n_field_n 4 <diff >diff.4 &&
+ test_byte_count = 8 diff.4 &&
- test_must_fail git -c core.abbrev=-1 diff --no-index --raw X Y 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ test_must_fail git -c core.abbrev=-1 diff --no-index --raw X Y >diff &&
+ cut_tr_d_n_field_n 3 <diff >diff.3 &&
+ test_byte_count = 6 diff.3 &&
+ cut_tr_d_n_field_n 4 <diff >diff.4 &&
+ test_byte_count = 6 diff.4 &&
test_must_fail git diff --no-index --raw --abbrev=+1 X Y >diff &&
cut_tr_d_n_field_n 3 <diff >diff.3 &&
- test_byte_count = 4 diff.3 &&
+ test_byte_count = 8 diff.3 &&
cut_tr_d_n_field_n 4 <diff >diff.4 &&
- test_byte_count = 4 diff.4 &&
+ test_byte_count = 8 diff.4 &&
test_must_fail git diff --no-index --raw --abbrev=-1 X Y >diff &&
cut_tr_d_n_field_n 3 <diff >diff.3 &&
- test_byte_count = 4 diff.3 &&
+ test_byte_count = 6 diff.3 &&
cut_tr_d_n_field_n 4 <diff >diff.4 &&
- test_byte_count = 4 diff.4
+ test_byte_count = 6 diff.4
'
test_expect_success 'diff --raw core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 diff HEAD~ 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ git -c core.abbrev=+1 diff --raw HEAD~ >diff &&
+ cut_tr_d_n_field_n 3 <diff >diff.3 &&
+ test_byte_count = 8 diff.3 &&
+ cut_tr_d_n_field_n 4 <diff >diff.4 &&
+ test_byte_count = 8 diff.4 &&
- test_must_fail git -c core.abbrev=-1 diff HEAD~ 2>stderr &&
- test_i18ngrep "abbrev length out of range" stderr &&
+ git -c core.abbrev=-1 diff --raw HEAD~ >diff &&
+ cut_tr_d_n_field_n 3 <diff >diff.3 &&
+ test_byte_count = 6 diff.3 &&
+ cut_tr_d_n_field_n 4 <diff >diff.4 &&
+ test_byte_count = 6 diff.4 &&
- git diff --raw --abbrev=+1 HEAD~ >diff &&
+ git diff --raw --abbrev=+1 --raw HEAD~ >diff &&
cut_tr_d_n_field_n 3 <diff >diff.3 &&
- test_byte_count = 4 diff.3 &&
+ test_byte_count = 8 diff.3 &&
cut_tr_d_n_field_n 4 <diff >diff.4 &&
- test_byte_count = 4 diff.4 &&
+ test_byte_count = 8 diff.4 &&
- git diff --raw --abbrev=-1 HEAD~ >diff &&
+ git diff --raw --abbrev=-1 --raw HEAD~ >diff &&
cut_tr_d_n_field_n 3 <diff >diff.3 &&
- test_byte_count = 40 diff.3 &&
+ test_byte_count = 6 diff.3 &&
cut_tr_d_n_field_n 4 <diff >diff.4 &&
- test_byte_count = 40 diff.4
+ test_byte_count = 6 diff.4
'
for i in $(test_seq 4 40)
@@ -342,13 +387,16 @@ done
test_expect_success 'ls-files core.abbrev=[-+]1 and --abbrev=[-+]1' '
test_must_fail git -c core.abbrev=+1 ls-files --stage A.t | cut_tr_d_n_field_n 2 >ls-files &&
+ test_byte_count = 40 ls-files &&
+
test_must_fail git -c core.abbrev=-1 ls-files --stage A.t | cut_tr_d_n_field_n 2 >ls-files &&
+ test_byte_count = 40 ls-files &&
git ls-files --abbrev=-1 --stage A.t | cut_tr_d_n_field_n 2 >ls-files &&
- test_byte_count = 4 ls-files &&
+ test_byte_count = 6 ls-files &&
git ls-files --abbrev=+1 --stage A.t | cut_tr_d_n_field_n 2 >ls-files &&
- test_byte_count = 4 ls-files
+ test_byte_count = 8 ls-files
'
for i in $(test_seq 4 40)
@@ -362,14 +410,17 @@ do
done
test_expect_success 'ls-tree core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 ls-tree HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
- test_must_fail git -c core.abbrev=-1 ls-tree HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
+ git -c core.abbrev=+1 ls-tree HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
+ test_byte_count = 40 ls-tree &&
+
+ git -c core.abbrev=-1 ls-tree HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
+ test_byte_count = 40 ls-tree &&
git ls-tree --abbrev=-1 HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
- test_byte_count = 4 ls-tree &&
+ test_byte_count = 6 ls-tree &&
git ls-tree --abbrev=+1 HEAD A.t | cut -f 1 | cut_tr_d_n_field_n 3 >ls-tree &&
- test_byte_count = 4 ls-tree
+ test_byte_count = 8 ls-tree
'
for i in $(test_seq 4 40)
@@ -385,14 +436,17 @@ do
done
test_expect_success 'show-ref core.abbrev=[-+]1 and --abbrev=[-+]1' '
- test_must_fail git -c core.abbrev=+1 show-ref --hash refs/heads/master | tr_d_n >show-ref &&
- test_must_fail git -c core.abbrev=-1 show-ref --hash refs/heads/master | tr_d_n >show-ref &&
+ git -c core.abbrev=+1 show-ref --hash refs/heads/master | tr_d_n >show-ref &&
+ test_byte_count = 40 show-ref &&
+
+ git -c core.abbrev=-1 show-ref --hash refs/heads/master | tr_d_n >show-ref &&
+ test_byte_count = 40 show-ref &&
git show-ref --abbrev=-1 --hash refs/heads/master | tr_d_n >show-ref &&
- test_byte_count = 4 show-ref &&
+ test_byte_count = 6 show-ref &&
git show-ref --abbrev=+1 --hash refs/heads/master | tr_d_n >show-ref &&
- test_byte_count = 4 show-ref
+ test_byte_count = 8 show-ref
'
test_done
--
2.17.0.290.gded63e768a
next prev parent reply other threads:[~2018-06-08 22:42 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-08 22:41 [PATCH 00/20] unconditional O(1) SHA-1 abbreviation Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 01/20] t/README: clarify the description of test_line_count Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 02/20] test library: add a test_byte_count Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 03/20] blame doc: explicitly note how --abbrev=40 gives 39 chars Ævar Arnfjörð Bjarmason
2018-06-12 18:10 ` Junio C Hamano
2018-06-08 22:41 ` [PATCH 04/20] abbrev tests: add tests for core.abbrev and --abbrev Ævar Arnfjörð Bjarmason
2018-06-12 18:31 ` Junio C Hamano
2018-06-08 22:41 ` [PATCH 05/20] abbrev tests: test "git-blame" behavior Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 06/20] blame: fix a bug, core.abbrev should work like --abbrev Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 07/20] abbrev tests: test "git branch" behavior Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 08/20] abbrev tests: test for "git-describe" behavior Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 09/20] abbrev tests: test for "git-log" behavior Ævar Arnfjörð Bjarmason
2018-06-09 8:43 ` Martin Ågren
2018-06-09 9:56 ` Ævar Arnfjörð Bjarmason
2018-06-09 13:56 ` Martin Ågren
2018-06-08 22:41 ` [PATCH 10/20] abbrev tests: test for "git-diff" behavior Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 11/20] abbrev tests: test for plumbing behavior Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 12/20] abbrev tests: test for --abbrev and core.abbrev=[+-]N Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 13/20] parse-options-cb.c: convert uses of 40 to GIT_SHA1_HEXSZ Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 14/20] config.c: use braces on multiple conditional arms Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 15/20] parse-options-cb.c: " Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 16/20] abbrev: unify the handling of non-numeric values Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 17/20] abbrev: unify the handling of empty values Ævar Arnfjörð Bjarmason
2018-06-09 14:24 ` Martin Ågren
2018-06-09 14:31 ` Martin Ågren
2018-06-08 22:41 ` [PATCH 18/20] abbrev parsing: use braces on multiple conditional arms Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` Ævar Arnfjörð Bjarmason [this message]
2018-06-09 15:38 ` [PATCH 19/20] abbrev: support relative abbrev values Martin Ågren
2018-06-12 19:16 ` Junio C Hamano
2018-06-13 22:22 ` Ævar Arnfjörð Bjarmason
2018-06-13 22:34 ` Junio C Hamano
2018-06-14 7:36 ` Ævar Arnfjörð Bjarmason
2018-06-14 15:50 ` Junio C Hamano
2018-06-14 19:07 ` Ævar Arnfjörð Bjarmason
2018-06-08 22:41 ` [PATCH 20/20] abbrev: add a core.validateAbbrev setting Ævar Arnfjörð Bjarmason
2018-06-09 15:47 ` Martin Ågren
2018-06-12 18:58 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180608224136.20220-20-avarab@gmail.com \
--to=avarab@gmail.com \
--cc=dstolee@microsoft.com \
--cc=git@jeffhostetler.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=johannes.schindelin@gmx.de \
--cc=jrnieder@gmail.com \
--cc=peff@peff.net \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).