Skip to content

[ci] Change soname of _experimental.so#127

Closed
JSUYA wants to merge 1 commit intoflutter-tizen:masterfrom
JSUYA:change_soname_experimental
Closed

[ci] Change soname of _experimental.so#127
JSUYA wants to merge 1 commit intoflutter-tizen:masterfrom
JSUYA:change_soname_experimental

Conversation

@JSUYA
Copy link
Copy Markdown
Member

@JSUYA JSUYA commented Nov 10, 2025

Force changing SONAME because experimental so can not loaded in C++ host application.
#110 (comment)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we dynamically load libvulkan at runtime (e.g. volk)? I believe you have already considered this approach but wanted to ask anyway. Please ignore if it's too hard to explain.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oh, I hadn't thought about dynamically loading libvulkan. Since the vulkan support PR has already been merged, i think, there's no need to apply this right now, but it seems like it could be improved using this method. Thanks for your information.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

+)
We'll review this comment here. If you have any suggestions, please leave them.
#128

@xiaowei-guan
Copy link
Copy Markdown
Contributor

@JSUYA Hello, I have a query, :
how to load _experimental.so if we run command 'flutter-tizen run --dart-define=USE_FLUTTER_TIZEN_EXPERIMENTAL'?
I originally thought it would link _experimental.so when the USE_FLUTTER_TIZEN_EXPERIMENTAL parameter was added.

@JSUYA JSUYA mentioned this pull request Nov 11, 2025
@JSUYA
Copy link
Copy Markdown
Member Author

JSUYA commented Nov 11, 2025

how to load _experimental.so if we run command 'flutter-tizen run --dart-define=USE_FLUTTER_TIZEN_EXPERIMENTAL'?
I originally thought it would link _experimental.so when the USE_FLUTTER_TIZEN_EXPERIMENTAL parameter was added.

Hi @xiaowei-guan
The work of downloading experimental so and including it with app compile and packaging it together should be done in flutter-tizen. flutter-tizen/flutter-tizen#684

@xiaowei-guan
Copy link
Copy Markdown
Contributor

how to load _experimental.so if we run command 'flutter-tizen run --dart-define=USE_FLUTTER_TIZEN_EXPERIMENTAL'?
I originally thought it would link _experimental.so when the USE_FLUTTER_TIZEN_EXPERIMENTAL parameter was added.

Hi @xiaowei-guan The work of downloading experimental so and including it with app compile and packaging it together should be done in flutter-tizen. flutter-tizen/flutter-tizen#684

Hi,@JSUYA
Yes, flutter-tizen will downloading common so and experimental so.
And I think if we execute 'flutter-tizen run', we will link libflutter_tizen_common.so. If we execute ‘flutter-tizen run --dart-define=USE_FLUTTER_TIZEN_EXPERIMENTAL’, we will link libflutter_tizen_common_experimental.so. I'm not sure if my understanding matches what you're thinking.If the ideas are consistent, I don't think it's necessary to modify the library name using the patchelf command.

And Previously, we encountered errors when running C++ host because using simple file renaming commands (such as mv) only changed the file name in the file system, but the internal SONAME of the library file remained the old name. The application would then fail because it couldn't find a library with the correct name. Therefore, we used patchelf to modify the SONAME.

@JSUYA
Copy link
Copy Markdown
Member Author

JSUYA commented Nov 12, 2025

Yes, that's correct. As per your review, patchelf doesn't seem to be needed. I've created a flutter-tizen PR that uses experimental.so. Please review it. flutter-tizen/flutter-tizen#686

@JSUYA JSUYA closed this Nov 12, 2025
@JSUYA JSUYA deleted the change_soname_experimental branch March 6, 2026 08:02
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.

4 participants