diff mbox series

[1/2] package/python-iniparse: add host-python-iniparse

Message ID 20220920232154.1219792-1-KAMenyaev@sberdevices.ru
State Accepted
Headers show
Series [1/2] package/python-iniparse: add host-python-iniparse | expand

Commit Message

Konstantin Menyaev Sept. 20, 2022, 11:21 p.m. UTC
From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>

Make the host variant of python-iniparse
as host-crudini dependency.

Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
---
 package/Config.in.host                     |  1 +
 package/python-iniparse/Config.in.host     | 11 +++++++++++
 package/python-iniparse/python-iniparse.mk |  1 +
 3 files changed, 13 insertions(+)
 create mode 100644 package/python-iniparse/Config.in.host

Comments

Heiko Thiery Feb. 6, 2023, 10:12 a.m. UTC | #1
Hi,

Am Mi., 21. Sept. 2022 um 01:24 Uhr schrieb <KAMenyaev@sberdevices.ru>:
>
> From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
>
> Make the host variant of python-iniparse
> as host-crudini dependency.
>
> Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>

Reviewed-by: Heiko Thiery <heiko.thiery@gmail.com>

> ---
>  package/Config.in.host                     |  1 +
>  package/python-iniparse/Config.in.host     | 11 +++++++++++
>  package/python-iniparse/python-iniparse.mk |  1 +
>  3 files changed, 13 insertions(+)
>  create mode 100644 package/python-iniparse/Config.in.host
>
> diff --git a/package/Config.in.host b/package/Config.in.host
> index f437ef680c..fb805577d1 100644
> --- a/package/Config.in.host
> +++ b/package/Config.in.host
> @@ -72,6 +72,7 @@ menu "Host utilities"
>         source "package/pkgconf/Config.in.host"
>         source "package/pru-software-support/Config.in.host"
>         source "package/pwgen/Config.in.host"
> +       source "package/python-iniparse/Config.in.host
>         source "package/python-cython/Config.in.host"
>         source "package/python-greenlet/Config.in.host"
>         source "package/python-kflash/Config.in.host"
> diff --git a/package/python-iniparse/Config.in.host b/package/python-iniparse/Config.in.host
> new file mode 100644
> index 0000000000..e93022e815
> --- /dev/null
> +++ b/package/python-iniparse/Config.in.host
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_HOST_PYTHON_INIPARSE
> +       bool "host python-iniparse"
> +       select BR2_PACKAGE_HOST_PYTHON_SIX # runtime
> +       help
> +         iniparse is an INI parser for Python which is API compatible
> +         with the standard library's ConfigParser, preserves
> +         structure of INI files (order of sections & options,
> +         indentation, comments, and blank lines are preserved when
> +         data is updated), and is more convenient to use.
> +
> +         https://pypi.python.org/pypi/iniparse
> diff --git a/package/python-iniparse/python-iniparse.mk b/package/python-iniparse/python-iniparse.mk
> index d31a490bd8..e76987d1e9 100644
> --- a/package/python-iniparse/python-iniparse.mk
> +++ b/package/python-iniparse/python-iniparse.mk
> @@ -12,3 +12,4 @@ PYTHON_INIPARSE_LICENSE_FILES = LICENSE-PSF LICENSE
>  PYTHON_INIPARSE_SETUP_TYPE = setuptools
>
>  $(eval $(python-package))
> +$(eval $(host-python-package))
> --
> 2.37.3
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Yann E. MORIN Feb. 6, 2023, 10:56 a.m. UTC | #2
Konstantin, All,

On 2022-09-21 02:21 +0300, KAMenyaev@sberdevices.ru spake thusly:
> From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
> 
> Make the host variant of python-iniparse
> as host-crudini dependency.

I added a note why we need a Config.in.host entry (usually, for libs, we
do not add one, but here we must slect pythn-six, which has one).

> Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
> ---
[--SNIP--]
> diff --git a/package/Config.in.host b/package/Config.in.host
> index f437ef680c..fb805577d1 100644
> --- a/package/Config.in.host
> +++ b/package/Config.in.host
> @@ -72,6 +72,7 @@ menu "Host utilities"
>  	source "package/pkgconf/Config.in.host"
>  	source "package/pru-software-support/Config.in.host"
>  	source "package/pwgen/Config.in.host"
> +	source "package/python-iniparse/Config.in.host

I reordered this in alphabetical order.

Applied to master, thanks.

Regards,
Yann E. MORIN.

>  	source "package/python-cython/Config.in.host"
>  	source "package/python-greenlet/Config.in.host"
>  	source "package/python-kflash/Config.in.host"
> diff --git a/package/python-iniparse/Config.in.host b/package/python-iniparse/Config.in.host
> new file mode 100644
> index 0000000000..e93022e815
> --- /dev/null
> +++ b/package/python-iniparse/Config.in.host
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_HOST_PYTHON_INIPARSE
> +	bool "host python-iniparse"
> +	select BR2_PACKAGE_HOST_PYTHON_SIX # runtime
> +	help
> +	  iniparse is an INI parser for Python which is API compatible
> +	  with the standard library's ConfigParser, preserves
> +	  structure of INI files (order of sections & options,
> +	  indentation, comments, and blank lines are preserved when
> +	  data is updated), and is more convenient to use.
> +
> +	  https://pypi.python.org/pypi/iniparse
> diff --git a/package/python-iniparse/python-iniparse.mk b/package/python-iniparse/python-iniparse.mk
> index d31a490bd8..e76987d1e9 100644
> --- a/package/python-iniparse/python-iniparse.mk
> +++ b/package/python-iniparse/python-iniparse.mk
> @@ -12,3 +12,4 @@ PYTHON_INIPARSE_LICENSE_FILES = LICENSE-PSF LICENSE
>  PYTHON_INIPARSE_SETUP_TYPE = setuptools
>  
>  $(eval $(python-package))
> +$(eval $(host-python-package))
> -- 
> 2.37.3
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
James Hilliard Feb. 6, 2023, 11:32 a.m. UTC | #3
On Mon, Feb 6, 2023 at 3:12 AM Heiko Thiery <heiko.thiery@gmail.com> wrote:
>
> Hi,
>
> Am Mi., 21. Sept. 2022 um 01:24 Uhr schrieb <KAMenyaev@sberdevices.ru>:
> >
> > From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
> >
> > Make the host variant of python-iniparse
> > as host-crudini dependency.
> >
> > Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
>
> Reviewed-by: Heiko Thiery <heiko.thiery@gmail.com>
>
> > ---
> >  package/Config.in.host                     |  1 +
> >  package/python-iniparse/Config.in.host     | 11 +++++++++++
> >  package/python-iniparse/python-iniparse.mk |  1 +
> >  3 files changed, 13 insertions(+)
> >  create mode 100644 package/python-iniparse/Config.in.host
> >
> > diff --git a/package/Config.in.host b/package/Config.in.host
> > index f437ef680c..fb805577d1 100644
> > --- a/package/Config.in.host
> > +++ b/package/Config.in.host
> > @@ -72,6 +72,7 @@ menu "Host utilities"
> >         source "package/pkgconf/Config.in.host"
> >         source "package/pru-software-support/Config.in.host"
> >         source "package/pwgen/Config.in.host"
> > +       source "package/python-iniparse/Config.in.host

Missing a trailing quote:
package/Config.in.host:81:warning: multi-line strings not supported

> >         source "package/python-cython/Config.in.host"
> >         source "package/python-greenlet/Config.in.host"
> >         source "package/python-kflash/Config.in.host"
> > diff --git a/package/python-iniparse/Config.in.host b/package/python-iniparse/Config.in.host
> > new file mode 100644
> > index 0000000000..e93022e815
> > --- /dev/null
> > +++ b/package/python-iniparse/Config.in.host
> > @@ -0,0 +1,11 @@
> > +config BR2_PACKAGE_HOST_PYTHON_INIPARSE
> > +       bool "host python-iniparse"
> > +       select BR2_PACKAGE_HOST_PYTHON_SIX # runtime
> > +       help
> > +         iniparse is an INI parser for Python which is API compatible
> > +         with the standard library's ConfigParser, preserves
> > +         structure of INI files (order of sections & options,
> > +         indentation, comments, and blank lines are preserved when
> > +         data is updated), and is more convenient to use.
> > +
> > +         https://pypi.python.org/pypi/iniparse
> > diff --git a/package/python-iniparse/python-iniparse.mk b/package/python-iniparse/python-iniparse.mk
> > index d31a490bd8..e76987d1e9 100644
> > --- a/package/python-iniparse/python-iniparse.mk
> > +++ b/package/python-iniparse/python-iniparse.mk
> > @@ -12,3 +12,4 @@ PYTHON_INIPARSE_LICENSE_FILES = LICENSE-PSF LICENSE
> >  PYTHON_INIPARSE_SETUP_TYPE = setuptools
> >
> >  $(eval $(python-package))
> > +$(eval $(host-python-package))
> > --
> > 2.37.3
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Yann E. MORIN Feb. 6, 2023, 11:38 a.m. UTC | #4
James, All,

On 2023-02-06 04:32 -0700, James Hilliard spake thusly:
> On Mon, Feb 6, 2023 at 3:12 AM Heiko Thiery <heiko.thiery@gmail.com> wrote:
> > Am Mi., 21. Sept. 2022 um 01:24 Uhr schrieb <KAMenyaev@sberdevices.ru>:
[--SNIP--]
> > > diff --git a/package/Config.in.host b/package/Config.in.host
> > > index f437ef680c..fb805577d1 100644
> > > --- a/package/Config.in.host
> > > +++ b/package/Config.in.host
> > > @@ -72,6 +72,7 @@ menu "Host utilities"
> > >         source "package/pkgconf/Config.in.host"
> > >         source "package/pru-software-support/Config.in.host"
> > >         source "package/pwgen/Config.in.host"
> > > +       source "package/python-iniparse/Config.in.host
> Missing a trailing quote:
> package/Config.in.host:81:warning: multi-line strings not supported

Fixed, thanks!

Regards,
Yann E. MORIN.
diff mbox series

Patch

diff --git a/package/Config.in.host b/package/Config.in.host
index f437ef680c..fb805577d1 100644
--- a/package/Config.in.host
+++ b/package/Config.in.host
@@ -72,6 +72,7 @@  menu "Host utilities"
 	source "package/pkgconf/Config.in.host"
 	source "package/pru-software-support/Config.in.host"
 	source "package/pwgen/Config.in.host"
+	source "package/python-iniparse/Config.in.host
 	source "package/python-cython/Config.in.host"
 	source "package/python-greenlet/Config.in.host"
 	source "package/python-kflash/Config.in.host"
diff --git a/package/python-iniparse/Config.in.host b/package/python-iniparse/Config.in.host
new file mode 100644
index 0000000000..e93022e815
--- /dev/null
+++ b/package/python-iniparse/Config.in.host
@@ -0,0 +1,11 @@ 
+config BR2_PACKAGE_HOST_PYTHON_INIPARSE
+	bool "host python-iniparse"
+	select BR2_PACKAGE_HOST_PYTHON_SIX # runtime
+	help
+	  iniparse is an INI parser for Python which is API compatible
+	  with the standard library's ConfigParser, preserves
+	  structure of INI files (order of sections & options,
+	  indentation, comments, and blank lines are preserved when
+	  data is updated), and is more convenient to use.
+
+	  https://pypi.python.org/pypi/iniparse
diff --git a/package/python-iniparse/python-iniparse.mk b/package/python-iniparse/python-iniparse.mk
index d31a490bd8..e76987d1e9 100644
--- a/package/python-iniparse/python-iniparse.mk
+++ b/package/python-iniparse/python-iniparse.mk
@@ -12,3 +12,4 @@  PYTHON_INIPARSE_LICENSE_FILES = LICENSE-PSF LICENSE
 PYTHON_INIPARSE_SETUP_TYPE = setuptools
 
 $(eval $(python-package))
+$(eval $(host-python-package))