Marius Bakke <mbakke@fastmail.com> writes:

> Kei Kebreau <kei@openmailbox.org> writes:
>
>> * gnu/packages/game-development.scm (nml): New variable.
>
> [...]
>
>> +    (build-system python-build-system)
>> +    (inputs
>> +     `(("python-pillow" ,python-pillow)
>> +       ("python-ply" ,python-ply)))
>
> Should these not be propagated?

They should! Nice catch.

>
>> +    (home-page "http://dev.openttdcoop.org/projects/nml")
>> +    (synopsis "NML compiler")
>> +    (description
>> +     "NML is a a python-based compiler, capable of compiling NML files (along
>> +with their associated language, sound and graphic files) into grf and/or nfo
>> +files.")
>
> Could you mention what NML is here with a @dfn{}? Maybe also add
> @file{.grf} and @file{.nfo} if they are file extensions. LGTM otherwise!

Sure thing. I am doing the same for "GRF" in the grfcodec description.