Conversation
That's because the CI for Xcode 12 doesn't support the current version of Bundler. See https://app.circleci.com/pipelines/github/Automattic/Automattic-Tracks-iOS/370/workflows/0712811a-2403-4555-8562-36914fd831f2/jobs/898
The project changes were made by the Xcode upgrade wizard.
Because of the build system changes in Xcode 12 due to the support of Apple Silicon, we now need to explicitly exclude the arm64 architecture from the build, otherwise xcodebuild will fail to build. I opened an issue in the Sodium repo, see jedisct1/swift-sodium#213, but I haven't received feedback yet. In order to unblock our progress, I created a fork of Sodium, Sodium-Fork, which incorporate the necessary changes to pass the validation. I published the fork on CocoaPods, but unfortunately CI seems to fail to find it. See this build: https://app.circleci.com/pipelines/github/Automattic/Automattic-Tracks-iOS/380/workflows/c0206d34-0f8a-4913-8b5d-b05e6dcb7c5b/jobs/936 I tried to set the `update-specs-sources` parameter to true, but it didn't make any difference. Interestingly, CI shows that the command runs, but there is no output and the execution time is 1s, too little to fetch the CocoaPods specs repo. This makes me think there might be something wrong in the setup or in how CI runs it. To address this issue, I published the pod on my private specs repo, and added its URL to the sources for the validation. Funny enough, while the spec validation fails to find the new pod, the `pod install` command in a different job has no problems with it.
jkmassel
approved these changes
Oct 8, 2020
Contributor
|
I'm sorry I'm late to the party!! changes look great!! |
mokagio
added a commit
that referenced
this pull request
Dec 8, 2020
Now that CocoaPods 1.10.0 is out, we can go back to the original Sodium library and decommission the fork we've been recently to work around some Apple Silicon related limitations. You can verify this works by running ``` bundle exec pod lib lint --allow-warnings ``` For extra context see: - jedisct1/swift-sodium#213 - jedisct1/swift-sodium#214 - #148
mokagio
added a commit
that referenced
this pull request
Dec 9, 2020
Now that CocoaPods 1.10.0 is out, we can go back to the original Sodium library and decommission the fork we've been recently to work around some Apple Silicon related limitations. You can verify this works by running ``` bundle exec pod lib lint --allow-warnings ``` For extra context see: - jedisct1/swift-sodium#213 - jedisct1/swift-sodium#214 - #148
mokagio
added a commit
that referenced
this pull request
Dec 9, 2020
Now that CocoaPods 1.10.0 is out, we can go back to the original Sodium library and decommission the fork we've been recently to work around some Apple Silicon related limitations. You can verify this works by running ``` bundle exec pod lib lint --allow-warnings ``` For extra context see: - jedisct1/swift-sodium#213 - jedisct1/swift-sodium#214 - #148
mokagio
added a commit
that referenced
this pull request
Jan 11, 2021
Now that CocoaPods 1.10.0 is out, we can go back to the original Sodium library and decommission the fork we've been recently to work around some Apple Silicon related limitations. You can verify this works by running ``` bundle exec pod lib lint --allow-warnings ``` For extra context see: - jedisct1/swift-sodium#213 - jedisct1/swift-sodium#214 - #148
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the CI config to run on Xcode 12 and iOS 14. The project changes were made by the Xcode upgrade wizard.
This fixes #149, although it's really a workaround: switch to a fork of the pod while waiting for a proper/different fix upstream, see jedisct1/swift-sodium#213.