* [Buildroot] [PATCH 1/2] Add pyyaml package @ 2015-07-26 12:12 Geoffrey Ragot 2015-07-26 12:12 ` [Buildroot] [PATCH 2/2] Add pycli package Geoffrey Ragot 2015-07-27 22:11 ` [Buildroot] [PATCH 1/2] Add pyyaml package Thomas Petazzoni 0 siblings, 2 replies; 4+ messages in thread From: Geoffrey Ragot @ 2015-07-26 12:12 UTC (permalink / raw) To: buildroot Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com> --- package/Config.in | 1 + package/python-pyyaml/Config.in | 8 ++++++++ package/python-pyyaml/python-pyyaml.mk | 15 +++++++++++++++ 3 files changed, 24 insertions(+) create mode 100644 package/python-pyyaml/Config.in create mode 100644 package/python-pyyaml/python-pyyaml.mk diff --git a/package/Config.in b/package/Config.in index cfa7512..8bc27a4 100644 --- a/package/Config.in +++ b/package/Config.in @@ -649,6 +649,7 @@ menu "external python modules" source "package/python-pyusb/Config.in" source "package/python-pyxb/Config.in" source "package/python-pyxml/Config.in" + source "package/python-pyyaml/Config.in" source "package/python-pyzmq/Config.in" source "package/python-requests/Config.in" source "package/python-rtslib-fb/Config.in" diff --git a/package/python-pyyaml/Config.in b/package/python-pyyaml/Config.in new file mode 100644 index 0000000..a63c445 --- /dev/null +++ b/package/python-pyyaml/Config.in @@ -0,0 +1,8 @@ +comment "python-pyyaml require libyaml" + depends on !BR2_PACKAGE_LIBYAML + +config BR2_PACKAGE_PYTHON_PYYAML + bool "python-pyyaml" + depends on BR2_PACKAGE_LIBYAML + help + The PyYAML package contains binding to the libyaml API. diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk new file mode 100644 index 0000000..ef10dac --- /dev/null +++ b/package/python-pyyaml/python-pyyaml.mk @@ -0,0 +1,15 @@ +################################################################################ +# +# python-pyyaml +# +################################################################################ + +PYTHON_PYYAML_VERSION = 3.11 +PYTHON_PYYAML_SOURCE = PyYAML-$(PYTHON_PYYAML_VERSION).tar.gz +PYTHON_PYYAML_SITE = https://pypi.python.org/packages/source/P/PyYAML/ +PYTHON_PYYAML_SETUP_TYPE = distutils +PYTHON_PYYAML_LICENSE = Python software foundation license v2 +PYTHON_PYYAML_LICENSE_FILES = LICENSE +PYTHON_PYYAML_DEPENDENCIES = libyaml + +$(eval $(python-package)) -- 2.4.6 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2/2] Add pycli package 2015-07-26 12:12 [Buildroot] [PATCH 1/2] Add pyyaml package Geoffrey Ragot @ 2015-07-26 12:12 ` Geoffrey Ragot 2015-07-28 22:04 ` Thomas Petazzoni 2015-07-27 22:11 ` [Buildroot] [PATCH 1/2] Add pyyaml package Thomas Petazzoni 1 sibling, 1 reply; 4+ messages in thread From: Geoffrey Ragot @ 2015-07-26 12:12 UTC (permalink / raw) To: buildroot Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com> --- package/Config.in | 1 + package/python-pycli/Config.in | 4 ++++ package/python-pycli/python-pycli.mk | 15 +++++++++++++++ 3 files changed, 20 insertions(+) create mode 100644 package/python-pycli/Config.in create mode 100644 package/python-pycli/python-pycli.mk diff --git a/package/Config.in b/package/Config.in index 8bc27a4..06a4326 100644 --- a/package/Config.in +++ b/package/Config.in @@ -633,6 +633,7 @@ menu "external python modules" source "package/python-protobuf/Config.in" source "package/python-psutil/Config.in" source "package/python-pyasn/Config.in" + source "package/python-pycli/Config.in" source "package/python-pycrypto/Config.in" source "package/python-pydal/Config.in" source "package/python-pyftpdlib/Config.in" diff --git a/package/python-pycli/Config.in b/package/python-pycli/Config.in new file mode 100644 index 0000000..a4f157e --- /dev/null +++ b/package/python-pycli/Config.in @@ -0,0 +1,4 @@ +config BR2_PACKAGE_PYTHON_PYCLI + bool "python-pycli" + help + The cli package is a framework for making simple, correct command line applications in Python. diff --git a/package/python-pycli/python-pycli.mk b/package/python-pycli/python-pycli.mk new file mode 100644 index 0000000..6857425 --- /dev/null +++ b/package/python-pycli/python-pycli.mk @@ -0,0 +1,15 @@ +################################################################################ +# +# python-pycli +# +################################################################################ + +PYTHON_PYCLI_VERSION = devel +PYTHON_PYCLI_SOURCE = pyCLI-$(PYTHON_PYCLI_VERSION).tar.gz +PYTHON_PYCLI_SITE = https://pypi.python.org/packages/source/p/pyCLI/ +PYTHON_PYCLI_SETUP_TYPE = distutils +PYTHON_PYCLI_LICENSE = Python software foundation license v2 +PYTHON_PYCLI_LICENSE_FILES = LICENSE +PYTHON_PYCLI_SETUP_TYPE = setuptools + +$(eval $(python-package)) -- 2.4.6 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2/2] Add pycli package 2015-07-26 12:12 ` [Buildroot] [PATCH 2/2] Add pycli package Geoffrey Ragot @ 2015-07-28 22:04 ` Thomas Petazzoni 0 siblings, 0 replies; 4+ messages in thread From: Thomas Petazzoni @ 2015-07-28 22:04 UTC (permalink / raw) To: buildroot Dear Geoffrey Ragot, On Sun, 26 Jul 2015 14:12:18 +0200, Geoffrey Ragot wrote: > Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com> > --- > package/Config.in | 1 + > package/python-pycli/Config.in | 4 ++++ > package/python-pycli/python-pycli.mk | 15 +++++++++++++++ > 3 files changed, 20 insertions(+) > create mode 100644 package/python-pycli/Config.in > create mode 100644 package/python-pycli/python-pycli.mk I've applied this patch, but had to fix a number of different things before that. See below. > diff --git a/package/python-pycli/Config.in b/package/python-pycli/Config.in > new file mode 100644 > index 0000000..a4f157e > --- /dev/null > +++ b/package/python-pycli/Config.in > @@ -0,0 +1,4 @@ > +config BR2_PACKAGE_PYTHON_PYCLI > + bool "python-pycli" > + help > + The cli package is a framework for making simple, correct command line applications in Python. This line is too long. Text should be wrapped so that it fits in 72 columns. Also, Config.in help text should have the URL of the project homepage. > diff --git a/package/python-pycli/python-pycli.mk b/package/python-pycli/python-pycli.mk > new file mode 100644 > index 0000000..6857425 > --- /dev/null > +++ b/package/python-pycli/python-pycli.mk > @@ -0,0 +1,15 @@ > +################################################################################ > +# > +# python-pycli > +# > +################################################################################ > + > +PYTHON_PYCLI_VERSION = devel This is a strange version, which probably indicates they will change the tarball contents without changing the version. You missed the hash file, which would allow to check this, so I've added the hash file. > +PYTHON_PYCLI_SOURCE = pyCLI-$(PYTHON_PYCLI_VERSION).tar.gz > +PYTHON_PYCLI_SITE = https://pypi.python.org/packages/source/p/pyCLI/ > +PYTHON_PYCLI_SETUP_TYPE = distutils First definition of <pkg>_SETUP_TYPE. > +PYTHON_PYCLI_LICENSE = Python software foundation license v2 > +PYTHON_PYCLI_LICENSE_FILES = LICENSE There was no file named LICENSE in the source directory. And the contents of the source tree was a bit fuzzy in terms of license, mentioning some code having Apache-2.0, some other MIT, but the main file indicating an ISC-like license. So I opted for that one as the license. > +PYTHON_PYCLI_SETUP_TYPE = setuptools And second definition of <pkg>_SETUP_TYPE. So I kept only this one. Also, as I've said, I've added a hash file as well. See http://git.buildroot.net/buildroot/commit/?id=1ec19e5e47486d2ba61564b37f1c8b6e62ec6d28 for the final commit. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/2] Add pyyaml package 2015-07-26 12:12 [Buildroot] [PATCH 1/2] Add pyyaml package Geoffrey Ragot 2015-07-26 12:12 ` [Buildroot] [PATCH 2/2] Add pycli package Geoffrey Ragot @ 2015-07-27 22:11 ` Thomas Petazzoni 1 sibling, 0 replies; 4+ messages in thread From: Thomas Petazzoni @ 2015-07-27 22:11 UTC (permalink / raw) To: buildroot Geoffrey, The commit title should have been: python-pyyaml: new package (just to follow the Buildroot conventions). On Sun, 26 Jul 2015 14:12:17 +0200, Geoffrey Ragot wrote: > diff --git a/package/python-pyyaml/Config.in b/package/python-pyyaml/Config.in > new file mode 100644 > index 0000000..a63c445 > --- /dev/null > +++ b/package/python-pyyaml/Config.in > @@ -0,0 +1,8 @@ > +comment "python-pyyaml require libyaml" > + depends on !BR2_PACKAGE_LIBYAML This comment is not needed... > + > +config BR2_PACKAGE_PYTHON_PYYAML > + bool "python-pyyaml" > + depends on BR2_PACKAGE_LIBYAML ... because 'select' should be used instead of 'depends on' for such dependencies. > + help > + The PyYAML package contains binding to the libyaml API. We normally include the URL of the upstream project here. > diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk > new file mode 100644 > index 0000000..ef10dac > --- /dev/null > +++ b/package/python-pyyaml/python-pyyaml.mk > @@ -0,0 +1,15 @@ > +################################################################################ > +# > +# python-pyyaml > +# > +################################################################################ > + > +PYTHON_PYYAML_VERSION = 3.11 > +PYTHON_PYYAML_SOURCE = PyYAML-$(PYTHON_PYYAML_VERSION).tar.gz > +PYTHON_PYYAML_SITE = https://pypi.python.org/packages/source/P/PyYAML/ > +PYTHON_PYYAML_SETUP_TYPE = distutils > +PYTHON_PYYAML_LICENSE = Python software foundation license v2 > +PYTHON_PYYAML_LICENSE_FILES = LICENSE > +PYTHON_PYYAML_DEPENDENCIES = libyaml > + > +$(eval $(python-package)) The .mk file was OK, however, the .hash file was missing. I've fixed those issues: [Thomas: - use 'select' rather than 'depends on' for the yaml library dependency - add hash file. - add upstream URL in Config.in help text - rework commit title.] and applied. Thanks for your contribution! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-07-28 22:04 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-07-26 12:12 [Buildroot] [PATCH 1/2] Add pyyaml package Geoffrey Ragot 2015-07-26 12:12 ` [Buildroot] [PATCH 2/2] Add pycli package Geoffrey Ragot 2015-07-28 22:04 ` Thomas Petazzoni 2015-07-27 22:11 ` [Buildroot] [PATCH 1/2] Add pyyaml package Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox