all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Arun Isaac <arunisaac@systemreboot.net>
Cc: 28421@debbugs.gnu.org
Subject: [bug#28421] [PATCH] gnu: Add uim, uim-gtk and uim-qt.
Date: Tue, 26 Sep 2017 09:22:26 +0200	[thread overview]
Message-ID: <87poad3owd.fsf@gnu.org> (raw)
In-Reply-To: <97554f2b.ADkAADNjyywAAAAAAAAAAAOzWv8AAAACwQwAAAAAAAW9WABZyciA@mailjet.com> (Arun Isaac's message of "Tue, 26 Sep 2017 08:53:59 +0530")

Hi Arun,

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/xorg.scm (uim, uim-gtk, uim-qt): New variables.

[...]

> +    (home-page "https://github.com/uim/uim")
> +    (synopsis "Multilingual input method framework")
> +    (description "uim is a multilingual input method library and environment.
> +It provides a simple, easily extensible and high code-quality input method
> +development platform, and useful input method environment for users of desktop
> +and embedded platforms.")
> +    (license (list license:lgpl2.1+ ; scm/py.scm, pixmaps/*.{svg,png} (see pixmaps/README)
> +                   license:gpl2+ ; scm/pinyin-big5.scm
> +                   license:gpl3+ ; scm/elatin-rules.cm
> +                   license:public-domain ; scm/input-parse.scm, scm/match.scm
> +                   ;; gtk2/toolbar/eggtrayicon.{ch},
> +                   ;; qt3/chardict/kseparator.{cpp,h},
> +                   ;; qt3/pref/kseparator.{cpp,h}
> +                   license:lgpl2.0+
> +                   ;; pixmaps/*.{svg,png} (see pixmaps/README),
> +                   ;; all other files
> +                   license:bsd-3))))
> +
> +(define-public uim-gtk
> +  (package
> +    (inherit uim)
> +    (name "uim-gtk")
> +    (inputs
> +     `(("gtk" ,gtk+)
> +       ("gtk" ,gtk+-2)
> +       ,@(package-inputs uim)))
> +    (synopsis "uim built with GTK+ support")
> +    (description "This package contains uim built with GTK+ support.")))

What about “Multilingual input method framework (GTK+ support)” for the
synopsis, and leaving the description unchanged?

> +    (synopsis "uim built with Qt support")
> +    (description "This package contains uim built with Qt support.")))

Likewise here.

Otherwise LGTM.  Feel free to push with a change along these lines if
that’s fine with you.

Thanks for patiently working on this!

Ludo’.

       reply	other threads:[~2017-09-26  7:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <97554f2b.ADkAADNjyywAAAAAAAAAAAOzWv8AAAACwQwAAAAAAAW9WABZyciA@mailjet.com>
2017-09-26  7:22 ` Ludovic Courtès [this message]
2017-09-26 13:40   ` bug#28421: [PATCH] gnu: Add uim, uim-gtk and uim-qt Arun Isaac
2017-09-11 20:22 [bug#28421] [PATCH] gnu: Add uim Arun Isaac
2017-09-26  3:23 ` [bug#28421] [PATCH] gnu: Add uim, uim-gtk and uim-qt Arun Isaac

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87poad3owd.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=28421@debbugs.gnu.org \
    --cc=arunisaac@systemreboot.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.