all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#28421] [PATCH] gnu: Add uim, uim-gtk and uim-qt.
       [not found] <97554f2b.ADkAADNjyywAAAAAAAAAAAOzWv8AAAACwQwAAAAAAAW9WABZyciA@mailjet.com>
@ 2017-09-26  7:22 ` Ludovic Courtès
  2017-09-26 13:40   ` bug#28421: " Arun Isaac
  0 siblings, 1 reply; 2+ messages in thread
From: Ludovic Courtès @ 2017-09-26  7:22 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 28421

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’.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#28421: [PATCH] gnu: Add uim, uim-gtk and uim-qt.
  2017-09-26  7:22 ` [bug#28421] [PATCH] gnu: Add uim, uim-gtk and uim-qt Ludovic Courtès
@ 2017-09-26 13:40   ` Arun Isaac
  0 siblings, 0 replies; 2+ messages in thread
From: Arun Isaac @ 2017-09-26 13:40 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 28421-done

Ludovic Courtès <ludo@gnu.org> writes:

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

I made the change and pushed!

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-09-26 13:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <97554f2b.ADkAADNjyywAAAAAAAAAAAOzWv8AAAACwQwAAAAAAAW9WABZyciA@mailjet.com>
2017-09-26  7:22 ` [bug#28421] [PATCH] gnu: Add uim, uim-gtk and uim-qt Ludovic Courtès
2017-09-26 13:40   ` bug#28421: " Arun Isaac

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.