<div dir="ltr">For each tested line, it would be helpful if we also show the test (one is enough) that tested the line.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">--Junchao Zhang</div></div></div>
<br><div class="gmail_quote">On Fri, Mar 16, 2018 at 10:40 AM, Balay, Satish <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, 16 Mar 2018, Junchao Zhang wrote:<br>
<br>
> The coverage page at <a href="http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/index.html" rel="noreferrer" target="_blank">http://ftp.mcs.anl.gov/pub/<wbr>petsc/nightlylogs/index.html</a><br>
> is missing. When I read certain PETSc code, I feel some if conditions will<br>
> never be met (i.e., there are dead statements). To confirm that, I want to<br>
> know which tests run into that condition.<br>
<br>
</span>Thanks - its updated now.<br>
<br>
<br>
Jed,<br>
<br>
I must have messedup the testing of the latest gcov changes..<br>
<br>
>>>>>>><br>
[thwomp:petsc/nightlylogs/<wbr>gcov] petsc> cat ~/nightlylogs/archive/2018/03/<wbr>16/gcov-snapshot-update.log<br>
Traceback (most recent call last):<br>
  File "/sandbox/petsc/petsc.clone/<wbr>lib/petsc/bin/maint/gcov.py", line 18, in <module><br>
    PETSC_ARCH = os.environ['PETSC_ARCH']<br>
  File "/usr/lib/python2.7/UserDict.<wbr>py", line 23, in __getitem__<br>
    raise KeyError(key)<br>
KeyError: 'PETSC_ARCH'<br>
<<<<<<<<br>
<br>
So the issue is - this script is run in 2 different modes:<br>
<br>
lib/petsc/bin/maint/gcov.py:        print "         ./gcov.py -run_gcov      "<br>
lib/petsc/bin/maint/gcov.py:        print "         ./gcov.py -merge_gcov [LOC] tarballs"<br>
<br>
For the second mode is with [LOC] - i.e there is no prior build - i.e no PETSC_ARCH<br>
<br>
So how about the following change?<br>
<br>
diff --git a/lib/petsc/bin/maint/gcov.py b/lib/petsc/bin/maint/gcov.py<br>
index 6d0e1e0..0a49978 100755<br>
--- a/lib/petsc/bin/maint/gcov.py<br>
+++ b/lib/petsc/bin/maint/gcov.py<br>
@@ -14,10 +14,6 @@ import operator<br>
 import sys<br>
 from   time import gmtime,strftime<br>
<br>
-PETSC_DIR = os.environ['PETSC_DIR']<br>
-PETSC_ARCH = os.environ['PETSC_ARCH']<br>
-OBJDIR = os.path.join(PETSC_DIR, PETSC_ARCH, 'obj')<br>
-<br>
 def run_gcov(gcov_dir):<br>
<br>
     # 1. Runs gcov<br>
@@ -389,11 +385,15 @@ def make_htmlpage(gcov_dir,LOC,<wbr>tarballs):<br>
<br>
 def main():<br>
<br>
+    global PETSC_DIR,PETSC_ARCH,OBJDIR<br>
     USER = os.environ['USER']<br>
     gcov_dir = "/tmp/gcov-"+USER<br>
+    PETSC_DIR = os.environ['PETSC_DIR']<br>
<br>
     if (sys.argv[1] == "-run_gcov"):<br>
         print "Running gcov and creating tarball"<br>
+        PETSC_ARCH = os.environ['PETSC_ARCH']<br>
+        OBJDIR = os.path.join(PETSC_DIR, PETSC_ARCH, 'obj')<br>
         run_gcov(gcov_dir)<br>
         make_tarball(gcov_dir)<br>
     elif (sys.argv[1] == "-merge_gcov"):<br>
<br>
<br>
Satish<br>
</blockquote></div><br></div>