diff mbox

Make build-many-glibcs.py work on python3.2

Message ID 1484079117-17506-1-git-send-email-adhemerval.zanella@linaro.org
State Accepted
Commit 64235ccc11175e6d4186a8fa911816cd7674d453
Headers show

Commit Message

Adhemerval Zanella Jan. 10, 2017, 8:11 p.m. UTC
From: Szabolcs Nagy <szabolcs.nagy@arm.com>


This is an updated version based on Joseph's review [1] with comments
fixed.

This patch make build-many-glibcs.py work with python 3.2 by
adding fallback implementation to python 3.5 facilities if they
are not present.

Checked building a x86_64-linux-gnu toolchain with python 3.2
(ubuntu 12.04).

2016-11-22  Szabolcs Nagy  <szabolcs.nagy@arm.com>

	* scripts/build-many-glibcs.py (os.cpu_count): Add compatibility definition.
	(re.fullmatch, subprocess.run): Likewise.

[1] https://sourceware.org/ml/libc-alpha/2016-11/msg00799.html

---
 ChangeLog                    |  5 +++++
 scripts/build-many-glibcs.py | 37 +++++++++++++++++++++++++++++++++++++
 2 files changed, 42 insertions(+)

-- 
2.7.4

Comments

Joseph Myers Jan. 10, 2017, 9:06 p.m. UTC | #1
On Tue, 10 Jan 2017, Adhemerval Zanella wrote:

> From: Szabolcs Nagy <szabolcs.nagy@arm.com>

> 

> This is an updated version based on Joseph's review [1] with comments

> fixed.

> 

> This patch make build-many-glibcs.py work with python 3.2 by

> adding fallback implementation to python 3.5 facilities if they

> are not present.


OK.

-- 
Joseph S. Myers
joseph@codesourcery.com
Florian Weimer Jan. 11, 2017, 10:37 a.m. UTC | #2
* Adhemerval Zanella:

> +    re.fullmatch = lambda p,s,f=0: re.match(p+"\Z",s,f)


Please use: p + r"\Z" or p + "\\Z"

I'm surprised "\Z" isn't a syntax error.
Szabolcs Nagy Jan. 11, 2017, 12:33 p.m. UTC | #3
On 10/01/17 20:11, Adhemerval Zanella wrote:
> This is an updated version based on Joseph's review [1] with comments

> fixed.


thanks
diff mbox

Patch

diff --git a/ChangeLog b/ChangeLog
index ce632d7..e39b9f8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@ 
+2017-01-10  Szabolcs Nagy  <szabolcs.nagy@arm.com>
+
+	* scripts/build-many-glibcs.py (os.cpu_count): Add compatibility definition.
+	(re.fullmatch, subprocess.run): Likewise.
+
 2016-01-10  Siddhesh Poyarekar  <siddhesh@sourceware.org>
 
 	* elf/dl-tunables.c (tunables_unsetenv): Remove function.
diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py
index 60a7874..61d2121 100755
--- a/scripts/build-many-glibcs.py
+++ b/scripts/build-many-glibcs.py
@@ -49,6 +49,43 @@  import sys
 import time
 import urllib.request
 
+try:
+    os.cpu_count
+except:
+    import multiprocessing
+    os.cpu_count = lambda: multiprocessing.cpu_count()
+
+try:
+    re.fullmatch
+except:
+    re.fullmatch = lambda p,s,f=0: re.match(p+"\Z",s,f)
+
+try:
+    subprocess.run
+except:
+    class _CompletedProcess:
+        def __init__(self, args, returncode, stdout=None, stderr=None):
+            self.args = args
+            self.returncode = returncode
+            self.stdout = stdout
+            self.stderr = stderr
+
+    def _run(*popenargs, input=None, timeout=None, check=False, **kwargs):
+        assert(timeout is None)
+        with subprocess.Popen(*popenargs, **kwargs) as process:
+            try:
+                stdout, stderr = process.communicate(input)
+            except:
+                process.kill()
+                process.wait()
+                raise
+            returncode = process.poll()
+            if check and returncode:
+                raise subprocess.CalledProcessError(returncode, popenargs)
+        return _CompletedProcess(popenargs, returncode, stdout, stderr)
+
+    subprocess.run = _run
+
 
 class Context(object):
     """The global state associated with builds in a given directory."""