diff options
| author | David Spickett <david.spickett@linaro.org> | 2025-01-29 09:56:41 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-01-29 09:56:41 +0000 |
| commit | 9ea64dd8781328d831d7c69a586f0c84dece1c11 (patch) | |
| tree | c2fbaec5a5874a01e06695db85cf12071e1b9891 /lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp | |
| parent | 776ef9d1bec66875c554e8a5bd0e3ae8c9543d9a (diff) | |
[lldb] Make Python >= 3.8 required for LLDB 21 (#124735)
As decided on
https://discourse.llvm.org/t/rfc-lets-document-and-enforce-a-minimum-python-version-for-lldb/82731.
LLDB 20 recommended `>= 3.8` but did not remove support for anything
earlier. Now we are in what will become LLDB 21, so I'm removing that
support and making
`>= 3.8` required.
See https://docs.python.org/3/c-api/apiabiversion.html#c.PY_VERSION_HEX
for the format of PY_VERSION_HEX.
Diffstat (limited to 'lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp')
| -rw-r--r-- | lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp b/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp index 365ebc8e52c2..2dd92fc00fea 100644 --- a/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp +++ b/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp @@ -760,10 +760,6 @@ class NewStyle(object): EXPECT_EQ(arginfo.get().max_positional_args, 3u); } -#if PY_VERSION_HEX >= 0x03030000 - - // the old implementation of GetArgInfo just doesn't work on builtins. - { auto builtins = PythonModule::BuiltinsModule(); auto hex = As<PythonCallable>(builtins.GetAttribute("hex")); @@ -772,8 +768,6 @@ class NewStyle(object): ASSERT_THAT_EXPECTED(arginfo, llvm::Succeeded()); EXPECT_EQ(arginfo.get().max_positional_args, 1u); } - -#endif } TEST_F(PythonDataObjectsTest, TestScript) { |