<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Thomas,</div><div class="gmail_quote"><br></div><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 18:53:57 +0100, Samuel Martin wrote:<br>
<br>
> Well, right now, most of the packages I put in my BR2_EXTERNAL tree<br>
> are host tools to generate images.<br>
><br>
> With the previous version of this serie, the new menu entry was added<br>
> at the top level<br>
> of menuconfig, in which I added 2 submenus: "Target packages" and<br>
> "Host packages".<br>
> It was rather clear that all "company" stuff goes into this menu.<br>
><br>
> With the v2, I was a bit lost at first, the "company" menu got moved<br>
<br>
</div>I guess you meant "v3" here, because the "v2" included<br>
$(BR2_EXTERNAL)/Config.in in the top-level menu of menuconfig.<br></blockquote><div>Indeed. Sorry for the confusion.</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">


<div class="im"><br>
> under "Target packages". So, now I have a menu tree like this:<br>
><br>
> ---<br>
> Main menu<br>
>   ...<br>
>   Target packages ---><br>
>     ...<br>
>     Company ---><br>
>       Target packages ---><br>
>       Host packages ---><br>
>   ...<br>
>   Host packages ---><br>
> ---<br>
><br>
> It does not hurt that much, but it's not really nice IMHO.<br>
<br>
</div>Then please talk to the people who asked for enforcing<br>
$(BR2_EXTERNAL)/package/Config.in usage during the Buildroot Developers<br>
Days in Edinburgh.</blockquote><div>I thought the ml was the place for 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">

 This decision/choice is written very clearly in<br>
<a href="http://elinux.org/Buildroot:DeveloperDaysELCE2013#BR2_EXTERNAL" target="_blank">http://elinux.org/Buildroot:DeveloperDaysELCE2013#BR2_EXTERNAL</a> :<br>
<br>
"""<br>
Regarding the directory hierarchy in the external tree, it was agreed<br>
that it is a good idea to force three subdirectories: package, board,<br>
configs. Buildroot's package/Config.in will source<br>
$BR2_EXTERNAL/package/Config.in.<br>
"""<br></blockquote><div>Doh! I don't mean this at all! Looks like I was not clear enough.</div><div><br></div><div>I'm not talking about the directory hierarchy, I do follow the Buildroot hierarchy.</div>

<div>It does look like this:</div><div><br></div><div><div>BR2_EXTERNAL/</div><div>|`- board/</div><div>|    `- someboard/</div><div>|       |`- linux-myversion/</div><div>|       |    `- linux-0001-fix-something.patch</div>

<div>|       |`- busybox-1.21.1.config</div><div>|        `- post-build-script.sh<br></div><div>|`- configs/</div><div>|    `- someboard_defconfig</div><div> `- package/</div><div>    |`- Config.in<br></div><div>    |`- Config.in.host<br>

</div><div>    |`- foo/</div><div>    |   |`- <a href="http://foo.mk">foo.mk</a></div><div>    |   |`- Config.in</div><div>    |    `- Config.in.host</div><div><div>    |`- bar/</div><div>    |   |`- <a href="http://bar.mk">bar.mk</a></div>

<div>    |    `- Config.in.host<br></div></div></div><div><div>     `- toto/</div><div>        |`- <a href="http://toto.mk">toto.mk</a></div><div>         `- Config.in</div><div><br></div></div><div>With BR2_EXTERNAL/package/Config.in sourcing all Config.in files from BR2_EXTERNAL,</div>

<div>and BR2_EXTERNAL/package/Config.in.host sourcing all Config.in.host files from BR2_EXTERNAL,<br></div><div><br></div><div>My point was only about sourcing BR2_EXTERNAL/package/Config.in.host under the</div><div>"Host utilities" menu.</div>

<div><br></div><div>This is nothing more than consistency in the menuconfig rendering.</div><div>If no one thinks it's relevant, fair enough, I can live without it.</div><div> <br></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">


(It's even in bold in the report).<br></blockquote><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"><br>


I believe the most vocal person in favor of this was Arnout, so I've<br>
added him in Cc.<br>
<div class="im"><br>
> BTW, to generate this/these Config.in{,host} files in the<br>
> BR2_EXTERNAL tree, I already have a script scanning the tree and<br>
> updating these files, it could be included as a support script as<br>
> well.<br></div></blockquote><div>It only generate the BR2_EXTERNAL/package/Config.in{,.host} files because I'm lazy.</div><div><br></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">

<div class="im">
<br>
</div>Why is this needed? Just write Config.in like we do in the main<br>
Buildroot tree. For example, people may want to have menus and submenus<br>
in their $(BR2_EXTERNAL)/package/Config.in. Therefore, I don't think<br>
providing more and more and more scripts that match a very specific use<br>
case is really going to help our users, probably going to confuse them.<br></blockquote><div>Fair enough.</div><div><br></div></div>Regards,<br clear="all"><div><br></div>-- <br>Samuel
</div></div>