Page MenuHomePhabricator

[secp256k1] ci, gha: Run "x86_64: macOS Ventura" job on GitHub Actions
ClosedPublic

Authored by PiRK on Jan 30 2026, 17:06.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Tail of the build log:

Build 'Bitcoin ABC Diffs / Diff Testing' #N/A, branch 'refs/tags/phabricator/diff/58155'
Triggered 2026-01-30 17:06:20 by 'Phabricator Staging (phabricator-staging)'
Started 2026-01-30 17:06:26 on agent 'N/A'
Finished 2026-01-30 17:06:26 with status FAILURE 'Snapshot dependency failed to start: Automated Deployments / Bitcoin ABC Infra / Bitcoin-ABC Infra Checkout'
VCS revisions: 'BitcoinABC_BitcoinAbcStaging' (Git, instance id 22): '7efd9d11ee757a7395d216bbaabfddbb3ae97169' (branch: 'refs/tags/phabricator/diff/58155', checkout rules: '+:. => ./bitcoin-abc')
TeamCity URL https://build.bitcoinabc.org/buildConfiguration/BitcoinABC_BitcoinAbcStaging/1058001 
TeamCity server version is 2025.03.1 (build 186125), server timezone: GMT (UTC)

[17:06:20]W: bt15 (5s)
[17:06:20]i: TeamCity server version is 2025.03.1 (build 186125)
[17:06:20] : Finalize build settings
[17:06:20] : Collecting changes in 2 VCS roots
[17:06:20] :	 [Collecting changes in 2 VCS roots] VCS Root details
[17:06:20] :		 [VCS Root details] "Bitcoin ABC Staging" {instance id=22, parent internal id=3, parent id=BitcoinABC_BitcoinAbcStaging, description: "ssh://vcs@reviews.bitcoinabc.org:2221/source/bitcoin-abc-staging.git#refs/heads/master"}
[17:06:20] :		 [VCS Root details] "abc-infrastructure" {instance id=24, parent internal id=7, parent id=AutomatedDeployments_BitcoinAbcDeveloperTools_AbcInfrastructure, description: "ssh://vcs@reviews.bitcoinabc.org:2221/source/infrastructure.git#refs/heads/master"}
[17:06:20]i: Loading current repository state for VCS root 'Bitcoin ABC Staging' (4s)
[17:06:20]i:	 [Loading current repository state for VCS root 'Bitcoin ABC Staging'] VCS root 'Bitcoin ABC Staging': git -c credential.helper= -c credential.helper=/opt/teamcity/temp/credHelper9401844356104982014.sh ls-remote origin
[17:06:21]i:	 [Loading current repository state for VCS root 'Bitcoin ABC Staging'] VCS root 'Bitcoin ABC Staging': Warning: Permanently added '[reviews.bitcoinabc.org]:2221' (ED25519) to the list of known hosts.
[17:06:21]i:	 [Loading current repository state for VCS root 'Bitcoin ABC Staging'] Loading current repository state for VCS root 'abc-infrastructure' (4s)
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': git -c credential.helper= -c credential.helper=/opt/teamcity/temp/credHelper6749885330126589267.sh ls-remote origin
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': Warning: Permanently added '[reviews.bitcoinabc.org]:2221' (ED25519) to the list of known hosts.
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': vcs@reviews.bitcoinabc.org: Permission denied (publickey,keyboard-interactive).
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': fatal: Could not read from remote repository.
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': 
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': Please make sure you have the correct access rights
[17:06:21]i:		 [Loading current repository state for VCS root 'abc-infrastructure'] VCS root 'abc-infrastructure': and the repository exists.
[17:06:22]i: Waiting for completion of current operations for the VCS root 'Bitcoin ABC Staging'
[17:06:25]i: Detecting changes in VCS root 'Bitcoin ABC Staging' (used in 'Diff Testing')
[17:06:25]i: Will collect changes for 'Bitcoin ABC Staging' starting from revision cdf333d1118af2cd01de80e11ab7405291340f49
[17:06:25] : Compute revision for 'Bitcoin ABC Staging'
[17:06:25] :	 [Compute revision for 'Bitcoin ABC Staging'] Upper limit revision: 7efd9d11ee757a7395d216bbaabfddbb3ae97169
[17:06:25]i:	 [Compute revision for 'Bitcoin ABC Staging'] MaxModId = 90643
[17:06:25] :	 [Compute revision for 'Bitcoin ABC Staging'] Computed revision: 7efd9d11ee757a7395d216bbaabfddbb3ae97169
[17:06:26]W: Build was removed from the queue with comment: This build has not been started because some of the builds it depends on failed to start

Most jobs are passing, still trying to figure out distcheck and 'CPPFLAGS=-DVERIFY'
https://github.com/PiRK/secp256k1/actions/runs/21523227382/job/62019908554

right now these jobs seem stuck

PiRK edited the summary of this revision. (Show Details)

rebase on D19515 to fix the DVERIFY job, comment out the job that does not work for technical reasons (no space left on device

PiRK published this revision for review.Jan 31 2026, 14:16

I don't know yet how to fix the remaining commented-out task, but it seems to be unrelated to our code and related to technical limitations of github actions. I think we should go ahead and land this to get back some CI coverage for the standalone secp256k1 build, now that Cirrus is not working due to a monthly rate-limit.

src/secp256k1/.github/workflows/ci.yml
10 ↗(On Diff #58165)

the env section would be introduced in https://github.com/bitcoin-core/secp256k1/pull/1389/changes, which i skipped for now because our CMake build does not seems ready for native Windows.

This revision is now accepted and ready to land.Jan 31 2026, 22:02