public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Petr Vorel <pvorel@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] *.sh: convert hardcoded python2 to python3
Date: Thu, 25 Jul 2019 16:42:42 +0200	[thread overview]
Message-ID: <20190725144241.GA4568@dell5510> (raw)
In-Reply-To: <1563785844-28983-1-git-send-email-yi.zhao@windriver.com>

Hi Yi,

> We had converted python2 to python3 for all *.py in commit
> 5a8408bc32f0eada5cbf65ac63d5cd2410f95249. But there are still hardcoded
> python2 in some scripts. Convert these python2 to python3.

good point. I thought that with 5a8408bc3 ("*.py : convert python2 to python3")
we just added python3 support while keeping python, but we really also changed
the shebang to python3.

Acked-by: Petr Vorel <pvorel@suse.cz>

I guess we should state in AC_ARG_WITH([python] that we target to python3.

But the real help would be to rewrite these scripts into shell, so it can be run
also on embedded devices or other systems, which doesn't have any python.

Kind regards,
Petr

  reply	other threads:[~2019-07-25 14:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-22  8:57 [LTP] [PATCH] *.sh: convert hardcoded python2 to python3 Yi Zhao
2019-07-25 14:42 ` Petr Vorel [this message]
2019-07-26  9:43 ` Petr Vorel
2019-07-29  8:29 ` Petr Vorel

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=20190725144241.GA4568@dell5510 \
    --to=pvorel@suse.cz \
    --cc=ltp@lists.linux.it \
    /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