[PATCH] coverage/symbol-sets.ini : Add libtrace

Chris Johns chrisj at rtems.org
Mon Mar 29 21:44:33 UTC 2021


On 30/3/21 7:19 am, Alex White wrote:
> On Sat, Mar 27, 2021 at 9:46 AM Gedare Bloom <gedare at rtems.org> wrote:
>>
>> On Fri, Mar 12, 2021 at 10:17 AM Alex White <alex.white at oarcorp.com> wrote:
>>>
>>> ---
>>>  tester/rtems/testing/coverage/symbol-sets.ini | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/tester/rtems/testing/coverage/symbol-sets.ini b/tester/rtems/testing/coverage/symbol-sets.ini
>>> index 9617dd8..52e25ff 100644
>>> --- a/tester/rtems/testing/coverage/symbol-sets.ini
>>> +++ b/tester/rtems/testing/coverage/symbol-sets.ini
>>> @@ -29,7 +29,7 @@
>>>  #
>>>
>>>  [symbol-sets]
>>> -sets = score,rtems,sapi,posix,librfs,libpipe,libdosfs,libimfs,libjffs2,libcsupport,libbspcmdline,libcpuuse,libstackchk,libfsmount,libstringto,libdevnull,libdumpbuf,libuntar,libblock,libcrypt,libmd,libstdthreads
>>> +sets = score,rtems,sapi,posix,librfs,libpipe,libdosfs,libimfs,libjffs2,libcsupport,libbspcmdline,libcpuuse,libstackchk,libfsmount,libstringto,libdevnull,libdumpbuf,libuntar,libblock,libcrypt,libmd,libstdthreads,libtrace
>>>
>> ok, but this is really ugly. is the comma-separated list with no
>> whitespace mandatory, or can it be reformatted in a follow-up patch?
> 
> Currently, it is mandatory because the coverage.py script does not remove whitespace when processing the values, it simply splits them on ',' characters.
> 
> It can be reformatted in a follow-up patch to look something like this:
> 
> sets = score,rtems,sapi,posix,librfs,libpipe,libdosfs,libimfs,libjffs2,libcsupport,
> 	libbspcmdline,libcpuuse,libstackchk,libfsmount,libstringto,libdevnull,
> 	libdumpbuf,libuntar,libblock,libcrypt,libmd,libstdthreads,libtrace
> 
> As long as subsequent lines are indented (to comply with Python's ConfigParser format), it should work fine assuming logic is added to call strip() on the values in the parse() method of the symbol_parser class in coverage.py.

Maybe the better solution is to use the toolkit's `configuraiton` module which
manages a number of things including this (see comma_list [1]). I suggest you
investigate this path.

Chris

[1] https://git.rtems.org/rtems-tools/tree/rtemstoolkit/configuration.py#n141


More information about the devel mailing list