Skip to content

Fix link error and dependency issue in Tizen 10.0 #648

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 24, 2025
Merged

Fix link error and dependency issue in Tizen 10.0 #648

merged 1 commit into from
Jul 24, 2025

Conversation

@JSUYA
Copy link
Member

@JSUYA JSUYA commented Jul 23, 2025 •
edited
Loading

related issue: #647

When building the api-version="10.0" native plugin using the Tizen 10.0 dev SDK, we found that unintended library dependencies were added. These libraries are not actually used in Flutter and do not exist in TV profile.

Our analysis results show that the compilation's --as-needed option in SDK's makefile is searching for a specific symbol and the dependency is added, so we suspect that this issue comes from the SDK's toolchain's gcc was upgraded.
So, let's add -lstdc++ first for resolve this issue.
(However libBrokenLocale.so.1 is still added but it works fine on TV. (libBrokenLocale is not added in all cases.))
This PR is a temporary patch and managed as a known issue.

Test plugin&app: default plugin(flutter-tizen create --platforms tizen --template plugin test_plugin)

api-version="10.0"

$ readelf -d ./tizen/bin/Debug/tizen80/tpkroot/lib/libflutter_plugins.so | grep NEEDED
 0x00000001 (NEEDED)                     Shared library: [libpthread.so.0]
 0x00000001 (NEEDED)                     Shared library: [libcapi-nntrainer.so]
 0x00000001 (NEEDED)                     Shared library: [libcapi-system-info.so.0]
 0x00000001 (NEEDED)                     Shared library: [libc.so.6]
 0x00000001 (NEEDED)                     Shared library: [ld-linux.so.3]
 0x00000001 (NEEDED)                     Shared library: [libmmi_manager.so]
 0x00000001 (NEEDED)                     Shared library: [libmmi.so.2]
 0x00000001 (NEEDED)                     Shared library: [libflutter_tizen_tv.so]
 0x00000001 (NEEDED)                     Shared library: [libstdc++.so.6]
 0x00000001 (NEEDED)                     Shared library: [libgcc_s.so.1]

api-version="8.0"

$ readelf -d ./tizen/bin/Debug/tizen80/tpkroot/lib/libflutter_plugins.so | grep NEEDED
 0x00000001 (NEEDED)                     Shared library: [libpthread.so.0]
 0x00000001 (NEEDED)                     Shared library: [libcapi-system-info.so.0]
 0x00000001 (NEEDED)                     Shared library: [libc.so.6]
 0x00000001 (NEEDED)                     Shared library: [ld-linux.so.3]
 0x00000001 (NEEDED)                     Shared library: [libflutter_tizen_tv.so]
 0x00000001 (NEEDED)                     Shared library: [libstdc++.so.6]
 0x00000001 (NEEDED)                     Shared library: [libgcc_s.so.1]

PR after & api-version="10.0"

$ readelf -d ./tizen/bin/Release/tizen80/tpkroot/lib/libflutter_plugins.so

Dynamic section at offset 0x20ed8 contains 33 entries:
  Tag        Type                         Name/Value
 0x00000001 (NEEDED)                     Shared library: [libstdc++.so.6]
 0x00000001 (NEEDED)                     Shared library: [libpthread.so.0]
 0x00000001 (NEEDED)                     Shared library: [libBrokenLocale.so.1]
 0x00000001 (NEEDED)                     Shared library: [libcapi-system-info.so.0]
 0x00000001 (NEEDED)                     Shared library: [libc.so.6]
 0x00000001 (NEEDED)                     Shared library: [ld-linux.so.3]
 0x00000001 (NEEDED)                     Shared library: [libflutter_tizen_tv.so]
 0x00000001 (NEEDED)                     Shared library: [libgcc_s.so.1]
 0x0000000f (RPATH)                      Library rpath: [$ORIGIN]

@JSUYA JSUYA requested review from seungsoo47 and xiaowei-guan July 23, 2025 10:03
@JSUYA JSUYA mentioned this pull request Jul 24, 2025
Open
@JSUYA JSUYA merged commit 5d6d007 into flutter-tizen:master Jul 24, 2025
9 checks passed
@JSUYA JSUYA mentioned this pull request Jul 24, 2025
Closed
@JSUYA JSUYA mentioned this pull request Aug 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seungsoo47 seungsoo47 seungsoo47 approved these changes

@xiaowei-guan xiaowei-guan Awaiting requested review from xiaowei-guan

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JSUYA @seungsoo47