[PATCH] Add package: ibus-chewing

  • Open
  • quality assurance status badge
Details
2 participants
  • Guu, Jin-Cheng
  • Raghav Gururajan
Owner
unassigned
Submitted by
Guu, Jin-Cheng
Severity
normal

Debbugs page

Guu, Jin-Cheng wrote 4 years ago
(address . guix-patches@gnu.org)
CAErjG5c9zLn24hd0wPUdkHcDPuNzKu5YdBZJkMJ6iZjrKJ8HnA@mail.gmail.com
Hello all,

Attached is my first patch to guix. It's for the package
`ibus-chewing`, an IBus front-end of Chewing, which is an intelligent
Chinese input method for Zhuyin (BoPoMoFo) users. A dependency `gob`
is also added. Hopefully, this will attract more Chinese users to
guix.

Special thanks for their patience and guides to nckx and
raghavgururajan on #guix. Most of the code work came from
raghavgururajan.

Regards,
Jin
Attachment: 0001-gnu-Add-gob-and-ibus-chewing.patch
Raghav Gururajan wrote 4 years ago
(address . 47884@debbugs.gnu.org)
0a063953-4699-72e5-ead5-3d9bea22f7b9@raghavgururajan.name
Hi Jin-Cheng!

Congratulations on your first patch. \o/

You don't have to under-sell your work. I just cleaned up the pack-def
and fixed the glib schema error. All others are your work. :-)

Also, if anyone helps you with code/snippet in #guix, you don't have to
use the same code/snippet. You are free to adopt them in any way you see
fit. Folks usually share a code/snippet, purely for informational purposes.

REVIEW:

[1] As Julien mentioned in #guix, it is better to make two patches, one
for dependency package and one for the main package.

[2] Package gob appears to be a candidate for glib.scm. So its better to
move this package to glib.scm.

[3] In Guix, we try to avoid bundled stuff, for better security and
reproducibilty. It'd be great if you package cmake-fedora separately
(like gob) and remove the use of `recursive? #t`, in ibus-chewing.

As usual, if you have any questions, feel to free to ask around in
#guix. ;-)

Regards,
RG.
Attachment: OpenPGP_signature
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 47884@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 47884
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help