diff mbox

[1/3] python: new module flup

Message ID 1400550780-16669-1-git-send-email-oli.vogt.pub01@gmail.com
State Superseded
Headers show

Commit Message

Oli Vogt May 20, 2014, 1:52 a.m. UTC
This module is useful to implement a fastCGI webserver
Tested with ARM and Python 2.7.6

Signed-off-by: oli vogt <oli.vogt.pub01@gmail.com>
---
 package/Config.in                  |  1 +
 package/python-flup/Config.in      |  9 +++++++++
 package/python-flup/python-flup.mk | 12 ++++++++++++
 3 files changed, 22 insertions(+)
 create mode 100644 package/python-flup/Config.in
 create mode 100644 package/python-flup/python-flup.mk

Comments

Thomas Petazzoni May 20, 2014, 7:07 a.m. UTC | #1
Dear oli vogt,

On Tue, 20 May 2014 13:52:58 +1200, oli vogt wrote:
> This module is useful to implement a fastCGI webserver
> Tested with ARM and Python 2.7.6
> 
> Signed-off-by: oli vogt <oli.vogt.pub01@gmail.com>

Thanks for this contribution! See a few comments below.

>  source "package/python-json-schema-validator/Config.in"
> diff --git a/package/python-flup/Config.in b/package/python-flup/Config.in
> new file mode 100644
> index 0000000..cc0efa5
> --- /dev/null
> +++ b/package/python-flup/Config.in
> @@ -0,0 +1,9 @@
> +config BR2_PACKAGE_PYTHON_FLUP
> +	bool "python-flup"
> +	depends on BR2_PACKAGE_PYTHON

Have you been able to test if this module works with Python 3 ?

> diff --git a/package/python-flup/python-flup.mk b/package/python-flup/python-flup.mk
> new file mode 100644
> index 0000000..0525dc9
> --- /dev/null
> +++ b/package/python-flup/python-flup.mk
> @@ -0,0 +1,12 @@
> +################################################################################
> +#
> +# python-flup
> +#
> +################################################################################
> +
> +PYTHON_FLUP_VERSION = 1.0.3.dev-20110405
> +PYTHON_FLUP_SOURCE = flup-$(PYTHON_FLUP_VERSION).tar.gz
> +PYTHON_FLUP_SITE = http://pypi.python.org/packages/source/f/flup
> +PYTHON_FLUP_SETUP_TYPE = setuptools

Could you add PYTHON_FLUP_LICENSE and PYTHON_FLUP_LICENSE_FILES ?

Other than that, looks good to me!

Thanks again,

Thomas
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index 3bc8d24..9c61485 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -445,6 +445,7 @@  source "package/python-configobj/Config.in"
 source "package/python-crc16/Config.in"
 source "package/python-dialog/Config.in"
 source "package/python-dpkt/Config.in"
+source "package/python-flup/Config.in"
 source "package/python-id3/Config.in"
 source "package/python-ipy/Config.in"
 source "package/python-json-schema-validator/Config.in"
diff --git a/package/python-flup/Config.in b/package/python-flup/Config.in
new file mode 100644
index 0000000..cc0efa5
--- /dev/null
+++ b/package/python-flup/Config.in
@@ -0,0 +1,9 @@ 
+config BR2_PACKAGE_PYTHON_FLUP
+	bool "python-flup"
+	depends on BR2_PACKAGE_PYTHON
+	help
+	  flup provides of a collection of WSGI modules for Python.
+	  It allows a webserver to use scgi-mod and fastcgi-mod
+	  with python.
+
+	  http://www.saddi.com/software/flup/
diff --git a/package/python-flup/python-flup.mk b/package/python-flup/python-flup.mk
new file mode 100644
index 0000000..0525dc9
--- /dev/null
+++ b/package/python-flup/python-flup.mk
@@ -0,0 +1,12 @@ 
+################################################################################
+#
+# python-flup
+#
+################################################################################
+
+PYTHON_FLUP_VERSION = 1.0.3.dev-20110405
+PYTHON_FLUP_SOURCE = flup-$(PYTHON_FLUP_VERSION).tar.gz
+PYTHON_FLUP_SITE = http://pypi.python.org/packages/source/f/flup
+PYTHON_FLUP_SETUP_TYPE = setuptools
+
+$(eval $(python-package))