ci: Use Python virtual environment in "x86_64-macos-native" job #1771

pull hebasto wants to merge 1 commits into bitcoin-core:master from hebasto:251107-python-env changing 1 files +7 −12

hebasto commented at 2:20 PM on November 7, 2025: member
Fixes #1768.

This PR explicitly sets up a virtual environment instead of using uv, as suggested here, for the following reasons:
1. It does not require granting a third-party action access to the repository.
2. This approach is already used in other parts of the CI framework (it’s unclear why it was missed in #1359 in the first place).
real-or-random added the label bug on Nov 7, 2025
real-or-random added the label ci on Nov 7, 2025
in .github/workflows/ci.yml:472 in d3ae054474
```
 468 | @@ -469,9 +469,14 @@ jobs:
 469 |          env: ${{ matrix.env_vars }}
 470 |          run: ./ci/ci.sh
 471 |  
 472 | -      - name: Symbol check
 473 | +      - &SYMBOL_CHECK
```
real-or-random commented at 3:03 PM on November 7, 2025:

nit:

- &SYMBOL_CHECK_MACOS

because it's specific to macOS

avoid naming confusing with the env variable SYMBOL_CHECK
hebasto commented at 3:21 PM on November 7, 2025:

Thanks! Reworked per your feedback.
real-or-random approved
real-or-random commented at 3:05 PM on November 7, 2025: contributor

utACK d3ae05447429d66b065ce3982fb0c5864c18bb53

Let's wait for green.

Thanks for fixing CI so quickly. :)

I agree, that's simpler and does the job. I still think uv is a good idea for devs who want to run the script locally, but that's a different story then.
ci: Use Python virtual environment in "x86_64-macos-native" job f252da7e6e
hebasto force-pushed on Nov 7, 2025
real-or-random approved
real-or-random commented at 7:21 PM on November 7, 2025: contributor

ACK f252da7e6e1bbd0082fd43c4ffabfa90abafb248
real-or-random merged this on Nov 7, 2025
real-or-random closed this on Nov 7, 2025
hebasto deleted the branch on Nov 7, 2025

Contributors

hebasto

real-or-random

Labels

bug ci

Linked (view graph)

#1359 Fix symbol visibility issues, add test for it #1768 ci: python -m pip install fails with error: externally-managed-environment