<div dir="ltr">Thanks man.<div><br></div><div>  Matt</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 30, 2020 at 12:20 PM Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I was testing this with branch balay/test in both triangle repo and petsc repo<br>
<br>
If you are to work on this - I guess I should rename them more appropriately.<br>
<br>
Ok - for now will leave balay/test  in triangle [eventually this should become master when we switch]<br>
<br>
And I created balay/triangle-1.6 branch in petsc [will have to rebase this - once balay/gfortran-10-fixes/maint gets merged in.]<br>
<br>
thanks,<br>
Satish<br>
<br>
On Thu, 30 Apr 2020, Matthew Knepley wrote:<br>
<br>
> What branch should I make fixes in?<br>
> <br>
>   Thanks,<br>
> <br>
>     Matt<br>
> <br>
> On Thu, Apr 30, 2020 at 11:18 AM Satish Balay <<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>> wrote:<br>
> <br>
> > I've attempted a build with master [aka triangle-1.6] - there was a fix I<br>
> > had to add to triangle to compile.<br>
> ><br>
> > When I run CI - many petsc examples give diffs. Likely triangle-1.6 was<br>
> > never tested with petsc.<br>
> ><br>
> > <a href="https://gitlab.com/petsc/petsc/pipelines/140824859" rel="noreferrer" target="_blank">https://gitlab.com/petsc/petsc/pipelines/140824859</a><br>
> ><br>
> > So won't make this change now.<br>
> ><br>
> > Satish<br>
> ><br>
> > On Tue, 28 Apr 2020, Matthew Knepley wrote:<br>
> ><br>
> > > On Tue, Apr 28, 2020 at 10:23 AM Satish Balay <<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>> wrote:<br>
> > ><br>
> > > > And I see:<br>
> > > ><br>
> > > > $ git branch -r<br>
> > > >   origin/HEAD -> origin/master<br>
> > > >   origin/maint-1.3<br>
> > > >   origin/master<br>
> > > ><br>
> > > > master has triangle-1.6 changes. However petsc is installing and using<br>
> > > > triangle-1.3 i.e maint.<br>
> > > ><br>
> > > > Matt - any idea why this discrepancy?<br>
> > > ><br>
> > ><br>
> > > I have no idea. It should not be doing that.<br>
> > ><br>
> > > That change looks fine to me.<br>
> > ><br>
> > >   Thanks,<br>
> > ><br>
> > >      Matt<br>
> > ><br>
> > ><br>
> > > > Satish<br>
> > > ><br>
> > > > On Tue, 28 Apr 2020, Satish Balay via petsc-dev wrote:<br>
> > > ><br>
> > > > > I get the following error with triangle+spai build with gcc-10 [this<br>
> > is<br>
> > > > on linux though - ignore osx in PETSC_ARCH]<br>
> > > > ><br>
> > > > >      CLINKER arch-ci-osx-cxx-pkgs-opt/lib/libpetsc.so.3.13.0<br>
> > > > > /usr/bin/ld:<br>
> > > ><br>
> > /home/balay/petsc/arch-ci-osx-cxx-pkgs-opt/lib/libtriangle.a(triangle.o):/home/balay/petsc/arch-ci-osx-cxx-pkgs-opt/externalpackages/Triangle/src/triangle.c:667:<br>
> > > > multiple definition of `epsilon';<br>
> > > ><br>
> > /home/balay/petsc/arch-ci-osx-cxx-pkgs-opt/lib/libspai.a(spai.o):(.bss+0x188):<br>
> > > > first defined here<br>
> > > > ><br>
> > > > ><br>
> > > > > Matt,<br>
> > > > ><br>
> > > > > Any objection to this change to triangle?<br>
> > > > ><br>
> > > > > diff --git a/src/triangle.c b/src/triangle.c<br>
> > > > > index 8e915a1..76f6d84 100644<br>
> > > > > --- a/src/triangle.c<br>
> > > > > +++ b/src/triangle.c<br>
> > > > > @@ -664,7 +664,7 @@ long samples;                /* Number of random<br>
> > > > samples for point location. */<br>
> > > > >  unsigned long randomseed;                     /* Current random<br>
> > number<br>
> > > > seed. */<br>
> > > > ><br>
> > > > >  REAL splitter;       /* Used to split REAL factors for exact<br>
> > > > multiplication. */<br>
> > > > > -REAL epsilon;                             /* Floating-point machine<br>
> > > > epsilon. */<br>
> > > > > +static REAL epsilon;                      /* Floating-point machine<br>
> > > > epsilon. */<br>
> > > > >  REAL resulterrbound;<br>
> > > > >  REAL ccwerrboundA, ccwerrboundB, ccwerrboundC;<br>
> > > > >  REAL iccerrboundA, iccerrboundB, iccerrboundC;<br>
> > > > ><br>
> > > > > Satish<br>
> > > > ><br>
> > > ><br>
> > > ><br>
> > ><br>
> > ><br>
> ><br>
> ><br>
> <br>
> <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>