Skip to content

Commit 4b0891f

Browse files
committed
More topics for 2.8.4
Signed-off-by: Junio C Hamano <[email protected]>
1 parent 3296e1a commit 4b0891f

File tree

1 file changed

+28
-0
lines changed

1 file changed

+28
-0
lines changed

Documentation/RelNotes/2.8.4.txt

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,32 @@ Fixes since v2.8.3
2424
* Many 'linkgit:<git documentation page>' references were broken,
2525
which are all fixed with this.
2626

27+
* "git describe --contains" often made a hard-to-justify choice of
28+
tag to give name to a given commit, because it tried to come up
29+
with a name with smallest number of hops from a tag, causing an old
30+
commit whose close descendant that is recently tagged were not
31+
described with respect to an old tag but with a newer tag. It did
32+
not help that its computation of "hop" count was further tweaked to
33+
penalize being on a side branch of a merge. The logic has been
34+
updated to favor using the tag with the oldest tagger date, which
35+
is a lot easier to explain to the end users: "We describe a commit
36+
in terms of the (chronologically) oldest tag that contains the
37+
commit."
38+
39+
* Running tests with '-x' option to trace the individual command
40+
executions is a useful way to debug test scripts, but some tests
41+
that capture the standard error stream and check what the command
42+
said can be broken with the trace output mixed in. When running
43+
our tests under "bash", however, we can redirect the trace output
44+
to another file descriptor to keep the standard error of programs
45+
being tested intact.
46+
47+
* "http.cookieFile" configuration variable clearly wants a pathname,
48+
but we forgot to treat it as such by e.g. applying tilde expansion.
49+
50+
* When de-initialising all submodules, "git submodule deinit" gave a
51+
faulty recommendation to use "git submodule deinit .", which would
52+
result in a strange error message in a pathological corner case.
53+
This has been corrected to suggest "submodule deinit --all" instead.
54+
2755
Also contains other minor documentation updates and code clean-ups.

0 commit comments

Comments
 (0)