<div dir="ltr">This has been introduced in f42deef95efdb32c74e20b044efbd18de6bff042.<div>That commit also broke another feature that I find very useful, which is to rerun only the failed tests.</div><div>Now, the echofailures.sh file is no longer updated unless we do full testing.</div><div>Can we please revert that commit?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il giorno mer 20 gen 2021 alle ore 14:11 Stefano Zampini <<a href="mailto:stefano.zampini@gmail.com">stefano.zampini@gmail.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">This is an issue with the default shell used by the makefile. Below is my fix. We should probably have a CI machine that checks for these shell-related errors.<div><br><div>diff --git a/gmakefile.test b/gmakefile.test<br>index c38e37f..ffd7bdb 100644<br>--- a/gmakefile.test<br>+++ b/gmakefile.test<br>@@ -379,10 +379,11 @@ starttime: pre-clean $(libpetscall)<br>        @$(eval STARTTIME := $(shell date +%s))<br> <br> report_tests: starttime $(TESTTARGETS)<br>+ifeq ($(showreport),true)<br>        @$(eval ENDTIME := $(shell date +%s))<br>-       -@if test ${showreport} == "true"; then  elapsed_time=$$(($(ENDTIME)- $(STARTTIME))) && \<br>-       $(PYTHON) $(CONFIGDIR)/report_tests.py -m $(MAKE) -d $(TESTDIR)/counts -t 5 -e $${elapsed_time};\<br>-    fi<br>+       elapsed_time=$$(($(ENDTIME)- $(STARTTIME))) && \<br>+       $(PYTHON) $(CONFIGDIR)/report_tests.py -m $(MAKE) -d $(TESTDIR)/counts -t 5 -e $${elapsed_time};<br>+endif<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il giorno mar 19 gen 2021 alle ore 20:41 Scott Kruger <<a href="mailto:kruger@txcorp.com" target="_blank">kruger@txcorp.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF">
    <br>
    <br>
    <font size="+1">I can't reproduce this with the latest master:<br>
      <br>
      hip 1261: git pull<br>
      Already up to date.<br>
      hip 1262: make -f gmakefile.test test search='notatest'<br>
      Using MAKEFLAGS: -- search=notatest<br>
      hip 1263:<br>
      <br>
      <br>
    </font><br>
    <div>On 1/19/21 8:19 AM, Stefano Zampini
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">Just rebased over latest master and got this
        <div><br>
        </div>
        <div>zampins@vulture:~/Devel/petsc$ make -f gmakefile.test test
          search='notatest'<br>
          Using MAKEFLAGS: -- search=notatest<br>
          /bin/sh: 1: test: false: unexpected operator<br clear="all">
          <div><br>
          </div>
          -- <br>
          <div dir="ltr">Stefano</div>
        </div>
      </div>
    </blockquote>
    <br>
    <pre cols="72">-- 
Tech-X Corporation               <a href="mailto:kruger@txcorp.com" target="_blank">kruger@txcorp.com</a>
5621 Arapahoe Ave, Suite A       Phone: (720) 466-3196
Boulder, CO 80303                Fax:   (303) 448-7756</pre>
  </div>

</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr">Stefano</div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">Stefano</div>