Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build #99

Merged
merged 12 commits into from
May 10, 2023
Merged

Fix build #99

merged 12 commits into from
May 10, 2023

Conversation

stackchain
Copy link
Member

  • Bumped min sdk version + removed hack Big Sur

@stackchain stackchain added the fix label Apr 20, 2023
@stackchain stackchain self-assigned this Apr 20, 2023
@stackchain stackchain requested a review from lisicky April 20, 2023 23:25
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stackchain stackchain requested review from lisicky and removed request for lisicky May 6, 2023 00:31
@@ -33,10 +25,14 @@ cd "${ROOT_DIR}"
if [[ "$TARGET_DEVICE_PLATFORM_NAME" == "iphonesimulator" ]] && [[ "$MAC_CURRENT_ARCH" == "arm64" ]]; then
# If we're building for the arm simulator on an M1 Mac, we need to use the x86_64-apple-ios-sim target.
# Otherwise, lipo will compile for arm64 iphone that can't run on the simulator.
ACTUAL_SDK_PATH=$(xcrun --sdk iphonesimulator --show-sdk-path)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stackchain I found more general way to retrieve sdk path. Revert it if won't work for you.

@stackchain stackchain merged commit 72b5dd0 into main May 10, 2023
@stackchain stackchain deleted the chore/bump-ios-min-sdk branch May 10, 2023 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants