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
Use get_python_lib to look for lldb python lib dir. #7685
Conversation
@swift-ci please test |
@rintaro @modocache Hi, you both helped to review the original change that introduced this functionality + test. Could you review the above PR? Background: Attempts to make Swift build Fedora is currently hitting issues where _lldb.so end up in different dir (lib64 vs lib) and aren't correctly located by the test. Full discussion on SR-100. |
The change itself LGTM. |
@swift-ci please smoke test |
@@ -34,6 +34,7 @@ config.android_ndk_path = "@SWIFT_ANDROID_NDK_PATH@" | |||
config.android_ndk_gcc_version = "@SWIFT_ANDROID_NDK_GCC_VERSION@" | |||
|
|||
config.coverage_mode = "@SWIFT_ANALYZE_CODE_COVERAGE@" | |||
config.lldb_build_root = "@LLDB_BUILD_DIR@" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, we need this line in validation-test/lit.site.cfg.in
too. Otherwise,
File "/home/rintaro/Documents/swift-oss/swift/test/lit.cfg", line 1073, in <module>
if config.lldb_build_root != "":
AttributeError: TestingConfig instance has no attribute 'lldb_build_root'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, fixed that already, thanks.
f4cc383
to
e07eb41
Compare
@swift-ci please smoke test |
Fedora builds are currently failing due to https://github.com/apple/swift/blob/202ee90b5d62c55582cc451c0ea2854a0f6a4cca/test/Runtime/linux-fatal-backtrace.swift failing to find the lldb bindings in lldb-python-path.
Prerequisite to resolve SR-100.