GNU bug report logs - #29719
[PATCH] gnu: virtualization: Add lookingglass.

Previous Next

Full log

View this message in rfc822 format

From: Rutger Helling <rhelling <at> mykolab.com>
To: ludo <at> gnu.org
Cc: 29719 <at> debbugs.gnu.org
Subject: [bug#29719] [PATCH] gnu: virtualization: Add lookingglass.
Date: Fri, 15 Dec 2017 16:11:51 +0100

[Message part 1 (text/plain, inline)]

Forgot to put two spaces in the lines for the description.
Here's the proper patch, sorry about that. 

On 2017-12-15 16:05, Rutger Helling wrote:

> Thanks for reviewing! Here's an updated patch based on your feedback. 
> 
> On 2017-12-15 15:16, ludo <at> gnu.org wrote: 
> Hello!
> 
> Rutger Helling <rhelling <at> mykolab.com> skribis:
> 
> From ccea9a6c9dd71aea5ef5c07802f7e376a7f86a96 Mon Sep 17 00:00:00 2001
> From: Rutger Helling <rhelling <at> mykolab.com>
> Date: Fri, 15 Dec 2017 11:58:10 +0100
> Subject: [PATCH] gnu: virtualization: Add lookingglass.
> 
> * gnu/packages/virtualization.scm (lookingglass): New variable. 
> [...]
> 
> +(define-public lookingglass
> +  (package
> +   (name "lookingglass") 
> Please run:
> 
> ./etc/indent-code.el gnu/packages/virtualization.scm lookingglass
> 
> +   (version "a5")
> +   (source
> +    (origin
> +     (method url-fetch)
> +     (uri (string-append "https://github.com/gnif/LookingGlass/archive/"
> +                         version ".tar.gz"))
> +     (file-name (string-append name "-" version))
> +     (sha256
> +      (base32
> +       "0lrb821914fp27xaq0spwhbblssz55phiygvdlvcrkifa138v8pf"))))
> +   (build-system gnu-build-system)
> +   (inputs `(("fontconfig" ,fontconfig)
> +             ("glu" ,glu)
> +             ("mesa" ,mesa)
> +             ("openssl" ,openssl)
> +             ("sdl2" ,sdl2)
> +             ("sdl2-ttf" ,sdl2-ttf)
> +             ("spice-protocol" ,spice-protocol)))
> +   (native-inputs `(("pkg-config", pkg-config)))
> +   (arguments
> +    `(#:tests? #f ;; No tests are available.
> +      #:phases (modify-phases %standard-phases
> +                 (replace 'configure
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (chdir "client"))) 
> Return #t.
> 
> +                  (replace 'install
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (mkdir-p (string-append %output "/bin"))
> +                      (copy-file "bin/looking-glass-client" (string-append
> +                                 %output "/bin/looking-glass-client"))))))) 
> Same here.
> 
> Use the 'outputs' argument instead of the '%output' global variable.
> 
> Also, you can replace the mkdir-p/copy-file pair with just:
> 
> (install-file "bin/looking-glass-client"
> (string-append (assoc-ref outputs "out") "/bin"))
> 
> +   (home-page "https://looking-glass.hostfission.com")
> +   (synopsis "Low-latency KVMFR implementation for QEMU guests with VGA PCI
> +passthrough")
> +   (description "LookingGlass is an extremely low latency KVMFR (KVM FrameRelay)
> +implementation for QEMU guests with VGA PCI Passthrough.") 
> Could you shorten the synopsis and expound the description a little bit?
> :-)  I find it hard to understand what this is about looking at this.
> 
> +   (license gpl2))) 
> AFAICS source file headers explicitly say "or any later version", so
> this should be 'gpl2+'.
> 
> The rest LGTM.  Could you send an updated patch?
> 
> Thank you!
> 
> Ludo'.

[Message part 2 (text/html, inline)]

[0001-gnu-virtualization-Add-lookingglass-v3.patch (text/x-diff, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.