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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Support bundle.ios.frameworks #9992

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

stijnfrishert
Copy link

@stijnfrishert stijnfrishert commented Jun 5, 2024

This PR adds support for bundle.ios.frameworks for mobile iOS apps. This should close #9962.

Work

From what I gathered, these were the necessary steps:

  • Add theframeworks field to IosConfig, just like it exists for MacConfig.
  • Copy the value over from IosConfig to the MetaData passed into cargo_mobile2 that generates the Xcode project.

Moreover, I found a bug in the project.yml handlebars file, where the frameworks weren't unrolled correctly.

Checks

I've inspected the resulting project.yml and Xcode project and the framework is listed in the Link Binary with Libraries step.

I get a build error in the Xcode build phase running cargo tauri ios xcode-script, but I'm not sure how to figure out what's wrong. If any of the iOS devs could help me out, that would be sublime. :)

Questions

  • I am not sure if the comment within for the IosConfig::frameworks field is entirely correct. I copy/pasted it from MacConfig and replaced MacOS with iOS. Can we use full paths here as well, like it states?
  • I'm reading the frameworks from IosConfig, but do we want to check for an environment variable first (just like the development team id)? What should its name be?
  • I have no idea if this PR requires a new version. 馃し How do I decide that?

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.

[feat] v2: Frameworks aren't being picked up for iOS builds
1 participant