Subject: Re: [bug#41849] [PATCH] Add frozen-bubble

From: Ludovic Courtès <[email-masked]>
Date: Wed, 17 Jun 2020 11:35:08 +0200

Hi,

Nicolas Goaziou <[email-masked]> skribis:

>>>From 2586c90f5a25985bd02b0ea5bc65c5fc982b50d8 Mon Sep 17 00:00:00 2001
> From: Nicolas Goaziou <[email-masked]>
> Date: Sun, 14 Jun 2020 11:50:43 +0200
> Subject: [PATCH 1/3] gnu: Add perl-alien-sdl.
>
> * gnu/packages/perl.scm (perl-alien-sdl): New variable.

LGTM.

>>>From 64744949f1d80b40317d942807b871f3c3f0f490 Mon Sep 17 00:00:00 2001
> From: Nicolas Goaziou <[email-masked]>
> Date: Sun, 14 Jun 2020 12:44:28 +0200
> Subject: [PATCH 2/3] gnu: Add perl-sdl.
>
> * gnu/packages/perl.scm (perl-sdl): New variable.

LGTM.

>>>From 6ff230c8ec69cdc21d6b8148b54f170b3fd859e9 Mon Sep 17 00:00:00 2001
> From: Nicolas Goaziou <[email-masked]>
> Date: Sun, 14 Jun 2020 12:50:34 +0200
> Subject: [PATCH 3/3] gnu: Add frozen-bubble.
>
> * gnu/packages/games.scm (frozen-bubble): New variable.

[...]

> +(define-public frozen-bubble
> +  (let ((commit "d6a029110ad6ab9e4960052e175addc98807fb7e")
> +        (revision "1"))

Please add a comment stating why we’re pick this revision.

> +           (add-after 'unpack 'fix-permissions
> +             (lambda _ (invoke "chmod" "-R" "+w" "server")))

How about: (for-each make-file-writable
                     (find-files "servers" "." #:directories? #t))

Otherwise LGTM.

I can’t wait to try it again, so many years later.  :-)

Thanks!

Ludo’.