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

Update compiling_for_ios.rst #8778

Closed
wants to merge 1 commit into from
Closed

Conversation

ivanfemia
Copy link

Based on previous statements the bin folder contains the new libraries

@AThousandShips
Copy link
Member

You should target master not 4.2, changes can then be cherry picked, see here.

If you need more information about the update process please see: here. If wanted I can fix the branch for you.

@AThousandShips AThousandShips added the needs work Needs additional work by the original author, someone else or in another repo. label Jan 15, 2024
@AThousandShips AThousandShips changed the base branch from 4.2 to master January 15, 2024 20:23
@AThousandShips
Copy link
Member

You didn't follow the instructions, you need to rebase not remove the changes manually 🙂

Based on previous statements the bin folder contains the new libraries
@AThousandShips
Copy link
Member

Cleaned up your branch for you 🙂 in the future make sure you follow the instructions

@AThousandShips AThousandShips removed the needs work Needs additional work by the original author, someone else or in another repo. label Jan 20, 2024
$ cp libgodot.ios.template_debug.arm64.a ios_xcode/libgodot.ios.debug.xcframework/ios-arm64/libgodot.a
$ lipo -create libgodot.ios.template_debug.arm64.simulator.a libgodot.ios.template_debug.x86_64.simulator.a -output ios_xcode/libgodot.ios.debug.xcframework/ios-arm64_x86_64-simulator/libgodot.a
$ cp bin/libgodot.ios.template_debug.arm64.a ios_xcode/libgodot.ios.debug.xcframework/ios-arm64/libgodot.a
$ lipo -create bin/libgodot.ios.template_debug.arm64.simulator.a bin/libgodot.ios.template_debug.x86_64.simulator.a -output ios_xcode/libgodot.ios.debug.xcframework/ios-arm64_x86_64-simulator/libgodot.a

$ cp libgodot.ios.template_release.arm64.a ios_xcode/libgodot.ios.release.xcframework/ios-arm64/libgodot.a
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
$ cp libgodot.ios.template_release.arm64.a ios_xcode/libgodot.ios.release.xcframework/ios-arm64/libgodot.a
$ cp bin/libgodot.ios.template_release.arm64.a ios_xcode/libgodot.ios.release.xcframework/ios-arm64/libgodot.a

You forgot a bin here I'd say

@AThousandShips AThousandShips added enhancement area:contributing Issues and PRs related to the Contributing/Development section of the documentation labels Jan 20, 2024
@ivanfemia
Copy link
Author

Cleaned up your branch for you 🙂 in the future make sure you follow the instructions

Sorry I missed your previous message
Thanks!

@mhilbrunner mhilbrunner requested a review from bruvzg January 24, 2024 10:20
Copy link
Member

@bruvzg bruvzg left a comment

Choose a reason for hiding this comment

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

I would change

$ cp -r misc/dist/ios_xcode .

to

$ cd bin
$ cp -r ../misc/dist/ios_xcode .

instead, since it's not a good idea to do it in the root folder (it will interfere with git actions later), bin is untracked.

@skyace65 skyace65 added the platform:ios Issues or PRs related specifically to ios label Sep 5, 2024
@skyace65
Copy link
Contributor

skyace65 commented Jan 21, 2025

Closing this PR since with #10096 merged this change is no longer needed.

Thank you for taking the time to make this PR though and please consider contributing in the future if you find something you want to update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
archived area:contributing Issues and PRs related to the Contributing/Development section of the documentation enhancement platform:ios Issues or PRs related specifically to ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants