Skip to content

Commit

Permalink
[lldb/Test] Skip script interpreter tests reading from stdin for lldb…
Browse files Browse the repository at this point in the history
…-repro

The reproducers currently only shadow the command interpreter. It would
be possible to make it work for the Lua interpreter which uses the
IOHandlerEditline under the hood, but the Python one runs a REPL in
Python itself so there's no (straightforward) way to shadow that.

Given that we already capture any API calls, this isn't super high on my
list of priorities.
  • Loading branch information
JDevlieghere committed Jan 22, 2020
1 parent 4c2b0a6 commit 6ae61f7
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 0 deletions.
2 changes: 2 additions & 0 deletions lldb/test/Shell/ScriptInterpreter/Lua/bindings.test
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
# REQUIRES: lua
# UNSUPPORTED: lldb-repro
#
# RUN: cat %s | %lldb --script-language lua 2>&1 | FileCheck %s
script
debugger = lldb.SBDebugger.Create()
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# REQUIRES: lua
# UNSUPPORTED: lldb-repro
#
# This tests that the convenience variables are not nil. Given that there is no
# target we only expect the debugger to be valid.
Expand Down
1 change: 1 addition & 0 deletions lldb/test/Shell/ScriptInterpreter/Python/crashlog.test
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
# -*- python -*-
# REQUIRES: system-darwin
# UNSUPPORTED: lldb-repro
# DEBUG: cd %S/../../../../examples/python && cat %s | %lldb && false
# RUN: cd %S/../../../../examples/python && cat %s | %lldb | FileCheck %s
# CHECK-LABEL: {{S}}KIP BEYOND CHECKS
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# REQUIRES: python
# UNSUPPORTED: lldb-repro
#
# Test that the scripting language argument to "breakpoint command" is honored
# even if the global scripting language is different.
Expand Down

0 comments on commit 6ae61f7

Please sign in to comment.