fix: complete SPM migration#882
Conversation
… update gitignore
…ir name for local path deps)
…tion with explanatory comment
…d build phase to replace SPM thin wrapper
…n model dylib + build script
…rny/google-mlkit-swiftpm - Delete all 22 CocoaPods podspecs across all packages - Update all 11 Package.swift files to reference arrrrny/google-mlkit-swiftpm - Add per-language OCR resource bundles (Latin, Chinese, Japanese, Korean, Devanagari) - Set -ObjC linker flag in example apps for ObjC category support - Create focused text_recognition_example app - Update Package.resolved files for SPM consistency - Clean up stale CocoaPods artifacts and build caches - Fixes text recognition 'Invalid model path' crash (d-date#106) - Fixes dyld flat-namespace crash on TestFlight/Release builds
…en 9.0.0-1 tag The 9.0.0-1 tag on arrrrny/google-mlkit-swiftpm points to an old commit with duplicate binary target entries. Use the fixed commit directly.
…te files These files were accidentally generated by flutter create when the example app's iOS configuration was regenerated. They are not part of the iOS SPM migration and add noise to the PR.
|
This is much cleaner, you can see all at once. I tested on device, it works. |
|
@arrrrny : fix this: |
|
merging to test |
|
@arrrrny: Thanks for the PR and your contributions! After merging the PR, I was able to run the app, but not all the plugins are working yet (Face Detection, for instance). We need to continue working on this branch, as we cannot release it until everything is fully functional. If you would like to continue contributing, your help is absolutely welcome. One quick note regarding your PRs: I noticed you were overriding the DEVELOPMENT_TEAM and PRODUCT_BUNDLE_IDENTIFIER directly within the Xcode project settings. I have been reverting those changes after merging. I just added instructions to the example app's README so developers know how to configure these for local testing without committing the changes. Also, for future PRs, please sync your local branch with the latest version of If you don't have the time to contribute further right now, feel free to use this plugin in your Flutter project by pointing your pubspec.yaml to this branch, as shown in this example: Once again, thank you for contributing! |
|
you are welcome @fbernaly I am aware, I focused on text recognition since I need in my app and I am sure most others need and can migrate referencing to this merged git branch. make sure to git reference the mlkit commons as well. If any other devs are willing to take the rest of the work, that would be great, since this was a tedious job and I paved the road for them. Otherwise I will probably complete the rest |
2 participants
No description provided.