<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/28 Thomas Petazzoni <span dir="ltr"><<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Dear Samuel Martin,<br>
<div class="im"><br>
On Thu, 28 Nov 2013 14:24:35 +0100, Samuel Martin wrote:<br>
<br>
> In this new version of this patch set, the all target and host<br>
> packages entries end up under the "Target packages" menu.<br>
> In my use of this feature, I also have a couple of host tools to<br>
> integrate. So, how about having adding the following source line in<br>
> package/Config.in.host:<br>
> ---<br>
> source "$BR2_EXTERNAL/package/Config.in.host"<br>
> ---<br>
> Of course, this requires to add an<br>
> empty support/dummy-external/package/Config.in.host file.<br>
<br>
</div>This means the user would always have to create both package/Config.in<br>
and package/Config.in.host, which gets annoying.</blockquote><div>Yes, that's the only drawback of this. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

 That's why in the v2 I<br>
did include $(BR2_EXTERNAL)/Config.in, without enforcing a<br>
$(BR2_EXTERNAL)/package/ subdirectory, and leave more freedom to the<br>
user. However, several people at the ELCE meeting said that we should<br>
enforce the package/ directory, and therefore only include<br>
$(BR2_EXTERNAL)/package/Config.in.<br>
<br>
Just curious, why can't you simply add your host packages also in<br>
$(BR2_EXTERNAL)/package/Config.in ? It's not because the main Buildroot<br>
doesn't do it this way that you can't do it in your BR2_EXTERNAL<br>
directory.<br></blockquote><div>Well, right now, most of the packages I put in my BR2_EXTERNAL tree are host</div><div>tools to generate images.</div><div><br></div><div>With the previous version of this serie, the new menu entry was added at the top level</div>

<div>of menuconfig, in which I added 2 submenus: "Target packages" and "Host packages".</div><div>It was rather clear that all "company" stuff goes into this menu.</div><div><br></div><div>With the v2, I was a bit lost at first, the "company" menu got moved under</div>

<div>"Target packages". So, now I have a menu tree like this:</div><div><br></div><div>---</div><div>Main menu</div><div>  ...</div><div>  Target packages ---></div><div>    ...</div><div>    Company ---></div>

<div>      Target packages ---></div><div><div>      Host packages ---></div></div><div>  ...</div><div><div>  Host packages ---></div></div><div>---</div><div><br></div><div>It does not hurt that much, but it's not really nice IMHO.</div>

<div><br></div><div>I agree that maybe most of the users may not need to add host packages;</div><div>that does not mean there is no room for improvement ;)</div><div><br></div><div>BTW, to generate this/these Config.in{,host} files in the BR2_EXTERNAL tree,</div>

<div>I already have a script scanning the tree and updating these files, it could be </div><div>included as a support script as well.</div><div><br></div><div><br></div><div>Regards,</div></div><div><br></div>-- <br>Samuel
</div></div>