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

Re: [PATCH] use "git init-db" in tests

From
Junio C Hamano <junkio@cox.net>
Date
Dec 9, 2005, 20:46 UTC
Message-ID
<7vmzja2e2s.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<81b0412b0512090259l5f06f6d6n5416e271af36d6a1@mail.gmail.com>
Alex Riesen <raa.lkml@gmail.com> writes:
> In my case it was the freshly build directory where a chmod 0666 * was
> done. This directory wont rebuild (the dates are correct), and the
> tests run, as if nothing happened.

I see. I appreciate your honesty ;-), it is a pilot error alright, but surely it would be nicer if we catch it.

Although your proposed change makes it harder to implement my desire to be able to run tests with installed binaries than it already is, I'd take the patch for now. It would catch this particular pilot error.

I however would think adding a dependency in t/Makefile to make sure the top level is built before starting to run test a overkill (I think I've seen some projects to do that), so I'd not go there.

Previous: Alex RiesenNext: Alex Riesen
Message 10 of 11 in “use "git init-db" in tests”
  1. use "git init-db" in testsAlex Riesen, Dec 8, 2005
  2. Junio C HamanoDec 8, 2005
  3. Alex RiesenDec 8, 2005
  4. Junio C HamanoDec 8, 2005
  5. Alex RiesenDec 9, 2005
  6. Junio C HamanoDec 9, 2005
  7. Junio C HamanoDec 9, 2005
  8. Petr BaudisDec 9, 2005
  9. Alex RiesenDec 9, 2005
  10. Junio C HamanoDec 9, 2005
  11. Alex RiesenDec 8, 2005

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.