Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-27 Thread wdoekes
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/#review13187 --- Ship it! I think the output is far from friendly right now,

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-26 Thread Tzafrir Cohen
On Aug. 25, 2014, 5:53 p.m., Matt Jordan wrote: How is this different than the -l option? George Joseph wrote: Just to chip in... The -l option lists everything even though the test might not actually get run because of some constraint. I use something like the proposed dry-run

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-26 Thread Tzafrir Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/ --- (Updated Aug. 26, 2014, 4:50 p.m.) Review request for Asterisk

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-25 Thread wdoekes
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/#review13162 --- /asterisk/trunk/runtests.py

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-25 Thread wdoekes
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/#review13163 --- /asterisk/trunk/runtests.py

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-25 Thread Matt Jordan
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/#review13170 --- How is this different than the -l option? - Matt Jordan On

Re: [asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-25 Thread George Joseph
On Aug. 25, 2014, 11:53 a.m., Matt Jordan wrote: How is this different than the -l option? Just to chip in... The -l option lists everything even though the test might not actually get run because of some constraint. I use something like the proposed dry-run so I can know how many tests

[asterisk-dev] [Code Review] 3931: runtests.py: -d (--dry-run)

2014-08-24 Thread Tzafrir Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/ --- Review request for Asterisk Developers. Repository: testsuite