From: Alexander Dahl via ptxdist <ptxdist@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Alexander Dahl <ada@thorsis.com>
Subject: [ptxdist] [PATCH v2 5/5] libubootenv: Make yaml support optional
Date: Fri, 19 Sep 2025 12:06:43 +0200 [thread overview]
Message-ID: <20250919100643.199174-6-ada@thorsis.com> (raw)
In-Reply-To: <20250919100643.199174-1-ada@thorsis.com>
If just used as drop in replacement with the old configuration format,
libyaml dependency is not required. You only need to enable it, if you
intend to use the new, extended configuration format.
Link: https://github.com/sbabic/libubootenv/blob/master/docs/fw_env_config.md#configuration-file-in-yaml
Signed-off-by: Alexander Dahl <ada@thorsis.com>
---
Notes:
v2:
- new patch, not in v1
rules/libubootenv.in | 8 +++++++-
rules/libubootenv.make | 2 +-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/rules/libubootenv.in b/rules/libubootenv.in
index 317a440fc..3bee47481 100644
--- a/rules/libubootenv.in
+++ b/rules/libubootenv.in
@@ -3,7 +3,7 @@
menuconfig LIBUBOOTENV
tristate
select HOST_CMAKE
- select LIBYAML
+ select LIBYAML if LIBUBOOTENV_YAML_CONFIG
select ZLIB
prompt "libubootenv "
help
@@ -21,6 +21,12 @@ config LIBUBOOTENV_TOOLS
help
Install fw_printenv and fw_setenv to target.
+config LIBUBOOTENV_YAML_CONFIG
+ bool
+ prompt "extended conf file format"
+ help
+ Support the new, extended, yaml based configuration file format.
+
comment "u-boot-tools is selected!"
depends on U_BOOT_TOOLS
diff --git a/rules/libubootenv.make b/rules/libubootenv.make
index d6753cd87..1ec86693a 100644
--- a/rules/libubootenv.make
+++ b/rules/libubootenv.make
@@ -35,7 +35,7 @@ LIBUBOOTENV_CONF_TOOL := cmake
LIBUBOOTENV_CONF_OPT := \
$(CROSS_CMAKE_USR) \
-DBUILD_DOC=OFF \
- -DNO_YML_SUPPORT=OFF
+ -DNO_YML_SUPPORT=$(call ptx/ifdef, PTXCONF_LIBUBOOTENV_YAML_CONFIG, OFF, ON)
# ----------------------------------------------------------------------------
# Target-Install
--
2.39.5
next prev parent reply other threads:[~2025-09-19 10:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-19 10:06 [ptxdist] [PATCH v2 0/5] libubootenv: New package to replace target u-boot-tools Alexander Dahl via ptxdist
2025-09-19 10:06 ` [ptxdist] [PATCH v2 1/5] libubootenv: Introduce new package Alexander Dahl via ptxdist
2025-09-22 11:04 ` [ptxdist] [APPLIED] " Michael Olbrich
2025-09-19 10:06 ` [ptxdist] [PATCH v2 2/5] libubootenv: Add option for installing tools Alexander Dahl via ptxdist
2025-09-22 11:04 ` [ptxdist] [APPLIED] " Michael Olbrich
2025-09-19 10:06 ` [ptxdist] [PATCH v2 3/5] libubootenv: Install config file to target Alexander Dahl via ptxdist
2025-09-22 11:04 ` [ptxdist] [APPLIED] " Michael Olbrich
2025-09-19 10:06 ` [ptxdist] [PATCH v2 4/5] libubootenv: Import some fixes from master Alexander Dahl via ptxdist
2025-09-22 11:04 ` [ptxdist] [APPLIED] " Michael Olbrich
2025-09-19 10:06 ` Alexander Dahl via ptxdist [this message]
2025-09-22 11:04 ` [ptxdist] [APPLIED] libubootenv: Make yaml support optional Michael Olbrich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250919100643.199174-6-ada@thorsis.com \
--to=ptxdist@pengutronix.de \
--cc=ada@thorsis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox