<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 23, 2013 at 9:42 AM, Scott Griepentrog <span dir="ltr"><<a href="mailto:sgriepentrog@digium.com" target="_blank">sgriepentrog@digium.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="color:#660000">I was worried that might be a problem.  What about using an environment variable as the trigger instead?  That would allow it to be easily inherited by any language, and easily ignored if not implemented.</div>

<div style="color:#660000"><br></div><div style="color:#660000">You could even set it externally to runtests.py at first for testing or automation:</div><div style="color:#660000">
<br></div><div style="color:#660000">export RUNTEST_VALGRIND='YES'</div><div style="color:#660000">./runtests.py</div><div style="color:#660000"><br></div></div></blockquote><div><br></div><div style>The Test Suite typically uses the YAML files to determine how it runs. If you want valgrind to be applied globally, there is a top level "global" configuration file that runtests.py also parses (also test-config.yaml). I'd prefer that valgrind can, at a minimum, be enabled using said configuration files.</div>
<div style><br></div><div style>That being said, the environment variable approach is an acceptable alternative. The Asterisk library could look for the variable and spawn Asterisk with valgrind if present.</div></div><br clear="all">
<div style>Matt</div><div style><br></div>-- <br><div dir="ltr"><div>Matthew Jordan<br></div><div>Digium, Inc. | Engineering Manager</div><div>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA</div><div>Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a></div>
</div>
</div></div>