Re: [PATCH] Add ZynAddSubFX

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> From 3926ec4243d497438b080b83678b2bd05105024d Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Mon, 31 Aug 2015 17:33:56 +0200
> Subject: [PATCH 1/2] gnu: Add minixml.
>
> * gnu/packages/xml.scm (minixml): New variable.

[...]

> +    (description
> +     "Mini-XML is a small XML library that you can use to read and write XML
> +and XML-like data files in your application without requiring large
> +non-standard libraries.
> +
> +Mini-XML supports reading of UTF-8 and UTF-16 and writing of UTF-8 encoded 
> XML
> +files and strings. Data is stored in a linked-list tree structure, preserving
> +the XML data hierarchy, and arbitrary element names, attributes, and 
> attribute
> +values are supported with no preset limits, just available memory.")

Maybe something neutral?  Like “Mini-XML is a small C library to read
and write XML files in UTF-8 and UTF-16.”

> +    ;; LGPL 2.0 with additional exceptions for static linking
> +    (license license:lgpl2.0+)))

2.0 or later?

OK with this/these change(s).

> From 93385af3c66a35c7102229a49b571655145e5898 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Mon, 31 Aug 2015 17:34:28 +0200
> Subject: [PATCH 2/2] gnu: Add ZynAddSubFX.
>
> * gnu/packages/music.scm (zynaddsubfx): New variable.

[...]

> +    (home-page "http://zynaddsubfx.sf.net/";)
> +    (synopsis "Software synthesizer")
> +    (description
> +     "ZynAddSubFX is a feature heavy realtime software synthesizer.")

You’ll have to expound a bit, I think.  :-)

Otherwise OK.

(I like the name, BTW.)

Ludo’.