unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* gtk-vnc package
@ 2016-01-26 21:20 rennes
  2016-01-27  9:57 ` 宋文武
  0 siblings, 1 reply; 2+ messages in thread
From: rennes @ 2016-01-26 21:20 UTC (permalink / raw)
  To: guix-devel

[-- 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


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

* Re: gtk-vnc package
  2016-01-26 21:20 gtk-vnc package rennes
@ 2016-01-27  9:57 ` 宋文武
  0 siblings, 0 replies; 2+ messages in thread
From: 宋文武 @ 2016-01-27  9:57 UTC (permalink / raw)
  To: rennes; +Cc: guix-devel

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+'.

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

end of thread, other threads:[~2016-01-27  9:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-26 21:20 gtk-vnc package rennes
2016-01-27  9:57 ` 宋文武

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).