<div dir="ltr"><div dir="ltr">On Thu, Jul 11, 2019 at 8:46 PM Smith, Barry F. <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> On Jul 11, 2019, at 6:02 PM, Matthew Knepley via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov" target="_blank">petsc-dev@mcs.anl.gov</a>> wrote:<br>
> <br>
> Barry, <br>
> <br>
> Do you want to handle the revert? Satish, do you?<br>
<br>
   The problem is the old code doesn't properly handle the dependencies and always produces the error about deleting non-empty directory. I'd like to understand the underlying problem rather then just  go from a new broken thing to an old broken thing.<br></blockquote><div><br></div><div>Okay, that is reasonable.</div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
>   Thanks,<br>
> <br>
>     Matt<br>
> <br>
> On Thu, Jul 11, 2019 at 3:02 PM Balay, Satish via petsc-maint <<a href="mailto:petsc-maint@mcs.anl.gov" target="_blank">petsc-maint@mcs.anl.gov</a>> wrote:<br>
> BTW: I think this commit also triggers the following error - but I haven't yet debugged this..<br>
> <br>
> <a href="http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2019/07/10/examples_master_arch-osx-10.6-cxx-pkgs-opt_ipro.log" rel="noreferrer" target="_blank">http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2019/07/10/examples_master_arch-osx-10.6-cxx-pkgs-opt_ipro.log</a><br>
> <br>
> Satish<br>
> <br>
> On Thu, 11 Jul 2019, Balay, Satish via petsc-dev wrote:<br>
> <br>
> > On Thu, 11 Jul 2019, Balay, Satish via petsc-dev wrote:<br>
> > <br>
> > > On Thu, 11 Jul 2019, Matthew Knepley via petsc-dev wrote:<br>
> > > <br>
> > > > After my latest pull of master, making the 'test' target no longer rebuilds<br>
> > > > the library. I have tested this on a few arches, and rebuilt. This is<br>
> > > > pretty inconvenient, but I do not know how to fix it.<br>
> > > <br>
> > > <br>
> > > git bisect gives the following.<br>
> > > <br>
> > > Satish<br>
> > > <br>
> > > -----<br>
> > > <br>
> > > 27d73d1f0a5c445a3a02971e31a2a1a02ed6d224 is the first bad commit<br>
> > > commit 27d73d1f0a5c445a3a02971e31a2a1a02ed6d224<br>
> > > Author: Barry Smith <<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>><br>
> > > Date:   Sat Jun 22 22:56:05 2019 -0500<br>
> > > <br>
> > >     Fix the error from gmakefile.test test about trying to remove a non-empty directory<br>
> > >     <br>
> > >     The problem was the target to rm -r the directory was running at the same time as<br>
> > >     tests tests where generatering new files in the directory<br>
> > >     <br>
> > >     Commit-type: bug-fix<br>
> > > <br>
> > > :100644 100644 67a00e0c1c9ce3eb91a88f314e81a74ac278131f 9c0c1b310a238fc27df3af114ec96336ef5640d2 M      gmakefile.test<br>
> > <br>
> > specifically the following change (with recursive make) likely broke the dependencies..<br>
> > <br>
> > >>>>>>>>>>>>><br>
> > -test: pre-clean report_tests<br>
> > +test:<br>
> > +     +@${OMAKE} -f gmakefile.test pre-clean<br>
> > +     +@${OMAKE} -f gmakefile.test report_tests<br>
> > <<<<<<<<<<<<br>
> > <br>
> > Satish<br>
> > <br>
> <br>
> <br>
> <br>
> -- <br>
> What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
> -- Norbert Wiener<br>
> <br>
> <a href="https://www.cse.buffalo.edu/~knepley/" rel="noreferrer" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div><div><br></div><div><a href="http://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br></div></div></div></div></div></div></div></div>