public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH v2 0/2] Make Python scripts compatible with older versions
Date: Thu, 07 Aug 2014 11:33:35 -0600	[thread overview]
Message-ID: <53E3B86F.6060703@wwwdotorg.org> (raw)
In-Reply-To: <20140807165719.GT19374@bill-the-cat>

On 08/07/2014 10:57 AM, Tom Rini wrote:
> On Thu, Aug 07, 2014 at 04:17:21PM +0300, Igor Grinberg wrote:
>> On 08/07/14 13:57, Tom Rini wrote:
..
>>> 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.
>
> I agree.  You don't need MAKEALL or buildman to do basic source builds.
> Doing 'make foo_defconfig' doesn't require re-creating boards.cfg.
>
> To me, the gray area is people doing SoC level (or higher) changes that
> want to be good and test more areas.  That's when MAKEALL or buildman
> become handy and some sort of win over a shell forloop.

Why on earth isn't relying specifically on either Python2 (with the 
current script code) or Python3 (after porting the code) a good practice?

There was a major release of the language, and the language changed. 
Such a change is just like any other scripting language. Scripts written 
for one particular version of the language should specify python2 or 
python3 not just python in their #! line. Problem solved.

For now at least, all distros I know of provide both python2 and 
python3, so there's no issue doing this. In N years when there's only 
python3 available, I assumed the scripts would be updated, in the same 
way we use new C features (e.g. bool) and drop support for older tools 
of other kinds.

Banning or replacing the use of Python just because they cleaned up 
their language seems like poking your eye out to spite your nose (or 
whatever the expression is). The same thing will happen with Perl, and 
happened with dtc, etc.

  reply	other threads:[~2014-08-07 17:33 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-04 10:23 [U-Boot] [RFC PATCH v2 0/2] Make Python scripts compatible with older versions Masahiro Yamada
2014-08-04 10:23 ` [U-Boot] [RFC PATCH v2 1/2] kconfig: make multiconfig.py compatible with Python 2.4 Masahiro Yamada
2014-08-04 10:23 ` [U-Boot] [RFC PATCH v2 2/2] tools: make genboardscfg.py compatible with Python 2.5 Masahiro Yamada
2014-08-22  6:01   ` Masahiro Yamada
2014-08-22  6:18     ` Masahiro Yamada
2014-08-23 12:44     ` Tom Rini
2014-08-25  1:53       ` Masahiro Yamada
2014-08-04 10:40 ` [U-Boot] [RFC PATCH v2 0/2] Make Python scripts compatible with older versions Simon Glass
2014-08-04 11:21 ` Igor Grinberg
2014-08-04 12:00   ` Masahiro Yamada
2014-08-04 12:59     ` Masahiro Yamada
2014-08-04 13:17       ` Igor Grinberg
2014-08-04 13:28         ` Masahiro Yamada
2014-08-07 10:57         ` Tom Rini
2014-08-07 13:17           ` Igor Grinberg
2014-08-07 16:57             ` Tom Rini
2014-08-07 17:33               ` Stephen Warren [this message]
2014-08-07 17:53                 ` Tom Rini
2014-08-10  8:49                 ` Igor Grinberg
2014-08-10 11:14                   ` Tom Rini
2014-08-11 10:18                     ` Igor Grinberg
2014-08-11 13:12                       ` Tom Rini
2014-08-11 23:03                         ` Masahiro YAMADA
2014-08-12  1:53                           ` Tom Rini
2014-08-18  4:45                             ` Masahiro Yamada
2014-08-11 16:58                   ` Stephen Warren
2014-08-12 10:32                     ` Igor Grinberg
2014-08-04 13:50 ` Tom Rini
2014-08-05  2:05   ` Yan, Miao
2014-08-05 14:18     ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53E3B86F.6060703@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox