[U-Boot] [PATCH v2] kconfig: convert Kconfig helper script into a shell script

[Tom Rini]

On Tue, Aug 19, 2014 at 11:34:38AM +0900, Masahiro Yamada wrote:
> Hi.
> 
> 
> 
> On Mon, 18 Aug 2014 10:27:29 -0600
> Simon Glass <sjg at chromium.org> wrote:
> 
> > Hi,
> > 
> > On 18 August 2014 10:20, Stephen Warren <swarren at wwwdotorg.org> wrote:
> > 
> > > On 08/16/2014 10:56 PM, Masahiro Yamada wrote:
> > >
> > >> Commit 51148790 added scripts/multiconfig.py written in Python 2
> > >> to adjust Kconfig for U-Boot.
> > >>
> > >> It has been hard for Python 3 users because Python 2 and Python 3
> > >> are not compatible with each other.
> > >>
> > >> We are not happy about adding a new host tool dependency
> > >> (in this case, Python version dependency) for the core build process.
> > >> After some discussion, we decided to use only basic tools.
> > >>
> > >> The script may get a bit unreadable by shell scripting, but we believe
> > >> it is worthwhile.
> > >>
> > >> In addition, this commit revives "<board>_config" target that is
> > >> equivalent to "<board>_defconfig" for backwards compatibility.
> > >> It is annoying to adjust various projects which use U-Boot.
> > >>
> > >
> > > Personally, I think this change isn't justified; I see no reason to
> > > replace the perfectly working existing script. Still, if that's what's
> > > desired...
> 
> In that case, I wanted you to stop me earlier.
> I wrote this patch during my summer vacation.  :-(

Thank you for doing it on your vacation!  Yes, I'm _not_ comfortable
making python a requirement for _everyone_ to build U-Boot.  I'm willing
to consider replacing MAKEALL with buildman since that's a more U-Boot
developer tool.  But for the average user we're making things more
complicated than we must just so they can use our software.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140819/9683f6d7/attachment.pgp>