This is an archive of the discontinued LLVM Phabricator instance.

Fix ImportError in lldb-mi tests on Linux
ClosedPublic

Authored by ki.stfu on Feb 24 2015, 3:30 AM.

Details

Summary

This patch fixes the following error:

 File "/home/llvmbb/llvm-build-dir/lldb-x86_64-clang-ubuntu-14.04/llvm/tools/lldb/test/tools/lldb-mi/signal/TestMiSignal.py", line 5, in <module>
    import lldbmi_testcase
ImportError: No module named lldbmi_testcase

Diff Detail

Event Timeline

ki.stfu updated this revision to Diff 20589.Feb 24 2015, 3:30 AM
ki.stfu retitled this revision from to Fix ImportError in lldb-mi tests on Linux.
ki.stfu updated this object.
ki.stfu edited the test plan for this revision. (Show Details)
ki.stfu added reviewers: abidh, clayborg, vharron.
ki.stfu added subscribers: abidh, clayborg, vharron, Unknown Object (MLST).
ki.stfu updated this revision to Diff 20590.EditedFeb 24 2015, 3:44 AM

Resolve path to lldbmi_testcase.py

In this patch I use:

sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))

instead of

sys.path.append(os.path.join(os.path.dirname(os.path.realpath(__file__)), '..'))
abidh accepted this revision.Feb 24 2015, 4:41 AM
abidh edited edge metadata.

make check-lldb is not showing any lldbmi failure after this change. So ok for me. But a person more familiar with python needs to have a look.

This revision is now accepted and ready to land.Feb 24 2015, 4:41 AM
clayborg accepted this revision.Feb 24 2015, 9:48 AM
clayborg edited edge metadata.
ki.stfu closed this revision.Feb 24 2015, 10:23 PM