Skip to content

fix(coverage-ios): support universal binaries and fix podspec metadata#135

Merged
V3RON merged 1 commit into
callstackincubator:mainfrom
mfazekas:fix/ios-coverage-universal-binary
Jun 3, 2026
Merged

fix(coverage-ios): support universal binaries and fix podspec metadata#135
V3RON merged 1 commit into
callstackincubator:mainfrom
mfazekas:fix/ios-coverage-universal-binary

Conversation

@mfazekas
Copy link
Copy Markdown
Contributor

@mfazekas mfazekas commented Jun 1, 2026

Fixes two issues that prevent native iOS coverage from working.

  • llvm-cov export failed on universal (fat) binaries with -arch specifier is invalid or missing. We now detect multi-arch binaries via lipo -archs and pass -arch (preferring arm64); single-arch binaries are unaffected.
  • The HarnessCoverage podspec failed CocoaPods validation because coverage-ios/package.json lacked author and homepage. Added both fields.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

@mfazekas is attempting to deploy a commit to the Callstack Team on Vercel.

A member of the Team first needs to authorize it.

Pass -arch to llvm-cov export when the app binary is a universal/fat
Mach-O (simulator builds with arm64 + x86_64), preferring arm64.
Add author and homepage to coverage-ios package.json so the
HarnessCoverage podspec passes CocoaPods validation.
@mfazekas mfazekas force-pushed the fix/ios-coverage-universal-binary branch from bf9daf0 to f1c430c Compare June 1, 2026 20:20
@V3RON V3RON merged commit b0ced8f into callstackincubator:main Jun 3, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants