Re: [PATCH] Add microscheme.

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> From 6fd419709fe93511fc519701d4e496f95aba84e4 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Fri, 6 Nov 2015 22:35:33 +0100
> Subject: [PATCH 1/2] gnu: Add prefix to license imports in (gnu packages avr).
>
> * gnu/packages/avr.scm: Import (guix licenses) with prefix "license:".

OK.

> From 1900f3015162cc777ee79883a0cae2a21216d99c Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Fri, 6 Nov 2015 22:37:40 +0100
> Subject: [PATCH 2/2] gnu: Add microscheme.
>
> * gnu/packages/avr.scm (microscheme): New variable.

[...]

> +    (home-page "http://microscheme.org/";)
> +    (synopsis "Scheme subset for Atmel microcontrollers")

“Small Scheme implementation targeting microcontrollers”?  (“Scheme
subset” is vague; “Scheme” alone is already quite vague.  ;-))

I would tend to put it in scheme.scm.  Also, it apparently builds on all
the CPUs that we use, so I’m not sure if it’s worth mentioning Atmel
here?

Otherwise LGTM, thank you!

Ludo’.