threads / patch / 17542

patch, 8 partsvalgrind series

Subject: [PATCH 0/8] valgrind series

## tl;dr

One message between Feb 3, 2009 and Feb 3, 2009. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Johannes Schindelin· Feb 3, 2009, 23:25 UTC · lore
Yes, it has grown a little bit.
The main differences since last time:
- split out some parts that Peff thought were independent of valgrind
- added some convenience, such as "--tee" to output not only to the 
  terminal, but also to files named after the tests
- fixed half a bug in test-lib.sh (the templates were assumed to be 
  located relative to the binaries, not relative to the TEST_DIRECTORY)
- found the cause (sort of) of the spurious zlib "error".  Mark promised 
  that next zlib will contain a fix for that.
- tested.  A lot.
Note: I am not quite sure if the order is okay, but I really spent quite 
some time to reorder and resplit all the patches.
Jeff King (1):
  valgrind: ignore ldso and more libz errors
Johannes Schindelin (7):
  Add valgrind support in test scripts
  Valgrind support: check for more than just programming errors
  test-lib.sh: optionally output to test-results/$TEST.out, too
  t/Makefile: provide a 'valgrind' target
  Add a script to coalesce the valgrind outputs
  Tests: let --valgrind imply --verbose and --tee
  test-lib: avoid assuming that templates/ are in the GIT_EXEC_PATH
 t/Makefile              |    5 ++-
 t/README                |   18 +++++++-
 t/test-lib.sh           |   91 +++++++++++++++++++++++++++++++++-
 t/valgrind/.gitignore   |    2 +
 t/valgrind/analyze.sh   |  123 +++++++++++++++++++++++++++++++++++++++++++++++
 t/valgrind/default.supp |   43 ++++++++++++++++
 t/valgrind/valgrind.sh  |   13 +++++
 7 files changed, 290 insertions(+), 5 deletions(-)
 create mode 100644 t/valgrind/.gitignore
 create mode 100755 t/valgrind/analyze.sh
 create mode 100644 t/valgrind/default.supp
 create mode 100755 t/valgrind/valgrind.sh

← back to recent threads