<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2024 at 2:11 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</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"><div dir="auto"><div dir="ltr"></div><div dir="ltr"></div><div dir="ltr"><blockquote type="cite">On 4 Apr 2024, at 3:52 am, Kinsey Moore <<a href="mailto:kinsey.moore@oarcorp.com" target="_blank">kinsey.moore@oarcorp.com</a>> wrote:<br></blockquote></div><blockquote type="cite"><div dir="ltr"><div dir="ltr"><div>Looks fine overall. Minor nits:</div><div>"host" is set to "freebsd" and is never used.</div></div></div></blockquote><div><br></div>The tool uses simhost so it could be any host listed in that module and it references that table. Nothing is built but needed to creat a build object. Simhost is used to get all sources for all hosts. </div></blockquote><div><br></div><div>I'm not referencing simhost. It's literally just an unused variable called "host" that gets set and never used.</div><div><br></div><div>Kinsey<br></div></div></div>