GNU bug report logs - #29742
[PATCH] gnu: Add mod-wsgi.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 29742 in the body.
You can then email your comments to 29742 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add mod-wsgi.
Date: Sat, 16 Dec 2017 20:29:11 +0000

* gnu/packages/web.scm (mod-wsgi): New variable.
---
 gnu/packages/web.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index eb0436cb3..9cda0f3e0 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -145,6 +145,37 @@ and its related documentation.")
     (license l:asl2.0)
     (home-page "https://httpd.apache.org/")))
 
+(define-public mod-wsgi
+  (package
+    (name "mod-wsgi")
+    (version "4.5.22")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/GrahamDumpleton/mod_wsgi/archive/"
+                    version ".tar.gz"))
+              (sha256
+               (base32
+                "0n1yhmrfp8mjbsngmyjl937c6rc0069p6wdi1lknrbn1q42hzw6q"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:tests? #f ;; TODO: Can't figure out if there are tests
+       #:make-flags (list
+                     (string-append "DESTDIR="
+                                    (assoc-ref %outputs "out"))
+                     "LIBEXECDIR=/modules")))
+    (inputs
+     `(("httpd" ,httpd)
+       ("python" ,python-wrapper)))
+    (synopsis "Apache module for applications using the Python WSGI
+specification")
+    (description
+     "The mod_wsgi module for the Apache @acronym{HTTP, HyperText Transfer
+Protocol} Server adds support for running applications that support the Python
+@acronym{WSGI, Web Server Gateway Interface} specification.")
+    (license l:asl2.0)
+    (home-page "https://modwsgi.readthedocs.io/")))
+
 (define-public nginx
   (package
     (name "nginx")
-- 
2.14.1

Message #8 received at 29742 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Christopher Baines <mail <at> cbaines.net>
Cc: 29742 <at> debbugs.gnu.org
Subject: Re: [bug#29742] [PATCH] gnu: Add mod-wsgi.
Date: Sun, 17 Dec 2017 17:15:15 +0100

Christopher Baines <mail <at> cbaines.net> 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!

Ludo’.

Message #13 received at 29742-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 29742-done <at> debbugs.gnu.org
Subject: Re: [bug#29742] [PATCH] gnu: Add mod-wsgi.
Date: Sun, 17 Dec 2017 20:56:12 +0000

[Message part 1 (text/plain, inline)]

Ludovic Courtès <ludo <at> gnu.org> writes:

> Christopher Baines <mail <at> cbaines.net> 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 :)

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.