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

New package: x-unikey-1.0.4 #32405

Closed
wants to merge 1 commit into from
Closed

Conversation

linhvng
Copy link
Contributor

@linhvng linhvng commented Aug 8, 2021

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

@linhvng
Copy link
Contributor Author

linhvng commented Aug 8, 2021

The patches follow what FreeBSD maintainer did to compile the package in FreeBSD. Here is the link to this package on FreeBSD: https://cgit.freebsd.org/ports/tree/vietnamese/x-unikey

I currently use it on my x86_64-glibc laptop 😃

distfiles="${SOURCEFORGE_SITE}/unikey/${pkgname}-${version}.tar.bz2"
checksum=aa7dd444853538bcba0f24c4c19692c34d4553a1df213a260c2628a7116b2dd9

CFLAGS+="-g -O2 -Wno-narrowing"
Copy link
Member

Choose a reason for hiding this comment

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

-g -O2 is set by xbps-src, I'll add a patch for musl, too.

makedepends="libX11-devel"
short_desc="Vietnamese input method for X"
maintainer="Linh Nguyễn <linhvng.2212@gmail.com>"
license="GPL-2.0-only"
Copy link
Member

Choose a reason for hiding this comment

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

Most of them are GPL-2.0-or-later, some files are licensed under MIT.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sorry, it's my first time adding a package and tracking down the license 😅 thought it was GPL2 because of the COPYING file... what license should i change it to?

@linhvng
Copy link
Contributor Author

linhvng commented Aug 8, 2021

  • remove -g -O2 from CFLAGS and CXXFLAGS
  • update license to GPL-2.0-or-later

@sgn
Copy link
Member

sgn commented Aug 8, 2021

I've fixed the build for musl libc, however, it's hung when typing.

Anyway, just out of curiosity, have you tried another input framework?
I'm not really sure if it's worth to include this package for 1 user. :-p

@linhvng
Copy link
Contributor Author

linhvng commented Aug 8, 2021

I've fixed the build for musl libc, however, it's hung when typing.

Anyway, just out of curiosity, have you tried another input framework?
I'm not really sure if it's worth to include this package for 1 user. :-p

I have tried fcitx but it puts my CPU to 100% for some reasons; I was using fluxbox and the tray icon for fcitx keeps flashing in the toolbar, which makes me suspect fluxbox might be trying to update/refresh the toolbar along with fcitx tray icon too much, thus pushing the CPU usage...

Your reason is totally understandable, it is a niche tool after all and the fact that it's broken on musl doesn't help either 😅

I'm totally happy with keeping it in my forked repo and I'll try to bother you again when I get it to work on musl once I get my hand on more hardware 😆 It's been a pleasant working with you 🤗

@sgn
Copy link
Member

sgn commented Aug 8, 2021

I've fixed the build for musl libc, however, it's hung when typing.
Anyway, just out of curiosity, have you tried another input framework?
I'm not really sure if it's worth to include this package for 1 user. :-p

I have tried fcitx but it puts my CPU to 100% for some reasons; I was using fluxbox and the tray icon for fcitx keeps flashing in the toolbar, which makes me suspect fluxbox might be trying to update/refresh the toolbar along with fcitx tray icon too much, thus pushing the CPU usage...

Hm, weird, I've installed fluxbox to try, and fcitx's resource consumption is pretty normal for me.
Anyway, I would recommend using fcitx5 instead, there's fcitx5-m17n to type Vietnamese.

In my experience, fcitx5 is a lot better than fcitx.
I also hear good feedback on ibus-bamboo.

Your reason is totally understandable, it is a niche tool after all and the fact that it's broken on musl doesn't help either sweat_smile

I'm totally happy with keeping it in my forked repo and I'll try to bother you again when I get it to work on musl once I get my hand on more hardware laughing It's been a pleasant working with you hugs

@linhvng
Copy link
Contributor Author

linhvng commented Aug 8, 2021

I didn't know about ibus-bamboo, it works flawlessly! Thank you very much! 🥂 I'm more than happy to have the pull request closed if you are 😄

@ericonr ericonr added the new-package This PR adds a new package label Aug 8, 2021
@sgn sgn closed this Aug 9, 2021
linhvng added a commit to linhvng/void-packages that referenced this pull request Aug 19, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants