git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: git 2.10.1 test regression in t4014-format-patch.sh

From
Jeremy Huddleston Sequoia <jeremyhu@freedesktop.org>
Date
Oct 10, 2016, 02:55 UTC
Message-ID
<DEE09909-A5C9-47D8-B71E-0EC0D6C11B3D@freedesktop.org>
In-Reply-To
<F069D0FD-C3CF-4F40-9338-D84E76DB310B@joshtriplett.org>
Show 39 quoted lines
> On Oct 9, 2016, at 17:18, Josh Triplett <josh@joshtriplett.org> wrote:
> 
> On October 9, 2016 5:15:22 PM PDT, Jeremy Huddleston Sequoia <jeremyhu@freedesktop.org> wrote:
>> Hey Josh,
>> 
>> Hope you're doing well.
>> 
>> I wanted to let you know that this patch of yours, which landed in git
>> 2.10.1, introduced some test failures, seen on macOS.
>> 
>> Let me know if you need any additional information to track these down.
>> 
>> Thanks,
>> Jeremy
>> 
>> 
>> not ok 65 - format-patch default signature
>> #	
>> #		git format-patch --stdout -1 | tail -n 3 >output &&
>> #		signature >expect &&
>> #		test_cmp expect output
>> #	
>> 
>> not ok 132 - format-patch --base
>> #	
>> #		git checkout side &&
>> #		git format-patch --stdout --base=HEAD~3 -1 | tail -n 7 >actual &&
>> #		echo >expected &&
>> #		echo "base-commit: $(git rev-parse HEAD~3)" >>expected &&
>> #		echo "prerequisite-patch-id: $(git show --patch HEAD~2 | git
>> patch-id --stable | awk "{print \$1}")" >>expected &&
>> #		echo "prerequisite-patch-id: $(git show --patch HEAD~1 | git
>> patch-id --stable | awk "{print \$1}")" >>expected &&
>> #		signature >> expected &&
>> #		test_cmp expected actual
>> #	
> 
> Can you run the test with the option to show the expected and actual strings?
> Did the testsuite run with the wrong git somehow?
Nope, it's the right version being tested.  The failure seems due to your git_version change not liking our formatting

$ git --version git version 2.10.1 (Apple Git-99)

(the 'Apple Git-XX' being added because this was from a build that had Apple's patch series, including https://github.com/jeremyhu/git/commit/f99905d0752d923e5ec61e14c675a300c6d04284)
We modify DEF_VER, which confused your regex.  Simple patch inc in a separate email.

Thanks, Jeremy

Previous: Josh Triplett
Message 3 of 3 in “git 2.10.1 test regression in t4014-format-patch.sh”
  1. Jeremy Huddleston SequoiaOct 10, 2016
  2. Josh TriplettOct 10, 2016
  3. Jeremy Huddleston SequoiaOct 10, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.