malor / cpython-lldb

LLDB extension for debugging Python programs
MIT License
80 stars 4 forks source link

fix: target selection #6

Closed crepererum closed 5 years ago

crepererum commented 5 years ago

It seems that lldb.target and lldb.process are only correctly initialized after some script command was executed. We cannot expect users to do so normally, use the variable state explicitly passed instead of the global state.

This also fixes the docker image build process to not rely on the unstable version. We can probably bump this to lldb-8 again after LLVM 8 hits stable (hopefully next week).

malor commented 5 years ago

Thank you, @crepererum ! I'm sorry I missed a notification about this PR :(

This looks good to me! 👍