On 27/04/2018 17:21, Cillian O'Donnell wrote: > On Fri, 27 Apr 2018, 04:48 Chris Johns, <chr...@rtems.org > <mailto:chr...@rtems.org>> wrote: > > On 27/04/2018 08:51, Joel Sherrill wrote: > > > > // Create the set of desired symbols. > > SymbolsToAnalyze = new Coverage::DesiredSymbols(); > > - SymbolsToAnalyze->load( symbolsFile ); > > > > > > I am having trouble seeing what happened to the DesiredSymbols class. Is > > it no longer needed? > > > It's still being used as before, the decision has just been split from did you > provided the symbols file or are you using symbol_set.cfg file. DesiredSymbols > is definitely still being used. I'm looking on my phone so maybe missing > something that would be clearer not on a phone.
I have renamed a few files, for example .. tester/rtems/testing/coverage/symbol_sets.cfg -> tester/rtems/testing/coverage/score-symbols.ini and that file now contains: [symbol-sets] sets = score [score] libraries=@BUILD-PREFIX@/c/@BSP@/cpukit/score/libscore.a I have added code to get the build path, BUILD-PREFIX and BSP from the executable's path and check each executable is from the same build tree. I may check these against the RLD's automatically detected values after loading the libraries. I have add the code to load the libraries and the required symbols. This means some of the code to read the sets and then load the libraries and create a symbol file can be removed. I have not attempted to build to the code and will do this over the weekend or next week. > Just to update, I didn't get time to look at the rsb build yesterday so it'll > have to be tonight. Sorry for the delay, my new job keeps me very busy during > the week, long commutes too. No problem, when you can. Thank you. Chris _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel