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

[Alex White]

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.

>
> >  [libraries]
> >  score         = @BUILD-TARGET@/@BSP@/cpukit/score/src
> > @@ -76,4 +76,5 @@ libblock      = @BUILD-TARGET@/@BSP@/cpukit/libblock/src
> >  libcrypt      = @BUILD-TARGET@/@BSP@/cpukit/libcrypt
> >  libmd         = @BUILD-TARGET@/@BSP@/cpukit/libmd
> >  libstdthreads = @BUILD-TARGET@/@BSP@/cpukit/libstdthreads
> > +libtrace      = @BUILD-TARGET@/@BSP@/cpukit/libtrace/record
> >  #zlib          = @BUILD-TARGET@/@BSP@/cpukit/zlib
> > --
> > 2.27.0
> >
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel