Re: Add Package Gl

[Nikita Karetnikov <nikita@karetnikov.org>]

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

Minor issues…

>    #:use-module ((guix licenses) 
>                   #:renamer (symbol-prefix-proc 'license:))

It’d better to use 'l: as a prefix to avoid (license license:something).
You could also use #:select if it’s possible.

>    #:use-module (gnu packages fontutils))
>
>

Extra newline.

> (define-public glu
>   (package
>      (name "glu")

Should be indented like so:

(package
  (name "glu")

>      (source (origin
> 	      (method url-fetch)
> 	      (uri (string-append "ftp://ftp.freedesktop.org/pub/mesa/glu/glu-" 
>                                    version ".tar.gz"))
>               (sha256
> 	        (base32 "0r72yyhj09x3krn3kn629jqbwyq50ji8w5ri2pn6zwrk35m4g1s3"))))

(source (origin
         (method url-fetch)
         (uri (string-append "ftp://ftp.freedesktop.org/pub/mesa/glu/glu-" 
                             version ".tar.gz"))
         (sha256
          (base32 "0r72yyhj09x3krn3kn629jqbwyq50ji8w5ri2pn6zwrk35m4g1s3"))))

>      (description 
> 	  "GLU, or OpenGL Utility Library provides some higher-level functionality 

(description 
 "GLU, or OpenGL Utility Library provides some higher-level functionality 

The rest of the file has the same problems.  But as Cyril said: “Other
than that, this looks good, thanks!”

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]