diff options
author | Yann E. MORIN <yann.morin.1998@free.fr> | 2018-06-03 11:08:21 +0200 |
---|---|---|
committer | Thomas Petazzoni <thomas.petazzoni@bootlin.com> | 2018-06-10 15:56:25 +0200 |
commit | 99723554c0c84002c25e98eb187fa667abee4ced (patch) | |
tree | a52bfef0dc46d4bde1d260515d29d981c4060146 /support/testing/infra/__init__.py | |
parent | a1f4421cc7d6c4c1a7678bcca477ac311d0b4116 (diff) | |
download | buildroot-99723554c0c84002c25e98eb187fa667abee4ced.tar.gz buildroot-99723554c0c84002c25e98eb187fa667abee4ced.zip |
support/testing: fix python syntax
Fix three issues with code style in our test infra:
- 'print' is now a function,
- exceptions need to be caught-assigned with the 'as' keyword,
- old-style "%s"%() formatting is deprecated.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com>
[Thomas: drop indices in format strings.]
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Diffstat (limited to 'support/testing/infra/__init__.py')
-rw-r--r-- | support/testing/infra/__init__.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/support/testing/infra/__init__.py b/support/testing/infra/__init__.py index b03e891771..e229e90852 100644 --- a/support/testing/infra/__init__.py +++ b/support/testing/infra/__init__.py @@ -34,17 +34,17 @@ def download(dldir, filename): os.makedirs(dldir) tmpfile = tempfile.mktemp(dir=dldir) - print "Downloading to {}".format(tmpfile) + print("Downloading to {}".format(tmpfile)) try: url_fh = urlopen(os.path.join(ARTIFACTS_URL, filename)) with open(tmpfile, "w+") as tmpfile_fh: tmpfile_fh.write(url_fh.read()) - except (HTTPError, URLError), err: + except (HTTPError, URLError) as err: os.unlink(tmpfile) raise err - print "Renaming from %s to %s" % (tmpfile, finalpath) + print("Renaming from {} to {}".format(tmpfile, finalpath)) os.rename(tmpfile, finalpath) return finalpath |