Skip to content

Conversation

@AshleyDumaine
Copy link
Collaborator

What this PR does / why we need it: Without this dependency in the devbox.json, mac users will run into errors with pkg/mod/golang.org/[email protected] such as:

(devbox) (⎈|N/A:N/A) ~/cluster-api-provider-linode   main  make kind-cluster
go install tool
# sigs.k8s.io/kustomize/kustomize/v5
/Users/adumaine/go/pkg/mod/golang.org/[email protected]/pkg/tool/darwin_arm64/link: running clang failed: exit status 1
/nix/store/zzx6cfl99zknfrj9v7lmrshwzslfjv26-clang-wrapper-19.1.7/bin/clang -arch arm64 -Wl,-headerpad,1144 -Wl,-flat_namespace -Wl,-bind_at_load -o $WORK/b1458/exe/a.out -Qunused-arguments /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/go.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000000.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000001.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000002.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000003.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000004.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000005.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000006.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000007.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-368
3341817/000008.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000009.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh
0000gq/T/go-link-3683341817/000010.o /var/folders/xx/4d8pt8m13w38gqz9nn9wf6vh0000gq/T/go-link-3683341817/000011.o -lresolv -O2 -g -O2
{
-g -framework CoreFoundation -framework CoreFoundation -framework Security
Undefined symbols for architecture arm64:
  "_SecTrustCopyCertificateChain", referenced from:
      _crypto/x509/internal/macos.x509_SecTrustCopyCertificateChain_trampoline.abi0 in go.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

See NixOS/nixpkgs#433688 (comment)

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

TODOs:

  • squashed commits
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Oct 13, 2025
@codecov
Copy link

codecov bot commented Oct 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.94%. Comparing base (a92e231) to head (3ae36e5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #916   +/-   ##
=======================================
  Coverage   65.94%   65.94%           
=======================================
  Files          71       71           
  Lines        7958     7958           
=======================================
  Hits         5248     5248           
  Misses       2408     2408           
  Partials      302      302           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AshleyDumaine AshleyDumaine enabled auto-merge (squash) October 13, 2025 17:37
@AshleyDumaine AshleyDumaine merged commit ed99e51 into main Oct 13, 2025
16 checks passed
@AshleyDumaine AshleyDumaine deleted the apple-sdk branch October 13, 2025 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants