[Asterisk-code-review] runtests: Add an option (-k|--keep-core) to keep a core file (testsuite[master])

Corey Farrell asteriskteam at digium.com
Sun Oct 4 20:48:34 CDT 2015


Corey Farrell has posted comments on this change.

Change subject: runtests: Add an option (-k|--keep-core) to keep a core file
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/1373/2/runtests.py
File runtests.py:

Line 199:                     if not self.options.keep_core:
I think the core file needs to be moved to the archive path for the test run if it's not deleted.  When running multiple tests the core file would be detected for every test after the first crash.


-- 
To view, visit https://gerrit.asterisk.org/1373
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5a468e98811b1ff814fdb64aa6808eba3c17460b
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list