fixes a bug introduced in #25900 ; see #25900 (comment)
the general idea of #25900 was to use a non-root user as much as possible to avoid modifying the user's local filesystem. however, it appears the root user is needed to correctly install clang.
ci: use ci_exec_root for clang install #26574
pull josibake wants to merge 1 commits into bitcoin:master from josibake:josibake-fix-ci-permissions changing 1 files +2 −2-
josibake commented at 1:17 PM on November 25, 2022: member
-
ci: use ci_exec_root for clang install 54dd8f51ce
-
DrahtBot commented at 1:17 PM on November 25, 2022: 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.
Type Reviewers ACK hebasto - maflcko renamed this:
bugfix: ci: use ci_exec_root for clang install
ci: use ci_exec_root for clang install
on Nov 25, 2022 - DrahtBot added the label Tests on Nov 25, 2022
- hebasto approved
-
hebasto commented at 4:17 PM on November 27, 2022: member
ACK 54dd8f51ce8815a93a6aceb36dd72bd449d6b327, tested on Ubuntu 22.04.
- maflcko merged this on Nov 28, 2022
- maflcko closed this on Nov 28, 2022
- sidhujag referenced this in commit 339228819e on Dec 1, 2022
- bitcoin locked this on Nov 28, 2023
- josibake deleted the branch on Jan 26, 2024
Labels