On 11 May 2018 at 11:50, Cillian O'Donnell <cpodonne...@gmail.com> wrote:

> At the moment when the tester runs covoar the error messages generated are
> a clear indicator of what is wrong, so task list is dictated by what
> they're saying at the moment. Fixing the mismatch between exe.cov and .cov
> is the next thing. The name is given as an argument to the -exec option of
> the qemu cmd and is picked up from leon3-qemu-cov. Have a look at that and
> see can you fix it.
>

The attached diff fixes the mismatch.
shall we start a new thread for tester support ?
diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc
index 5bed98f..95dc990 100644
--- a/tester/covoar/covoar.cc
+++ b/tester/covoar/covoar.cc
@@ -185,7 +185,7 @@ int main(
   Executables                   executablesToAnalyze;
   Coverage::ExecutableInfo*     executableInfo = NULL;
   std::string                   executableExtension = "exe";
-  std::string                   coverageExtension = "cov";
+  std::string                   coverageExtension = ".cov";
   Coverage::CoverageFormats_t   coverageFormat = Coverage::COVERAGE_FORMAT_QEMU;
   Coverage::CoverageReaderBase* coverageReader = NULL;
   char*                         executable = NULL;
@@ -317,11 +317,12 @@ int main(
         std::cerr << "warning: Unable to read executable: " << argv[i] << std::endl;
       } else {
         coverageFileName = argv[i];
-        coverageFileName.replace(
+	coverageFileName.append(coverageExtension);
+     /* coverageFileName.replace(
           coverageFileName.length() - executableExtension.size(),
           executableExtension.size(),
           coverageExtension
-        );
+        ); */
 
         if (!FileIsReadable( coverageFileName.c_str() )) {
           std::cerr << "warning: Unable to read coverage file: " << coverageFileName
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to