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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add mozc-ut #16428

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add mozc-ut #16428

wants to merge 1 commit into from

Conversation

amuramatsu
Copy link
Contributor

Add Japanese input method "Mozc" with UT dictionary
for #4819

@xtkoba
Copy link
Contributor

xtkoba commented Jul 5, 2023

First some cosmetic things.

Please squash all the commits into one. You don't have to wait for approval after you made any changes to the PR.

Please use the tab charater for indentation in build.sh.

@xtkoba
Copy link
Contributor

xtkoba commented Jul 5, 2023

The dependency is dubious, according to DT_NEEDED entries in the ELF objects.

mozc-ut

libQt5Core.so.5
libQt5Gui.so.5
libQt5Widgets.so.5
libandroid-execinfo.so
libandroid-posix-semaphore.so
libc++_shared.so
libc.so
libdl.so
libiconv.so
libm.so

So the dependency should at least include

fcitx5, libandroid-execinfo, libandroid-posix-semaphore, libc++, libiconv, qt5-qtbase

fcitx5-mozc-ut

libFcitx5Config.so
libFcitx5Core.so
libFcitx5Utils.so
libandroid-posix-semaphore.so
libc++_shared.so
libc.so
libdl.so
libm.so

This subpackage depends on the parent package and so no need to specify extra dependencies.

mozc-ut-emacs

libandroid-posix-semaphore.so
libc++_shared.so
libc.so
libm.so

No problem. (Obviously emacs should be depended on.)

@amuramatsu
Copy link
Contributor Author

Thank you for reviewing.

I refined package dependencies and squash all commits.

So the dependency should at least include

fcitx5, libandroid-execinfo, libandroid-posix-semaphore, libc++, libiconv, qt5-qtbase

Well, mozc-ut is not depend to fcitx5, I think.
(If you want to use mozc-ut only with emacs, you need to install mozc-ut and mozc-ut-emacs. Installing fcitx5-mozc-ut and fcitx5 are not required.)

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.

2 participants