On Tue, Jul 30, 2024 at 06:55:14AM +0200, Patrick Steinhardt wrote: > On Mon, Jul 29, 2024 at 01:12:42PM -0700, Junio C Hamano wrote: > > Patrick Steinhardt <ps@xxxxxx> writes: > > > On Fri, Jun 21, 2024 at 04:18:26PM -0700, Junio C Hamano wrote: > > "Skip non-diff garbage until we see a patch" is the mechanism used > > to skip the log message, so it would be a reasonable thing to skip > > such no-diff garbage between patches, no? > > Oh, yes, it is reasonable. I just didn't quite figure out the flow of > the above loop when reading through the code. As you say, it is somewhat > convoluted and not all that straight forward. As far as I can see we didn't have a test for this yet, so I did have a quick go at it to reassure myself that things work as expected before and after your change. Feel free to pick it up if you feel like it, or to just ignore it :) Patrick test_expect_success 'patch-id handles diffs with garbage in between' ' cat >diff-with-garbage <<-\EOF && $(git rev-parse HEAD) diff --git a/bar b/bar index bdaf90f..31051f6 100644 --- a/bar +++ b/bar @@ -2 +2,2 @@ b +c some garbage lines $(git rev-parse HEAD) diff --git a/car b/car index 00750ed..2ae5e34 100644 --- a/car +++ b/car @@ -1 +1,2 @@ 3 +d EOF cat >diff-without-garbage <<-\EOF && $(git rev-parse HEAD) diff --git a/bar b/bar index bdaf90f..31051f6 100644 --- a/bar +++ b/bar @@ -2 +2,2 @@ b +c $(git rev-parse HEAD) diff --git a/car b/car index 00750ed..2ae5e34 100644 --- a/car +++ b/car @@ -1 +1,2 @@ 3 +d EOF for stable in true false do test_config patchid.stable $stable && git patch-id <diff-with-garbage >id-with-garbage && git patch-id <diff-without-garbage >id-without-garbage && test_line_count -eq 2 id-with-garbage && test_cmp id-with-garbage id-without-garbage || return 1 done '
Attachment:
signature.asc
Description: PGP signature
