[PATCH] Generate coverage analysis Report
Chris Johns
chrisj at rtems.org
Wed Jun 6 02:54:27 UTC 2018
On 31/5/18 6:44 am, Vijay Kumar Banerjee wrote:
> On 31 May 2018 at 02:02, Joel Sherrill <joel at rtems.org <mailto:joel at rtems.org>>
> wrote:
> On Wed, May 30, 2018 at 3:29 PM, Vijay Kumar Banerjee
> <vijaykumar9597 at gmail.com <mailto:vijaykumar9597 at gmail.com>> wrote:
>
> On 31 May 2018 at 00:28, Joel Sherrill <joel at rtems.org
> <mailto:joel at rtems.org>> wrote:
> I may not understand correctly but there is test_run and
> coverage_run. Someone
> suggested making coverage_running an option to test_run. If that's
> what's being
> asked for, then I think doing it in a follow up patch is OK.
>
> If that's the intended request, perhaps a ticket should be filed.
>
>
> Sorry for all the confusion.
> This patch doesn't change the way test works. It only adds an option to run
> the coverage script. coverage_run just runs the coverage.coverage_run
>
>
> :) And I am saying if we want to have one test_run with an argument, do it as
> a future work iteration. File a ticket.
>
> We need to get the code working on the master.
>
> Okay, we can keep that as a future work (I haven't thought about it though). :)
> Getting it to work on master is our primary objective.
>
Was a ticket raised to removing 'coverage_run' and to use 'test_run'?
Chris
More information about the devel
mailing list