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

Re: [PATCH 1/2] t/perf: fix regression in testing older versions of git

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jun 22, 2016, 20:23 UTC
Message-ID
<alpine.DEB.2.20.1606222223100.10382@virtualbox>
In-Reply-To
<20160622194013.GA1697@sigill.intra.peff.net>
Hi Peff,
On Wed, 22 Jun 2016, Jeff King wrote:
Show 28 quoted lines
> Commit 7501b59 (perf: make the tests work in worktrees,
> 2016-05-13) introduced the use of "git rev-parse --git-path"
> in the perf-lib setup code. Because the to-be-tested version
> of git is at the front of the $PATH when this code runs,
> this means we cannot use modern versions of t/perf to test
> versions of git older than v2.5.0 (when that option was
> introduced).
> 
> This is a symptom of a more general problem. The t/perf
> suite is essentially independent of git versions, and
> ideally we would be able to run the most modern and complete
> set of tests across many historical versions (to see how
> they compare). But any setup code they run is therefore
> required to use the lowest common denominator we expect to
> test.
> 
> So let's introduce a new variable, $MODERN_GIT, that we can
> use both in perf-lib and in the test setup to get a reliable
> set of git features (we might change git and break some
> tests, of course, but $MODERN_GIT is tied to the same
> version of git as the t/perf scripts, so they can be fixed
> or adjusted together).
> 
> This commit fixes the "--git-path" case, but does not
> mass-convert existing setup code to use $MODERN_GIT. Most
> setup code is fairly vanilla and will work with effectively
> all versions. But now the tool is there to fix any other
> issues we find going forward.
Thanks for beating me to it!

Ciao, Dscho

Previous: Jeff KingNext: Johannes Schindelin
Message 3 of 8 in “t/perf tests against older versions”
  1. 0/2 t/perf tests against older versionsJeff King, Jun 22, 2016
  2. 1/2 t/perf: fix regression in testing older versions of gitJeff King, Jun 22, 2016
  3. Johannes SchindelinJun 22, 2016
  4. Johannes SchindelinJun 22, 2016
  5. Junio C HamanoJun 22, 2016
  6. Jeff KingJun 22, 2016
  7. Junio C HamanoJun 22, 2016
  8. 2/2 p4211: explicitly disable renames in no-rename testJeff King, Jun 22, 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.