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

Allow example app to be built in arm64 #180

Merged
merged 4 commits into from
Oct 29, 2024

Conversation

tung2744
Copy link
Contributor

ref DEV-2121

@@ -6,7 +6,7 @@ target 'ios_example' do
use_frameworks!

# Pods for ios_example
pod 'WechatOpenSDK', '~> 2.0.4'
pod 'WechatOpenSDK-XCFramework', '~> 2.0.4'
Copy link
Contributor

Choose a reason for hiding this comment

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

Please include this link as comment above this line. https://developers.weixin.qq.com/community/develop/doc/000c62b9b0ccf8c1d65fa90b751001?blockType=1

From the link, it says that we need to change the header include macro from "" to <>. I wonder why we did not do so.

For the removal of the .a and .h files, I guess Cocoapods has handled that for us. So no action is required from us.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

From the link, it says that we need to change the header include macro from "" to <>. I wonder why we did not do so.

It is because I found that "" seems also working. Let me change it back to <> according to the doc.

@louischan-oursky louischan-oursky self-requested a review October 29, 2024 04:41
@louischan-oursky louischan-oursky merged commit 617dde9 into authgear:main Oct 29, 2024
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