Before this all linters were ran by /usr/bin/env python3, no matter what was used to run test/lint/all-lint.py. This change allows to use non-default Python executable for test/lint/all-lint.py and then all subprocesses will also use same Python interpreter (for example, python3.10 ./test/lint/all-lint.py). See #26792 (comment) as use case.
test: Use same Python executable for subprocesses as for `all-lint.py` #26802
pull kristapsk wants to merge 1 commits into bitcoin:master from kristapsk:all-lint-python-executable changing 1 files +2 −1-
kristapsk commented at 9:28 PM on January 3, 2023: contributor
-
Use same Python executable for subprocesses as for all-lint.py f6eadaa413
-
DrahtBot commented at 9:28 PM on January 3, 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.
Type Reviewers ACK fanquake If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Jan 3, 2023
-
maflcko commented at 9:34 PM on January 3, 2023: member
lgtm. Suggested this in #24982 (review)
- fanquake approved
-
fanquake commented at 11:55 AM on January 4, 2023: member
ACK f6eadaa413f20c1853cf1bea5141560cd5e4b15a - did not test
- maflcko merged this on Jan 4, 2023
- maflcko closed this on Jan 4, 2023
- kristapsk deleted the branch on Jan 4, 2023
- sidhujag referenced this in commit 3eb0157842 on Jan 4, 2023
- bitcoin locked this on Jan 4, 2024
