]> granicus.if.org Git - python/commitdiff
bpo-37023: Skip test_gdb under PGO (GH-13555)
authorSteve Dower <steve.dower@python.org>
Fri, 24 May 2019 20:00:04 +0000 (13:00 -0700)
committerGitHub <noreply@github.com>
Fri, 24 May 2019 20:00:04 +0000 (13:00 -0700)
Lib/test/test_gdb.py

index 4d1ce4ed96c06d2d441c24bb4735421384077d63..dbcb5983e9ba1d7902bfc7b753aaff016eeed002 100644 (file)
@@ -52,6 +52,10 @@ if 'Clang' in platform.python_compiler() and sys.platform == 'darwin':
     raise unittest.SkipTest("test_gdb doesn't work correctly when python is"
                             " built with LLVM clang")
 
+if ((sysconfig.get_config_var('PGO_PROF_USE_FLAG') or 'xxx') in
+    (sysconfig.get_config_var('PY_CORE_CFLAGS') or '')):
+    raise unittest.SkipTest("test_gdb is not reliable on PGO builds")
+
 # Location of custom hooks file in a repository checkout.
 checkout_hook_path = os.path.join(os.path.dirname(sys.executable),
                                   'python-gdb.py')
@@ -272,7 +276,7 @@ class DebuggerTests(unittest.TestCase):
         # gdb can insert additional '\n' and space characters in various places
         # in its output, depending on the width of the terminal it's connected
         # to (using its "wrap_here" function)
-        m = re.match(r'.*#0\s+builtin_id\s+\(self\=.*,\s+v=\s*(.*?)\)\s+at\s+\S*Python/bltinmodule.c.*',
+        m = re.match(r'.*#0\s+builtin_id\s+\(self\=.*,\s+v=\s*(.*?)?\)\s+at\s+\S*Python/bltinmodule.c.*',
                      gdb_output, re.DOTALL)
         if not m:
             self.fail('Unexpected gdb output: %r\n%s' % (gdb_output, gdb_output))