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

Remove charcode plugin, etc. #1533

Merged
merged 3 commits into from
Dec 16, 2024
Merged

Conversation

WhiredPlanck
Copy link
Collaborator

@WhiredPlanck WhiredPlanck commented Dec 15, 2024

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes #1532

Feature

Describe features of this pull request

This PR is to deprecate charcode plugin, most of rime frontends (both officials and unofficials) don't integrate it by default, and upstream wouldn't like to maintain it anymore (the bundled emoji unicode data is out-of-date). By removing it, we can also remove libiconv and boost::locale, which can decrease Trime apk size about 1 MB.

By the way, I update librime to the latest commit, which updates bundled glog to 0.7.1, so we can remove a patch for it.

Code of conduct

Code style

Build pass

  • make debug

Manually test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub Action CI pass
  4. At least one contributor review and approve
  5. Merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login and download artifact at https://github.com/osfans/trime/actions

Additional Info

It's deprecated by upstream
This commit updates bundled glog to 0.7.1, which fix the Android detection, so we can remove our patch.
Copy link
Collaborator

@Bambooin Bambooin left a comment

Choose a reason for hiding this comment

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

No regression found, nice work!

And we reduce the size of apk with 7.57%!

(9187984 - 8492120) / 9187984 = 0.0757

@WhiredPlanck
Copy link
Collaborator Author

No regression found, nice work!

And we reduce the size of apk with 7.57%!

(9187984 - 8492120) / 9187984 = 0.0757

Wow~

@WhiredPlanck WhiredPlanck merged commit 2c1ba61 into osfans:develop Dec 16, 2024
3 checks passed
@WhiredPlanck WhiredPlanck deleted the native-20241215 branch December 16, 2024 15:40
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.

Upgrade libiconv dependency
2 participants