[Buildroot] [PATCH 30/38] python-keyring: new package

[Thomas De Schampheleire]

On Sun, Dec 8, 2013 at 11:14 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> From: Ludovic Desroches <ludovic.desroches at atmel.com>
>
> [Thomas: convert to the Python package infrastructure, improved the
> .zip handling, added license informations.]
>
> Signed-off-by: Ludovic Desroches <ludovic.desroches at atmel.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
>  package/Config.in                        |  1 +
>  package/python-keyring/Config.in         |  8 ++++++++
>  package/python-keyring/python-keyring.mk | 19 +++++++++++++++++++
>  3 files changed, 28 insertions(+)
>  create mode 100644 package/python-keyring/Config.in
>  create mode 100644 package/python-keyring/python-keyring.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 5f29c54..4f7c9cd 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -391,6 +391,7 @@ source "package/python-dialog/Config.in"
>  source "package/python-dpkt/Config.in"
>  source "package/python-id3/Config.in"
>  source "package/python-ipy/Config.in"
> +source "package/python-keyring/Config.in"
>  source "package/python-mad/Config.in"
>  source "package/python-meld3/Config.in"
>  source "package/python-msgpack/Config.in"
> diff --git a/package/python-keyring/Config.in b/package/python-keyring/Config.in
> new file mode 100644
> index 0000000..6a5c4c5
> --- /dev/null
> +++ b/package/python-keyring/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_PYTHON_KEYRING
> +       bool "python-keyring"
> +       depends on BR2_PACKAGE_PYTHON
> +       help
> +         The Python keyring lib provides a easy way to access the system

an easy way

> +         keyring service from python.

Inconsistent capitalization of python/Python in this same sentence.
Not sure which strategy to follow. For Lua we used capital L because
it is explicitly requested on the Lua website. We could follow this
strategy for Python too in help texts, but I have no strong opinion
here.

> +
> +         https://pypi.python.org/pypi/keyring
> diff --git a/package/python-keyring/python-keyring.mk b/package/python-keyring/python-keyring.mk
> new file mode 100644
> index 0000000..ead5184
> --- /dev/null
> +++ b/package/python-keyring/python-keyring.mk
> @@ -0,0 +1,19 @@
> +################################################################################
> +#
> +# python-keyring
> +#
> +################################################################################
> +
> +PYTHON_KEYRING_VERSION = 3.0.5
> +PYTHON_KEYRING_SOURCE  = keyring-$(PYTHON_KEYRING_VERSION).zip
> +PYTHON_KEYRING_SITE    = http://pypi.python.org/packages/source/k/keyring/
> +PYTHON_KEYRING_SETUP_TYPE = setuptools
> +PYTHON_KEYRING_LICENSE    = python software foundation license

Same comment about spaces.