perf python: Make clang_has_option() work on Python 3
authorEduardo Habkost <[email protected]>
Fri, 5 Oct 2018 20:40:57 +0000 (17:40 -0300)
committerArnaldo Carvalho de Melo <[email protected]>
Mon, 8 Oct 2018 17:30:44 +0000 (14:30 -0300)
Use a bytes literal so it works with Python 3's version of Popen().
Note that the b"..." syntax requires Python 2.6+.

Signed-off-by: Eduardo Habkost <[email protected]>
Cc: Adrian Hunter <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Wang Nan <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
tools/perf/util/setup.py

index 1942f6dd24f66af5067167d6c31e3c44cb04be34..261a55e7e1b218bcbffc440be37fd791ae155ac3 100644 (file)
@@ -5,7 +5,7 @@ from subprocess import Popen, PIPE
 from re import sub
 
 def clang_has_option(option):
-    return [o for o in Popen(['clang', option], stderr=PIPE).stderr.readlines() if "unknown argument" in o] == [ ]
+    return [o for o in Popen(['clang', option], stderr=PIPE).stderr.readlines() if b"unknown argument" in o] == [ ]
 
 cc = getenv("CC")
 if cc == "clang":