* [Buildroot] [PATCH 2/2] package/crudini: add host-crudini
2022-09-20 23:21 [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse KAMenyaev
@ 2022-09-20 23:21 ` Konstantin Menyaev via buildroot
2023-02-06 10:09 ` Heiko Thiery
2023-02-06 11:01 ` Yann E. MORIN
2023-02-06 10:12 ` [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse Heiko Thiery
2023-02-06 10:56 ` Yann E. MORIN
2 siblings, 2 replies; 8+ messages in thread
From: Konstantin Menyaev via buildroot @ 2022-09-20 23:21 UTC (permalink / raw)
To: buildroot
Cc: Konstantin Menyaev, Asaf Kahlon, Thomas Petazzoni,
sdfw_system_team, Zoltan Gyarmati
From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
Make the host variant of crudini package. This can be useful in
post-{build,image} scripts.
Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
---
package/Config.in.host | 1 +
package/crudini/Config.in.host | 9 +++++++++
package/crudini/crudini.mk | 1 +
3 files changed, 11 insertions(+)
create mode 100644 package/crudini/Config.in.host
diff --git a/package/Config.in.host b/package/Config.in.host
index fb805577d1..055e846e39 100644
--- a/package/Config.in.host
+++ b/package/Config.in.host
@@ -12,6 +12,7 @@ menu "Host utilities"
source "package/checksec/Config.in.host"
source "package/cmake/Config.in.host"
source "package/cramfs/Config.in.host"
+ source "package/crudini/Config.in.host"
source "package/cryptsetup/Config.in.host"
source "package/dbus-python/Config.in.host"
source "package/delve/Config.in.host"
diff --git a/package/crudini/Config.in.host b/package/crudini/Config.in.host
new file mode 100644
index 0000000000..5e0040883f
--- /dev/null
+++ b/package/crudini/Config.in.host
@@ -0,0 +1,9 @@
+config BR2_PACKAGE_HOST_CRUDINI
+ bool "host crudini"
+ depends on BR2_PACKAGE_HOST_PYTHON3
+ select BR2_PACKAGE_HOST_PYTHON_INIPARSE # runtime
+ help
+ A utility for handling ini files from the command line and
+ shell scripts
+
+ https://pypi.python.org/pypi/crudini
diff --git a/package/crudini/crudini.mk b/package/crudini/crudini.mk
index 7141e2b2a0..c376c5f777 100644
--- a/package/crudini/crudini.mk
+++ b/package/crudini/crudini.mk
@@ -11,3 +11,4 @@ CRUDINI_LICENSE = GPL-2.0
CRUDINI_LICENSE_FILES = COPYING
$(eval $(python-package))
+$(eval $(host-python-package))
--
2.37.3
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [Buildroot] [PATCH 2/2] package/crudini: add host-crudini
2022-09-20 23:21 ` [Buildroot] [PATCH 2/2] package/crudini: add host-crudini Konstantin Menyaev via buildroot
@ 2023-02-06 10:09 ` Heiko Thiery
2023-02-06 11:01 ` Yann E. MORIN
1 sibling, 0 replies; 8+ messages in thread
From: Heiko Thiery @ 2023-02-06 10:09 UTC (permalink / raw)
To: KAMenyaev
Cc: Zoltan Gyarmati, sdfw_system_team, Thomas Petazzoni, Asaf Kahlon,
buildroot
Hi,
Am Mi., 21. Sept. 2022 um 01:24 Uhr schrieb Konstantin Menyaev via
buildroot <buildroot@buildroot.org>:
>
> From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
>
> Make the host variant of crudini package. This can be useful in
> post-{build,image} scripts.
>
> Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
Reviewed-by: Heiko Thiery <heiko.thiery@gmail.com>
> ---
> package/Config.in.host | 1 +
> package/crudini/Config.in.host | 9 +++++++++
> package/crudini/crudini.mk | 1 +
> 3 files changed, 11 insertions(+)
> create mode 100644 package/crudini/Config.in.host
>
> diff --git a/package/Config.in.host b/package/Config.in.host
> index fb805577d1..055e846e39 100644
> --- a/package/Config.in.host
> +++ b/package/Config.in.host
> @@ -12,6 +12,7 @@ menu "Host utilities"
> source "package/checksec/Config.in.host"
> source "package/cmake/Config.in.host"
> source "package/cramfs/Config.in.host"
> + source "package/crudini/Config.in.host"
> source "package/cryptsetup/Config.in.host"
> source "package/dbus-python/Config.in.host"
> source "package/delve/Config.in.host"
> diff --git a/package/crudini/Config.in.host b/package/crudini/Config.in.host
> new file mode 100644
> index 0000000000..5e0040883f
> --- /dev/null
> +++ b/package/crudini/Config.in.host
> @@ -0,0 +1,9 @@
> +config BR2_PACKAGE_HOST_CRUDINI
> + bool "host crudini"
> + depends on BR2_PACKAGE_HOST_PYTHON3
> + select BR2_PACKAGE_HOST_PYTHON_INIPARSE # runtime
> + help
> + A utility for handling ini files from the command line and
> + shell scripts
> +
> + https://pypi.python.org/pypi/crudini
> diff --git a/package/crudini/crudini.mk b/package/crudini/crudini.mk
> index 7141e2b2a0..c376c5f777 100644
> --- a/package/crudini/crudini.mk
> +++ b/package/crudini/crudini.mk
> @@ -11,3 +11,4 @@ CRUDINI_LICENSE = GPL-2.0
> CRUDINI_LICENSE_FILES = COPYING
>
> $(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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Buildroot] [PATCH 2/2] package/crudini: add host-crudini
2022-09-20 23:21 ` [Buildroot] [PATCH 2/2] package/crudini: add host-crudini Konstantin Menyaev via buildroot
2023-02-06 10:09 ` Heiko Thiery
@ 2023-02-06 11:01 ` Yann E. MORIN
1 sibling, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2023-02-06 11:01 UTC (permalink / raw)
To: KAMenyaev
Cc: Zoltan Gyarmati, sdfw_system_team, Thomas Petazzoni, Asaf Kahlon,
buildroot
Konstantin, All,
On 2022-09-21 02:21 +0300, Konstantin Menyaev via buildroot spake thusly:
> From: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
>
> Make the host variant of crudini package. This can be useful in
> post-{build,image} scripts.
>
> Signed-off-by: Konstantin Menyaev <KAMenyaev@sberdevices.ru>
> ---
[--SNIP--]
> diff --git a/package/crudini/Config.in.host b/package/crudini/Config.in.host
> new file mode 100644
> index 0000000000..5e0040883f
> --- /dev/null
> +++ b/package/crudini/Config.in.host
> @@ -0,0 +1,9 @@
> +config BR2_PACKAGE_HOST_CRUDINI
> + bool "host crudini"
> + depends on BR2_PACKAGE_HOST_PYTHON3
I've changed that to a select, like allthe other host packages that need
host python3, and applied to master, thanks.
Regards,
Yann E. MORIN.
> + select BR2_PACKAGE_HOST_PYTHON_INIPARSE # runtime
> + help
> + A utility for handling ini files from the command line and
> + shell scripts
> +
> + https://pypi.python.org/pypi/crudini
> diff --git a/package/crudini/crudini.mk b/package/crudini/crudini.mk
> index 7141e2b2a0..c376c5f777 100644
> --- a/package/crudini/crudini.mk
> +++ b/package/crudini/crudini.mk
> @@ -11,3 +11,4 @@ CRUDINI_LICENSE = GPL-2.0
> CRUDINI_LICENSE_FILES = COPYING
>
> $(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 | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse
2022-09-20 23:21 [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse KAMenyaev
2022-09-20 23:21 ` [Buildroot] [PATCH 2/2] package/crudini: add host-crudini Konstantin Menyaev via buildroot
@ 2023-02-06 10:12 ` Heiko Thiery
2023-02-06 11:32 ` James Hilliard
2023-02-06 10:56 ` Yann E. MORIN
2 siblings, 1 reply; 8+ messages in thread
From: Heiko Thiery @ 2023-02-06 10:12 UTC (permalink / raw)
To: KAMenyaev
Cc: Zoltan Gyarmati, sdfw_system_team, Thomas Petazzoni, Asaf Kahlon,
buildroot
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
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse
2023-02-06 10:12 ` [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse Heiko Thiery
@ 2023-02-06 11:32 ` James Hilliard
2023-02-06 11:38 ` Yann E. MORIN
0 siblings, 1 reply; 8+ messages in thread
From: James Hilliard @ 2023-02-06 11:32 UTC (permalink / raw)
To: Heiko Thiery
Cc: sdfw_system_team, KAMenyaev, Asaf Kahlon, Thomas Petazzoni,
buildroot, Zoltan Gyarmati
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
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse
2023-02-06 11:32 ` James Hilliard
@ 2023-02-06 11:38 ` Yann E. MORIN
0 siblings, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2023-02-06 11:38 UTC (permalink / raw)
To: James Hilliard
Cc: sdfw_system_team, KAMenyaev, Asaf Kahlon, Thomas Petazzoni,
buildroot, Zoltan Gyarmati, Heiko Thiery
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.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse
2022-09-20 23:21 [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse KAMenyaev
2022-09-20 23:21 ` [Buildroot] [PATCH 2/2] package/crudini: add host-crudini Konstantin Menyaev via buildroot
2023-02-06 10:12 ` [Buildroot] [PATCH 1/2] package/python-iniparse: add host-python-iniparse Heiko Thiery
@ 2023-02-06 10:56 ` Yann E. MORIN
2 siblings, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2023-02-06 10:56 UTC (permalink / raw)
To: KAMenyaev
Cc: Zoltan Gyarmati, sdfw_system_team, Thomas Petazzoni, Asaf Kahlon,
buildroot
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
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 8+ messages in thread