Subject: Re: [bug#29742] [PATCH] gnu: Add mod-wsgi.
From: Christopher Baines <[email-masked]>
To: Ludovic Courtès <[email-masked]>
Cc: [email-masked]
Date: Sun, 17 Dec 2017 20:56:12 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Ludovic Courtès <[email-masked]> writes:

> Christopher Baines <[email-masked]> skribis:
>
>> * gnu/packages/web.scm (mod-wsgi): New variable.
>
> [...]
>
>> +    (arguments
>> +     '(#:tests? #f ;; TODO: Can't figure out if there are tests
>
> That’s OK at a first stab at least.  :-)
>
>> +    (synopsis "Apache module for applications using the Python WSGI
>> +specification")
>
> s/Apache/Apache httpd/
>
> and maybe “for …” → “for Python WSGI applications”
>
>> +    (description
>> +     "The mod_wsgi module for the Apache @acronym{HTTP, HyperText Transfer
>> +Protocol} Server adds support for running applications that support the Python
>
> I’d just write “for Apache httpd”.
>
> Otherwise LGTM, thanks!

Great, I've made these changes and pushed. Thanks for reviewing :)