Skip to content

fix: Flutter Kotlin issues, add SPM support, verify Flutter 3.44.1#305

Open
kmmhanan wants to merge 6 commits into
tensorflow:mainfrom
kmmhanan:main
Open

fix: Flutter Kotlin issues, add SPM support, verify Flutter 3.44.1#305
kmmhanan wants to merge 6 commits into
tensorflow:mainfrom
kmmhanan:main

Conversation

@kmmhanan

@kmmhanan kmmhanan commented Jun 6, 2026

Copy link
Copy Markdown

Release 0.12.2

  • Critical fix: resolved Flutter Kotlin capability issues on Android
  • Critical fix: added Swift Package Manager (SPM) support for iOS/macOS
  • Moved plugin source files into ios/ and macos/ folders to enable proper SPM integration
  • Verified compatibility with Flutter 3.44.1

@kmmhanan kmmhanan changed the title fix: Flutter Kotlin issues, add SPM support, verify Flutter 3.44.0 fix: Flutter Kotlin issues, add SPM support, verify Flutter 3.44.1 Jun 8, 2026

@jpdup jpdup left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tested/verified your PR #305 on our builds of iOS and macOS and Android

@jpdup jpdup left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I approved this PR for the Android portion

Tested/verified your PR #305 on our builds of iOS and macOS and Android

The PR builds but does not actually works at runtime, heres the details:

Logging is now in place. Now regarding the actual error — dlsym(RTLD_DEFAULT): symbol not found for 'TfLiteModelCreate' — this means the TensorFlowLiteC framework is not being linked into the iOS app. This is exactly the problem PR #305 was meant to fix via SPM. Let me check what that PR actually provides:

Now the root cause is clear. PR #305 adds a stub Package.swift that just registers the Flutter channel — but it doesn't declare TensorFlowLiteC as a binary dependency. The CocoaPods version pulled in TensorFlowLiteSwift pod (which contains the C library). SPM has no equivalent yet, so TfLiteModelCreate is simply never linked into the iOS app.

The PR fixes the SPM warning, but not the actual TFLite linkage on iOS.

@kmmhanan

kmmhanan commented Jun 8, 2026

Copy link
Copy Markdown
Author

Okay. I'll figure out the solution for it.

@kmmhanan

kmmhanan commented Jun 9, 2026

Copy link
Copy Markdown
Author

@jpdup I fixed the TensorFlowLiteC framework link issue in iOS. Check out the new commit #42c844f

@kmmhanan

kmmhanan commented Jun 9, 2026

Copy link
Copy Markdown
Author

@jpdup Now Fixed MacOS SPM issue. Currently MacOS will support SPM & CocoaPods.
Included MacOS setup documentation + script + integration test file
And I changed the version into v0.12.3.

@kmmhanan kmmhanan requested a review from jpdup June 9, 2026 06:25

@jpdup jpdup left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I tested v0.12.3 for both iOS and macOS
Happy to report that the builds are now clean and the runtime is working

@kmmhanan

kmmhanan commented Jun 9, 2026

Copy link
Copy Markdown
Author

Great, LMK once u merged & published. so I can use the official version.

@jpdup jpdup left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I tested against our fCards game. Detecting real physical cards values on a table

@kmmhanan

kmmhanan commented Jun 9, 2026

Copy link
Copy Markdown
Author

@PaulTR @margaretmz PR #305 is ready for merge.

Tested on iOS, macOS, and Android by @jpdup including real-world inference.
All checks passing, no conflicts.

@kmmhanan

Copy link
Copy Markdown
Author

@jpdup I wonder why @PaulTR not yet merge this PR?

@jpdup

jpdup commented Jun 12, 2026

Copy link
Copy Markdown

@jpdup I wonder why @PaulTR not yet merge this PR?

Seem like this repo is dead (zombie), last commit was 8 months ago
85fb92e

@kmmhanan

Copy link
Copy Markdown
Author

@jpdup I wonder why @PaulTR not yet merge this PR?

Seem like this repo is dead (zombie), last commit was 8 months ago 85fb92e

Disappointing

@PaulTR

PaulTR commented Jun 13, 2026 via email

Copy link
Copy Markdown
Collaborator

@kmmhanan

Copy link
Copy Markdown
Author

Thanks, @PaulTR . Appreciate you checking on the maintainer process. Please let me know if I should loop in the current ODML contact so this PR gets proper review.

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.

3 participants