<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6676">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Add .valgrindrc to .gitignore.<br><br>Tell git to ignore .valgrindrc and add comment to asterisk.py telling<br>valgrind users to add options to that config instead of modifying python<br>sources.<br><br>Change-Id: I8efff2f5d2c09dce91572653b7de04be966a1921<br>---<br>M .gitignore<br>M lib/python/asterisk/asterisk.py<br>2 files changed, 3 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/.gitignore b/.gitignore<br>index 6a73616..137bce5 100644<br>--- a/.gitignore<br>+++ b/.gitignore<br>@@ -7,3 +7,4 @@<br> /tests/custom<br> /fastagi<br> /run-local.conf<br>+/.valgrindrc<br>diff --git a/lib/python/asterisk/asterisk.py b/lib/python/asterisk/asterisk.py<br>index dc575b4..2302d54 100755<br>--- a/lib/python/asterisk/asterisk.py<br>+++ b/lib/python/asterisk/asterisk.py<br>@@ -512,6 +512,8 @@<br>         if os.getenv("VALGRIND_ENABLE") == "true":<br>             valgrind_path = test_suite_utils.which('valgrind')<br>             if valgrind_path:<br>+                # No need to modify this array to add options.  Instead create<br>+                # .valgrindrc in the testsuite root directory.<br>                 cmd_prefix = [<br>                     valgrind_path,<br>                     '--xml=yes',<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6676">change 6676</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6676"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I8efff2f5d2c09dce91572653b7de04be966a1921 </div>
<div style="display:none"> Gerrit-Change-Number: 6676 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>