Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lldb][test] elf-memory.test requires LLDB build with Python support #111810

Merged

Conversation

asb
Copy link
Contributor

@asb asb commented Oct 10, 2024

Otherwise it fails with "error: Embedded script interpreter unavailable. LLDB was built without scripting language support."

Otherwise it fails with "error: Embedded script interpreter unavailable.
LLDB was built without scripting language support."
@asb asb requested a review from DavidSpickett October 10, 2024 10:57
@asb asb requested a review from JDevlieghere as a code owner October 10, 2024 10:57
@llvmbot llvmbot added the lldb label Oct 10, 2024
@llvmbot
Copy link
Member

llvmbot commented Oct 10, 2024

@llvm/pr-subscribers-lldb

Author: Alex Bradbury (asb)

Changes

Otherwise it fails with "error: Embedded script interpreter unavailable. LLDB was built without scripting language support."


Full diff: https://github.com/llvm/llvm-project/pull/111810.diff

1 Files Affected:

  • (modified) lldb/test/Shell/ObjectFile/ELF/elf-memory.test (+1-1)
diff --git a/lldb/test/Shell/ObjectFile/ELF/elf-memory.test b/lldb/test/Shell/ObjectFile/ELF/elf-memory.test
index 0b1c01486a4b43..75a68edd2d3496 100644
--- a/lldb/test/Shell/ObjectFile/ELF/elf-memory.test
+++ b/lldb/test/Shell/ObjectFile/ELF/elf-memory.test
@@ -1,4 +1,4 @@
-// REQUIRES: system-linux, native
+// REQUIRES: system-linux, native, python
 
 // This test verifies that loading an ELF file from memory works and the new
 // features that were added when loading from memory work like:

Copy link
Collaborator

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DavidSpickett DavidSpickett merged commit 99cec7f into llvm:main Oct 10, 2024
7 of 8 checks passed
DanielCChen pushed a commit to DanielCChen/llvm-project that referenced this pull request Oct 16, 2024
…lvm#111810)

Otherwise it fails with "error: Embedded script interpreter unavailable.
LLDB was built without scripting language support."
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants