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

Re: "./t0001-init.sh --valgrind" is broken

From
Duy Nguyen <pclouds@gmail.com>
Date
Mar 3, 2016, 12:16 UTC
Message-ID
<CACsJy8AE9VovwuviwOLxRDTAbTXCivRVhk8ia4mdUnMN-0Y4OA@mail.gmail.com>
In-Reply-To
<CACsJy8CVfJdCmwWo50wHsc6DP6ft71d5bfBaFSJpv9XcNRwdSQ@mail.gmail.com>
On Thu, Mar 3, 2016 at 7:09 PM, Duy Nguyen <pclouds@gmail.com> wrote:
> But it's probably better that we inject valgrind command
> from inside bin-wrappers script, the same way we inject gdb, I think.

For the best of both worlds, we should recreate bin-wrappers in test-lib.sh (i.e. the valgrind way), not in Makefile. Somewhat unrelated, but because topdir is getting really crowded and bin-wrappers is used for the test suite only, it should be moved inside t/ (i'm going to move all test-* to t/ too, later).

-- 
Duy
Previous: Duy NguyenNext: Junio C Hamano
Message 6 of 9 in “"./t0001-init.sh --valgrind" is broken”
  1. Christian CouderMar 3, 2016
  2. Duy NguyenMar 3, 2016
  3. Junio C HamanoMar 3, 2016
  4. Johannes SixtMar 3, 2016
  5. Duy NguyenMar 3, 2016
  6. Duy NguyenMar 3, 2016
  7. Junio C HamanoMar 3, 2016
  8. Jeff KingMar 3, 2016
  9. Johannes SixtMar 3, 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.