gtk-vnc package

gtk-vnc package

[rennes]

[-- Attachment #1: Type: text/plain, Size: 91 bytes --]

hi,

can you help me to review this patch, gtk-vnc is a API for Vinagre 
program.

Regards.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-gtk-vnc.patch --]
[-- Type: text/x-diff; name=0001-gnu-Add-gtk-vnc.patch, Size: 2577 bytes --]

From cfeac9d0ca6505f98ba206aa4dae5170431baf8e Mon Sep 17 00:00:00 2001
From: Rene Saavedra <rennes@openmailbox.org>
Date: Tue, 26 Jan 2016 21:47:36 +0100
Subject: [PATCH] gnu: Add gtk-vnc.

---
 gnu-system.am            |  1 +
 gnu/packages/gtk-vnc.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 51 insertions(+)
 create mode 100644 gnu/packages/gtk-vnc.scm

diff --git a/gnu-system.am b/gnu-system.am
index 3426056..18be503 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -355,6 +355,7 @@ GNU_SYSTEM_MODULES =				\
   gnu/packages/zile.scm				\
   gnu/packages/zip.scm				\
   gnu/packages/zsh.scm				\
+  gnu/packages/gtk-vnc.scm                      \
 						\
   gnu/services.scm				\
   gnu/services/avahi.scm			\
diff --git a/gnu/packages/gtk-vnc.scm b/gnu/packages/gtk-vnc.scm
new file mode 100644
index 0000000..a7b8cf8
--- /dev/null
+++ b/gnu/packages/gtk-vnc.scm
@@ -0,0 +1,50 @@
+(define-module (gnu packages gtk-vnc)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module (gnu packages glib)
+  #:use-module (gnu packages gnome)
+  #:use-module (gnu packages gtk)
+  #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages tls)
+  #:use-module (gnu packages gnupg)
+  #:use-module (gnu packages pulseaudio))
+  
+(define-public gtk-vnc
+  (package
+    (name "gtk-vnc")
+    (version "0.5.4")
+    (source
+     (origin
+      (method url-fetch)
+       (uri (string-append "mirror://gnome/sources/" name "/"
+                           name "-" version ".tar.xz"))
+      (sha256
+       (base32
+        "1rwwdh7lb16xdmy76ca6mpqfc3zfl3a4bkcr0qb6hs6ffrxak2j8"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:configure-flags
+       (list "--with-gtk=3.0" "--enable-vala" "--without-sasl")))
+    (inputs
+     `(("gtk+" ,gtk+)				
+       ("vala" ,vala)				
+       ("gjs" ,gjs)				
+       ("libgcrypt" ,libgcrypt)			
+       ("gnutls" ,gnutls)			
+       ("glib" ,glib "bin")
+       ("pulseaudio" ,pulseaudio)))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)
+       ("intltool" ,intltool)))
+    (home-page "https://wiki.gnome.org/Projects/gtk-vnc")
+    (synopsis
+     "VNC viewer widget for GTK.")
+    (description
+     "GTK-VNC is an API used by Vinagre client application, Boxes and
+virt-viewer. GTK-VNC supports authentication extensions as SASL, TLS, VeNCrypt.
+Additionally supports encoding extensions.")
+    (license license:lgpl2.1)))
+
+gtk-vnc
-- 
2.6.3

Re: gtk-vnc package

[宋文武]

rennes@openmailbox.org writes:

> hi,
>
> can you help me to review this patch, gtk-vnc is a API for Vinagre
> program.
Welcome!
>
> Regards.
>
> From cfeac9d0ca6505f98ba206aa4dae5170431baf8e Mon Sep 17 00:00:00 2001
> From: Rene Saavedra <rennes@openmailbox.org>
> Date: Tue, 26 Jan 2016 21:47:36 +0100
> Subject: [PATCH] gnu: Add gtk-vnc.
Missing commit message, it should look like:

  * gnu/packages/gtk-vnc.scm: New file.
  * gnu-system (GNU_SYSTEM_MODULES): Add it.

But I'd like to put it (and vinagre) in gnome.scm.
>
> ---
>  gnu-system.am            |  1 +
>  gnu/packages/gtk-vnc.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 51 insertions(+)
>  create mode 100644 gnu/packages/gtk-vnc.scm
>
> diff --git a/gnu-system.am b/gnu-system.am
> index 3426056..18be503 100644
> --- a/gnu-system.am
> +++ b/gnu-system.am
> @@ -355,6 +355,7 @@ GNU_SYSTEM_MODULES =				\
>    gnu/packages/zile.scm				\
>    gnu/packages/zip.scm				\
>    gnu/packages/zsh.scm				\
> +  gnu/packages/gtk-vnc.scm                      \
>  						\
>    gnu/services.scm				\
>    gnu/services/avahi.scm			\
> diff --git a/gnu/packages/gtk-vnc.scm b/gnu/packages/gtk-vnc.scm
> new file mode 100644
> index 0000000..a7b8cf8
> --- /dev/null
> +++ b/gnu/packages/gtk-vnc.scm
> @@ -0,0 +1,50 @@
> +(define-module (gnu packages gtk-vnc)
> +  #:use-module ((guix licenses) #:prefix license:)
> +  #:use-module (guix packages)
> +  #:use-module (guix download)
> +  #:use-module (guix build-system gnu)
> +  #:use-module (gnu packages glib)
> +  #:use-module (gnu packages gnome)
> +  #:use-module (gnu packages gtk)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages tls)
> +  #:use-module (gnu packages gnupg)
> +  #:use-module (gnu packages pulseaudio))
> +  
> +(define-public gtk-vnc
> +  (package
> +    (name "gtk-vnc")
> +    (version "0.5.4")
> +    (source
> +     (origin
> +      (method url-fetch)
> +       (uri (string-append "mirror://gnome/sources/" name "/"
> +                           name "-" version ".tar.xz"))
> +      (sha256
> +       (base32
> +        "1rwwdh7lb16xdmy76ca6mpqfc3zfl3a4bkcr0qb6hs6ffrxak2j8"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:configure-flags
> +       (list "--with-gtk=3.0" "--enable-vala" "--without-sasl")))
> +    (inputs
> +     `(("gtk+" ,gtk+)				
> +       ("vala" ,vala)				
> +       ("gjs" ,gjs)				
> +       ("libgcrypt" ,libgcrypt)			
> +       ("gnutls" ,gnutls)			
> +       ("glib" ,glib "bin")
> +       ("pulseaudio" ,pulseaudio)))
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)
> +       ("intltool" ,intltool)))
I think glib:bin and vala belong to native-inputs, usually they
are only used during build.
> +    (home-page "https://wiki.gnome.org/Projects/gtk-vnc")
> +    (synopsis
> +     "VNC viewer widget for GTK.")
Put synopsis into one line, use `GTK+' and remove the `.'.
> +    (description
> +     "GTK-VNC is an API used by Vinagre client application, Boxes and
> +virt-viewer. GTK-VNC supports authentication extensions as SASL, TLS, VeNCrypt.
> +Additionally supports encoding extensions.")
> +    (license license:lgpl2.1)))
Most source files say LGPL-2.0+, while the COPYING.LIB is LGPL-2.1.
I think the license should be `lgpl2.1+'.