[U-Boot] [RFC PATCH v2 0/2] Make Python scripts compatible with older versions

[Igor Grinberg <grinberg@compulab.co.il>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 08/07/14 13:57, Tom Rini wrote:
> On Mon, Aug 04, 2014 at 04:17:00PM +0300, Igor Grinberg wrote:
>> Hi Masahiro,
>>
>> On 08/04/14 15:59, Masahiro Yamada wrote:
>>> Hi.
>>>
>>>>
>>>>> It seems that those scripts only work on Python versions 2.6 - 2.7?
>>>
>>>
>>>
>>> I took a quick look and I found  Python 3.x is not comatible 2.x at all.
>>>
>>> 3.x  requires the "print" is called like
>>> print(msg, file=sys.stderr)
>>> but it failes in Python 2.7.x.
>>
>> Last time I checked the print function, it worked with parenthesis on
>> both versions 2.7 and 3.x.
>>
>>>
>>> Moreover,  "except ... , ..." fails in 3.x
>>> while  "except ... as ..." fails in 2.5 or earlier.
>>>
>>> If the compatibility with python 3 is the requirement,
>>> I can't do this.  I must throw Python scripts away.
>>
>> I think we'd better replace these with something more stable
>> in terms of API... bash? perl?
> 
> I don't think API is an argument against python,

Not the API as stand alone, of course, but the burden to support
its changes and breakages.
Why should U-Boot even have python as a build prerequisite?
I really think that having shell, make, and $(cross)gcc should be
enough for the basic source build.

> we just need
> /usr/bin/env python2 as how we invoke our scripts.

This means impose python version dependency for U-Boot source build?
Correct me if you think I'm wrong, but I don't think this is a good
practice...
I think that for tools like buildman, patman, etc. - this is
perfectly fine to impose an interpreter/compiler version, but not
for the basic source builds.

> 
> The question is, what helper scripts do we really need to have around
> and expect many people to use.

IMO, helper scripts, that are not involved in U-Boot source build
can be written in any language/script.

- -- 
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBAgAGBQJT43xhAAoJEBDE8YO64EfaAt0QAIKiLhrxDpH2qPplAKr7SRrB
ohP+7aRvKAV175KwOGIbWEDNybmdii+/wW+Hl1pHkt7UL6jcJVQjQ8Qrv9vg44TZ
YFR14VE4lBMOU62NXaTBMaae49dT69Kq2LemOvizNbMr6PvWi3cvFTWS9voROVtE
ydWCpQhFzj6iau55Eg8anFe47jFGdsGyysdfCE7Vu+h1KOMGvEY613rZO+od3ZR4
vHAAtlD4wvIfg/r+8FRvbSnooI+HlTI/vlimcwkH1YlX1I256Db15FAYx9OnREPN
/9NTFXiFLAd7iucjI0ScF+hY+U0ni4qcA5le/C5/lgMgQi08vSlWKuziJNbGST1/
P4ijeb/kh34oTYphKcJytr5paml9Cd+ljIWDi3AWjSxGjQm4Oc3knk0CK9hfV3N7
A1Xs9hZgEMzU01peYtj8FyeEmpRXHpWhHAbPB5N2xcKW2rDrnVhX4+qupRxYP2VN
LcWuTrJGvFJUiG0T4+gsI/DiStLNQxI0M/UoBlIhSP9fIceVCr6NNFZRcPG1MBfy
MsOMSW4SgP1Yg/Gil47bIXiZDSNn0QNJtanAEBwHSZOnmvw+WO1XMoGAaAnAzRUw
xSd5+gsj2xeWOI4adc2ye95ZZ8FLh5rfaZoxv0q8aeuqnBsp4dBDQL95WNpDVDdG
Kpkpr6gloENHHwTptRVc
=iVHF
-----END PGP SIGNATURE-----