Hi

ludo@gnu.org (Ludovic Courtès) writes:

> Also, what about putting it in games.scm instead?

Done.

>> +    (build-system gnu-build-system)
>> +    (arguments
>> +     '(#:phases
>> +       (alist-cons-before
>> +        'patch-source-shebangs 'autoreconf
>> +        (lambda args
>> +          (system* "autoreconf" "-fi"))
>> +        %standard-phases)))
>
> This shouldn’t be needed because the tarballs includes ./configure,
> Makefile.in, etc.

Actually it is needed because the configure which comes with the tarball
was generated with a very old version of autotools and doesn't support
the passing of variables (like SHELL). 

>> +    (synopsis "CMatrix simulates the display from \"The Matrix\"")
>
> In general the synopsis shouldn’t repeat the package name, so:
>
>   Simulate the display from \"The Matrix\"

Done.

>
> Could you send an updated patch?

An updated patch is attached.

Regards
Cyrill