Skip to content

fix: use OpenSSL-Universal CocoaPods dependency instead of vendored xcframework#977

Merged
boorad merged 3 commits intomainfrom
fix/openssl-universal-dependency
Apr 7, 2026
Merged

fix: use OpenSSL-Universal CocoaPods dependency instead of vendored xcframework#977
boorad merged 3 commits intomainfrom
fix/openssl-universal-dependency

Conversation

@boorad
Copy link
Copy Markdown
Collaborator

@boorad boorad commented Apr 6, 2026

Summary

Replaces the self-managed OpenSSL xcframework download (curl + unzip during podspec evaluation and prepare_command) with a proper CocoaPods dependency on OpenSSL-Universal ~> 3.6.

closes #881
closes #900

Changes

  • Remove ~30 lines of brittle curl/unzip/cleanup logic from QuickCrypto.podspec
  • Add s.dependency "OpenSSL-Universal", "~> 3.6" — CocoaPods now handles downloading, caching, and linking OpenSSL
  • Use pessimistic version constraint (~> 3.6) to prevent accidental major-version jumps
  • Add prepare_command cleanup of stale OpenSSL.xcframework from pre-1.0.20 installs
  • Remove OpenSSL.xcframework from CI cache paths and .gitignore
  • Update Podfile.lock to reflect the new dependency
  • Fix example app to use workspace:* instead of pinned version

Testing

  • pod install in example/ios/ succeeds and resolves OpenSSL-Universal 3.6.0000
  • iOS build compiles and links against CocoaPods-provided OpenSSL headers
  • Existing crypto test suite passes unchanged

boorad added 3 commits April 6, 2026 19:32
…cframework

OpenSSL 3.6 is now available on CocoaPods Trunk via OpenSSL-Universal.
Replace the custom download/vendoring machinery with a standard pod
dependency, which lets CocoaPods deduplicate OpenSSL when other pods
(e.g. op-sqlite with SQLCipher) also depend on it.

Closes #881
…artifacts

Use pessimistic version (~> 3.6) to prevent accidental major-version
jumps. Add prepare_command cleanup of vendored OpenSSL.xcframework from
pre-1.0.20 installs. Remove extra blank line in .gitignore.
…space dep

Podfile.lock reflects the new OpenSSL-Universal CocoaPods dependency.
Example app now uses workspace:* to track the local package.
@boorad boorad self-assigned this Apr 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

🤖 End-to-End Test Results - Android

Status: ✅ Passed
Platform: Android
Run: 24056788945

📸 Final Test Screenshot

Maestro Test Results - android

Screenshot automatically captured from End-to-End tests and will expire in 30 days


This comment is automatically updated on each test run.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

🤖 End-to-End Test Results - iOS

Status: ✅ Passed
Platform: iOS
Run: 24056788943

📸 Final Test Screenshot

Maestro Test Results - ios

Screenshot automatically captured from End-to-End tests and will expire in 30 days


This comment is automatically updated on each test run.

@boorad boorad merged commit 86273d5 into main Apr 7, 2026
7 checks passed
@boorad boorad deleted the fix/openssl-universal-dependency branch April 7, 2026 00:02
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.

🔧 Cause: The 'Pods-MyApp' target has frameworks with conflicting names: openssl.xcframework.

1 participant