<div dir="auto">Looks fine to me. Chris will have the final say though.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, 12 Jun 2018, 08:00 Vijay Kumar Banerjee, <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, 12 Jun 2018, 12:27 Joel Sherrill, <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Chris needs to speak up since it is Python.</div><div dir="auto"><br></div><div dir="auto">I see changes to HTML output which I doubt are related to the core change.<br></div></div></blockquote></div></div><div dir="auto">There are no real changes to HTML,</div><div dir="auto">I have just added line breaks in the code</div><div dir="auto">because it was too long. </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Tue, Jun 12, 2018, 8:39 AM Vijay Kumar Banerjee <<a href="mailto:vijaykumar9597@gmail.com" rel="noreferrer noreferrer" target="_blank">vijaykumar9597@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">Please review this patch. :)<br><div class="gmail_quote">On 7 June 2018 at 16:38, Vijay Kumar Banerjee <span dir="ltr"><<a href="mailto:vijaykumar9597@gmail.com" rel="noreferrer noreferrer noreferrer" target="_blank">vijaykumar9597@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Close #3440<br>
---<br>
tester/rt/coverage.py | 6 ++++--<br>
tester/rt/test.py | 15 ++++++++++-----<br>
2 files changed, 14 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/tester/rt/coverage.py b/tester/rt/coverage.py<br>
index 54933d5..af24124 100644<br>
--- a/tester/rt/coverage.py<br>
+++ b/tester/rt/coverage.py<br>
@@ -163,7 +163,8 @@ class report_gen_html:<br>
row += " <td>" + summary.branches_uncovered + "</td>"<br>
row += " <td>" + summary.branches_total + "</td>"<br>
row += " <td> {:.3%} </td>".format(summary.percentage_branches_covered)<br>
- row += ' <td><progress value="{:.3}" max="100"></progress></td>'.format(100*summary.percentage_branches_covered)<br>
+ row += ' <td><progress value="{:.3}" max="100"></progress></td>'\<br>
+ .format(100*summary.percentage_branches_covered)<br>
row += "</tr>\n"<br>
return row<br>
<br>
@@ -299,7 +300,8 @@ class covoar(object):<br>
if (not path.exists(covoar_result_dir)):<br>
path.mkdir(covoar_result_dir)<br>
if (not path.exists(symbol_file)):<br>
- raise error.general('symbol set file: coverage %s was not created for covoar, skipping %s'% (symbol_file, set_name))<br>
+ raise error.general('symbol set file: %s was not '<br>
+ 'created for covoar, skipping %s'% (symbol_file, set_name))<br>
command = ('covoar -S ' + symbol_file<br>
+ ' -O ' + covoar_result_dir<br>
+ ' -E ' + self.explanations_txt<br>
diff --git a/tester/rt/test.py b/tester/rt/test.py<br>
index 0e744cd..cdb5157 100644<br>
--- a/tester/rt/test.py<br>
+++ b/tester/rt/test.py<br>
@@ -232,7 +232,7 @@ def run(command_path = None):<br>
'--filter': 'Glob that executables must match to run (default: ' +<br>
default_exefilter + ')',<br>
'--stacktrace': 'Dump a stack trace on a user termination (^C)',<br>
- '--coverage': 'Perform coverage analysis of test executables.'}<br>
+ '--coverage-sets': 'Perform coverage for specific sets'}<br>
mailer.append_options(optargs)<br>
opts = options.load(sys.argv,<br>
optargs = optargs,<br>
@@ -279,15 +279,20 @@ def run(command_path = None):<br>
else:<br>
rtems_tools = '%{_prefix}'<br>
bsp = opts.find_arg('--rtems-bsp')<br>
+ if 'cov' in bsp[1].split('-'):<br>
+ coverage_enabled = True<br>
if bsp is None or len(bsp) != 2:<br>
raise error.general('RTEMS BSP not provided or an invalid option')<br>
bsp = config.load(bsp[1], opts)<br>
bsp_config = opts.defaults.expand(opts.defaults['tester'])<br>
- coverage_enabled = opts.find_arg('--coverage')<br>
+ coverage_sets = opts.find_arg('--coverage-sets')<br>
if coverage_enabled:<br>
- if len(coverage_enabled) == 2:<br>
- coverage_runner = coverage.coverage_run(opts.defaults,<br>
- coverage_enabled[1],<br>
+ if coverage_sets:<br>
+ if len(coverage_sets) != 2:<br>
+ raise error.general('No sets provided in --coverage-sets')<br>
+ else :<br>
+ coverage_runner = coverage.coverage_run(opts.defaults,<br>
+ coverage_sets[1],<br>
executables)<br>
else:<br>
coverage_runner = coverage.coverage_run(opts.defaults, 0,<br>
<span class="m_3571457339250133045m_1889294988739220246m_-8598312699126845365HOEnZb"><font color="#888888">-- <br>
2.14.3<br>
<br>
</font></span></blockquote></div><br></div></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" rel="noreferrer noreferrer noreferrer" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div></div>
</blockquote></div></div></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>