<div dir="ltr">From my perspective, yes. If it breaks something else, we can fix that.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 30, 2019 at 9:47 AM Hesham Almatary <<a href="mailto:hesham.almatary@cl.cam.ac.uk">hesham.almatary@cl.cam.ac.uk</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">OK to push?<br>
<br>
On Tue, 29 Oct 2019 at 09:25, Hesham Almatary<br>
<<a href="mailto:hesham.almatary@cl.cam.ac.uk" target="_blank">hesham.almatary@cl.cam.ac.uk</a>> wrote:<br>
><br>
> On Mon, 28 Oct 2019 at 23:15, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
> ><br>
> ><br>
> ><br>
> > On Mon, Oct 28, 2019 at 5:20 AM Hesham Almatary <<a href="mailto:hesham.almatary@cl.cam.ac.uk" target="_blank">hesham.almatary@cl.cam.ac.uk</a>> wrote:<br>
> >><br>
> >> On Mon, 28 Oct 2019 at 04:01, Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>> wrote:<br>
> >> ><br>
> >> ><br>
> >> ><br>
> >> > On 28/10/19 9:21 am, Hesham Almatary wrote:<br>
> >> > ><br>
> >> > ><br>
> >> > > On Sun, 27 Oct 2019 at 20:54, Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a><br>
> >> > > <mailto:<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>>> wrote:<br>
> >> > ><br>
> >> > >     On 27/10/19 9:37 pm, Hesham Almatary wrote:<br>
> >> > >     > Yeah fesvr is now part of Spike in-tree. I didn't like it had to be<br>
> >> > >     > built separately either [1].<br>
> >> > >     ><br>
> >> > >     > Joel, AFAIR, dtc was always needed.<br>
> >> > ><br>
> >> > >     Please have the package build FDT if it is needed. Do no rely on it being<br>
> >> > >     installed as some hosts do not have a package to install.<br>
> >> > ><br>
> >> > > I think Joel has already added it<br>
> >> > > <a href="https://github.com/RTEMS/rtems-source-builder/blob/master/bare/config/devel/spike.bset#L7" rel="noreferrer" target="_blank">https://github.com/RTEMS/rtems-source-builder/blob/master/bare/config/devel/spike.bset#L7</a><br>
> >> > ><br>
> >> ><br>
> >> > Thanks.<br>
> >> ><br>
> >> > The config file has some issues. I am fixing the %hash issues. Is the patch<br>
> >> > still needed if the version used has the fix?<br>
> >> ><br>
> >> The version that has the fix is a recent one, and it has fesvr in its<br>
> >> source tree, hence, there's no separate fesvr any more. The patch will<br>
> >> still be needed to get rid of the separate fesvr build for this recent<br>
> >> Spike revision.<br>
> ><br>
> ><br>
> > If you are bumping to a version which doesn't need my patch, great!<br>
> ><br>
> Yes, that's the point of the patch along with removing fesvr build.<br>
><br>
> > Is this all pushed now?<br>
> ><br>
> Not yet, waiting for approval.<br>
><br>
> >><br>
> >> > Chris<br>
> >> > _______________________________________________<br>
> >> > devel mailing list<br>
> >> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> >> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
> >> _______________________________________________<br>
> >> devel mailing list<br>
> >> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> >> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>