[asterisk-dev] Change in testsuite[master]: Memory Debugging Improvements

[Corey Farrell (Code Review)]

Hello Ashley Sanders,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/15

to look at the new patch set (#4).

Change subject: Memory Debugging Improvements
......................................................................

Memory Debugging Improvements

* Enable XML output from valgrind.
* Display and save a summary of valgrind errors and leaks.
* Enable use of contrib/valgrind/suppressions.txt if it exists
  to suppress expected leaks or system library errors.  Added
  entry to .gitignore for this file.
* Supply a sample suppressions file that ignores some reachable
  memory.
* Create stdout_print for printing messages to the terminal and
  including in the failure message.
* Switch some failure notifications to use stdout_print() so the
  messages are included in asterisk-test-suite-report.xml.
* Create function for archiving a list of files from source folder
  to destination folder.  Switch all archive functions to use this.

Valgrind Summaries require the lxml module.  If this module is not
found summaries will not be produced, but valgrind XML output will
still be available in the Asterisk logs directory.

Change-Id: I21634673508a01eea1f489c751d3cf75ea55cf06
---
M .gitignore
M README.txt
A contrib/valgrind/summary-lines.xsl
A contrib/valgrind/suppressions-sample.txt
M lib/python/asterisk/asterisk.py
M runtests.py
6 files changed, 192 insertions(+), 47 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/15/15/4
-- 
To view, visit https://gerrit.asterisk.org/15
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I21634673508a01eea1f489c751d3cf75ea55cf06
Gerrit-PatchSet: 4
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>