This fixes some cases, i.e under --no-install-recommends, where libclang-rt-dev wouldn't be installed, and configuring would then fail.
Followup to #27444.
ci: explicitly install libclang-rt-dev in valgrind jobs #27459
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:explicit_libclang_rt_dev changing 2 files +2 −2-
fanquake commented at 10:23 AM on April 13, 2023: member
-
2c60826b50
ci: explicitly install libclang-rt-dev in valgrind jobs
This fixes some cases, i.e under --no-install-recommends, where libclang-rt-dev wouldn't be installed, and configuring would then fail. Followup to #27444.
-
DrahtBot commented at 10:23 AM on April 13, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
- fanquake requested review from maflcko on Apr 13, 2023
- DrahtBot added the label Tests on Apr 13, 2023
- maflcko approved
-
maflcko commented at 10:33 AM on April 13, 2023: member
lgtm. It is good to fix bugs, but it would be better to fix all related bugs. Maybe #27436 (comment) is related?
-
fanquake commented at 11:14 AM on April 13, 2023: member
Maybe #27436 (comment) is related?
I'm pretty certain this is https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1005341, which isn't related to this change, but seems to have been broken in Debian for libc++ & fuzzing for LLVM versions 12 through 16. I'll file a new issue with the libc++ 16 package, and debug futher.
- fanquake merged this on Apr 13, 2023
- fanquake closed this on Apr 13, 2023
- fanquake deleted the branch on Apr 13, 2023
- sidhujag referenced this in commit c369c43781 on Apr 13, 2023
- bitcoin locked this on Apr 12, 2024
Labels