This is an archive of the discontinued LLVM Phabricator instance.

[lldb/docs] Clarify python/swig version incompatibility
ClosedPublic

Authored by labath on Oct 6 2020, 8:44 AM.

Details

Summary

The problematic combo is a debug python>=3.7 && swig<4.0.

Diff Detail

Event Timeline

labath created this revision.Oct 6 2020, 8:44 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 6 2020, 8:44 AM
labath requested review of this revision.Oct 6 2020, 8:44 AM
labath added a comment.Oct 6 2020, 8:47 AM

Jonas, the notion that 3.7 is incompatible comes from ae7389116. Do you happen to remember what was the issue you ran into?

I find it unlikely that 3.7 is incompatible but 3.8 is ok, and I think you probably ran into https://github.com/swig/swig/issues/1321 (which I recently ran into as well).

tatyana-krasnukha accepted this revision.Oct 6 2020, 9:29 AM

I used Swig 3.0.12 + cherry-picked commit that fixes that bug (https://github.com/swig/swig/issues/1321) on Windows. It worked well with both Python 3.7 and 3.8.

The only issue I still have with Python 3.8 and later is importing liblldb.dll from a python script (https://llvm.org/pr46891) while running the test-suite.

This revision is now accepted and ready to land.Oct 6 2020, 9:29 AM

Jonas, the notion that 3.7 is incompatible comes from ae7389116. Do you happen to remember what was the issue you ran into?

I find it unlikely that 3.7 is incompatible but 3.8 is ok, and I think you probably ran into https://github.com/swig/swig/issues/1321 (which I recently ran into as well).

That's very possible. I don't remember the details, but when I used 3.7 on Windows all the tests were failing. I asked someone (it might have been @compnerd or @stella.stamenova) who told me it was a known issue.

lldb/docs/resources/build.rst
81

The rest of this page seems to be using LLDB rather than lldb. I have no preference as long as things are consistent.

JDevlieghere accepted this revision.Oct 6 2020, 9:41 AM
amccarth accepted this revision.Oct 6 2020, 10:10 AM

Yes, I also (independently) discovered this problem. I probably have notes somewhere with sources for details about the incompatibility. I believe I also brought it up on the lldb-dev list.

labath added a comment.Oct 7 2020, 6:20 AM

Yes, I also (independently) discovered this problem.

Yeah, that seems to be a fairly common problem. I'm gonna try to add some kind of a cmake warning about this.

lldb/docs/resources/build.rst
81

sounds good.

This revision was automatically updated to reflect the committed changes.