<tt><font size=2>Thomas, Jeremy, </font></tt>
<br>
<br><tt><font size=2>Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
wrote on 11/28/2013 05:33:59 AM:<br>
<br>
> Dear Jeremy Rosen,<br>
> <br>
> On Thu, 28 Nov 2013 10:37:27 +0100 (CET), Jeremy Rosen wrote:<br>
> <br>
> > my idea was more along the line of "if we are setting BR2_EXTERNAL<br>
> > and the file doesn't exist, then create an empty "Config.in"
file<br>
> > so the next call to make doesn't fail" <br>
> > <br>
> > this would be to avoid leaving the build system in a broken <br>
> > configuration after a call that sets up the build system for
the<br>
> > user.<br>
> > <br>
> > I can see pro and cons on this idea, there is a risk in touching<br>
> > a file the user is supposed to edit and the breakage itself could<br>
> > be considered a good way to make sure the user go look into the
<br>
> > file<br>
> > <br>
> > but on the other hand, for simple use cases where the user doesn't<br>
> > want to add new packages, but simply overlay a couple of config<br>
> > files, creating a valid infrastructure from the start might be
<br>
> > a good idea...<br>
> <br>
> Ok, I think I see what you mean: creating<br>
> $(BR2_EXTERNAL)/package/Config.in automatically if it doesn't exist.
I<br>
> don't have a really strong feeling about this, but my initial reaction<br>
> is that it's better to let the user do that by himself. BR2_EXTERNAL
is<br>
> part of the user source tree, and I hate when tools mess up with my<br>
> source tree by creating files here and there automatically.<br>
</font></tt>
<br><tt><font size=2>I'm in agreement with Thomas on this issue. I would
prefer it if the user has to create the file (even with it being blank).
However, maybe we could a test within the main makefile that if BR2_EXTERNAL
is used and there is no "package/Config.in" present we through
a make error prompt the user to create this file. I prefer this method
since it would be teaching the user how to correctly use this feature instead
of doing some hidden "auto-magic".</font></tt>
<br>
<br><tt><font size=2>Thanks,</font></tt>
<br><tt><font size=2>-Ryan</font></tt>
<br>