<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/t, branch v2.43.3</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.43.3</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.43.3'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2024-02-23T00:09:45Z</updated>
<entry>
<title>Merge branch 'la/trailer-cleanups' into maint-2.43</title>
<updated>2024-02-23T00:09:45Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-23T00:09:45Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=5dc73662976e81de19a5b3a76f4733962f541df5'/>
<id>urn:sha1:5dc73662976e81de19a5b3a76f4733962f541df5</id>
<content type='text'>
* la/trailer-cleanups:
  trailer: fix comment/cut-line regression with opts-&gt;no_divider
</content>
</entry>
<entry>
<title>trailer: fix comment/cut-line regression with opts-&gt;no_divider</title>
<updated>2024-02-20T03:06:18Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2024-02-20T01:09:36Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bc47139f4ff6c0c41d4c854ca74c35b1006a464a'/>
<id>urn:sha1:bc47139f4ff6c0c41d4c854ca74c35b1006a464a</id>
<content type='text'>
Commit 97e9d0b78a (trailer: find the end of the log message, 2023-10-20)
combined two code paths for finding the end of the log message. For the
"no_divider" case, we used to use find_trailer_end(), and that has now
been rolled into find_end_of_log_message(). But there's a regression;
that function returns early when no_divider is set, returning the whole
string.

That's not how find_trailer_end() behaved. Although it did skip the
"---" processing (which is what "no_divider" is meant to do), we should
still respect ignored_log_message_bytes(), which covers things like
comments, "commit -v" cut lines, and so on.

The bug is actually in the interpret-trailers command, but the obvious
way to experience it is by running "commit -v" with a "--trailer"
option. The new trailer will be added at the end of the verbose diff,
rather than before it (and consequently will be ignored entirely, since
everything after the diff's intro scissors line is thrown away).

I've added two tests here: one for interpret-trailers directly, which
shows the bug via the parsing routines, and one for "commit -v".

The fix itself is pretty simple: instead of returning early, no_divider
just skips the "---" handling but still calls ignored_log_message_bytes().

Reported-by: Philippe Blain &lt;levraiphilippeblain@gmail.com&gt;
Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'ps/report-failure-from-git-stash' into maint-2.43</title>
<updated>2024-02-13T22:44:49Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-13T22:44:49Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bd10c45672068811cb13cb0edba78470f537f233'/>
<id>urn:sha1:bd10c45672068811cb13cb0edba78470f537f233</id>
<content type='text'>
"git stash" sometimes was silent even when it failed due to
unwritable index file, which has been corrected.

* ps/report-failure-from-git-stash:
  builtin/stash: report failure to write to index
</content>
</entry>
<entry>
<title>Merge branch 'jk/diff-external-with-no-index' into maint-2.43</title>
<updated>2024-02-13T22:44:48Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-13T22:44:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=5071cb78a31b75e007b36f7edb4a4daa59cf3138'/>
<id>urn:sha1:5071cb78a31b75e007b36f7edb4a4daa59cf3138</id>
<content type='text'>
"git diff --no-index file1 file2" segfaulted while invoking the
external diff driver, which has been corrected.

* jk/diff-external-with-no-index:
  diff: handle NULL meta-info when spawning external diff
</content>
</entry>
<entry>
<title>Merge branch 'rs/parse-options-with-keep-unknown-abbrev-fix' into maint-2.43</title>
<updated>2024-02-13T22:44:48Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-13T22:44:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d982de5d32582bfebf30386468d43290efa258c5'/>
<id>urn:sha1:d982de5d32582bfebf30386468d43290efa258c5</id>
<content type='text'>
"git diff --no-rename A B" did not disable rename detection but did
not trigger an error from the command line parser.

* rs/parse-options-with-keep-unknown-abbrev-fix:
  parse-options: simplify positivation handling
  parse-options: fully disable option abbreviation with PARSE_OPT_KEEP_UNKNOWN
</content>
</entry>
<entry>
<title>Merge branch 'en/diffcore-delta-final-line-fix' into maint-2.43</title>
<updated>2024-02-13T22:44:48Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-13T22:44:47Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=904ca69428ccf38242ad08f60f516b90561ae1ed'/>
<id>urn:sha1:904ca69428ccf38242ad08f60f516b90561ae1ed</id>
<content type='text'>
Rename detection logic ignored the final line of a file if it is an
incomplete line.

* en/diffcore-delta-final-line-fix:
  diffcore-delta: avoid ignoring final 'line' of file
</content>
</entry>
<entry>
<title>Merge branch 'tc/show-ref-exists-fix' into maint-2.43</title>
<updated>2024-02-13T22:44:47Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-13T22:44:47Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=908fde12b0d59645cdb684ba9862e7665d89695e'/>
<id>urn:sha1:908fde12b0d59645cdb684ba9862e7665d89695e</id>
<content type='text'>
Update to a new feature recently added, "git show-ref --exists".

* tc/show-ref-exists-fix:
  builtin/show-ref: treat directory as non-existing in --exists
</content>
</entry>
<entry>
<title>Merge branch 'jx/sideband-chomp-newline-fix' into maint-2.43</title>
<updated>2024-02-09T00:22:11Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-09T00:22:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=5b49c1af03e600c286f63d9d9c9fb01403230b9f'/>
<id>urn:sha1:5b49c1af03e600c286f63d9d9c9fb01403230b9f</id>
<content type='text'>
Sideband demultiplexer fixes.

* jx/sideband-chomp-newline-fix:
  pkt-line: do not chomp newlines for sideband messages
  pkt-line: memorize sideband fragment in reader
  test-pkt-line: add option parser for unpack-sideband
</content>
</entry>
<entry>
<title>Merge branch 'jk/t1006-cat-file-objectsize-disk' into maint-2.43</title>
<updated>2024-02-09T00:22:11Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-09T00:22:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=fb3ead665bd3371e31cebe716c973c82ff8545eb'/>
<id>urn:sha1:fb3ead665bd3371e31cebe716c973c82ff8545eb</id>
<content type='text'>
Test update.

* jk/t1006-cat-file-objectsize-disk:
  t1006: prefer shell loop to awk for packed object sizes
  t1006: add tests for %(objectsize:disk)
</content>
</entry>
<entry>
<title>Merge branch 'en/header-cleanup' into maint-2.43</title>
<updated>2024-02-09T00:22:10Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2024-02-09T00:22:10Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=0f7a10a3aad344a28f0f30b32a979925a9629533'/>
<id>urn:sha1:0f7a10a3aad344a28f0f30b32a979925a9629533</id>
<content type='text'>
Remove unused header "#include".

* en/header-cleanup:
  treewide: remove unnecessary includes in source files
  treewide: add direct includes currently only pulled in transitively
  trace2/tr2_tls.h: remove unnecessary include
  submodule-config.h: remove unnecessary include
  pkt-line.h: remove unnecessary include
  line-log.h: remove unnecessary include
  http.h: remove unnecessary include
  fsmonitor--daemon.h: remove unnecessary includes
  blame.h: remove unnecessary includes
  archive.h: remove unnecessary include
  treewide: remove unnecessary includes in source files
  treewide: remove unnecessary includes from header files
</content>
</entry>
</feed>
