* [ptxdist] [PATCH] image-root-ext: add configuration option to set a volume label
@ 2024-06-06 17:00 Ian Abbott
2024-06-15 14:58 ` [ptxdist] [APPLIED] " Michael Olbrich
0 siblings, 1 reply; 2+ messages in thread
From: Ian Abbott @ 2024-06-06 17:00 UTC (permalink / raw)
To: ptxdist; +Cc: Ian Abbott
genimage supports volume labels for ext2/ext3/ext4 filesystems, so let
us allow it to be configured in PTXdist.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
config/images/ext.config | 1 +
platforms/image-root-ext.in | 8 ++++++++
rules/image-root-ext.make | 2 ++
3 files changed, 11 insertions(+)
diff --git a/config/images/ext.config b/config/images/ext.config
index 0ba21fc72..7082234b0 100644
--- a/config/images/ext.config
+++ b/config/images/ext.config
@@ -2,6 +2,7 @@
image @IMAGE@ {
@EXT_TYPE@ {
use-mke2fs = true
+ label = "@LABEL@"
}
size = @SIZE@
mountpoint = "/"
diff --git a/platforms/image-root-ext.in b/platforms/image-root-ext.in
index 950a59c41..a2c2b32cd 100644
--- a/platforms/image-root-ext.in
+++ b/platforms/image-root-ext.in
@@ -44,4 +44,12 @@ config IMAGE_ROOT_EXT_TYPE
default "ext3" if IMAGE_ROOT_EXT_EXT3
default "ext4" if IMAGE_ROOT_EXT_EXT4
+config IMAGE_ROOT_EXT_LABEL
+ string
+ default ""
+ prompt "volume label"
+ help
+ Set a volume label for the file system if the specified label
+ is non-empty. The maximum length of the label is 16 bytes.
+
endif
diff --git a/rules/image-root-ext.make b/rules/image-root-ext.make
index 4a51efa5c..e8eec464f 100644
--- a/rules/image-root-ext.make
+++ b/rules/image-root-ext.make
@@ -20,6 +20,7 @@ IMAGE_ROOT_EXT_IMAGE := $(IMAGEDIR)/root.ext2
IMAGE_ROOT_EXT_FILES := $(IMAGEDIR)/root.tgz
IMAGE_ROOT_EXT_CONFIG := ext.config
IMAGE_ROOT_EXT_SIZE := $(call remove_quotes,$(PTXCONF_IMAGE_ROOT_EXT_SIZE))
+IMAGE_ROOT_EXT_LABEL := $(call remove_quotes,$(PTXCONF_IMAGE_ROOT_EXT_LABEL))
# ----------------------------------------------------------------------------
# Image
@@ -35,6 +36,7 @@ IMAGE_ROOT_EXT_SCALE := $(subst %,,$(IMAGE_ROOT_EXT_SIZE))/80/1024/1024
IMAGE_ROOT_EXT_ENV += \
SIZE="$(shell echo $$(($(IMAGE_ROOT_EXT_BASE)*$(IMAGE_ROOT_EXT_SCALE)+1))M)"
endif
+IMAGE_ROOT_EXT_ENV += LABEL="$(IMAGE_ROOT_EXT_LABEL)"
ifdef PTXCONF_IMAGE_ROOT_EXT
$(IMAGE_ROOT_EXT_IMAGE):
--
2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [ptxdist] [APPLIED] image-root-ext: add configuration option to set a volume label
2024-06-06 17:00 [ptxdist] [PATCH] image-root-ext: add configuration option to set a volume label Ian Abbott
@ 2024-06-15 14:58 ` Michael Olbrich
0 siblings, 0 replies; 2+ messages in thread
From: Michael Olbrich @ 2024-06-15 14:58 UTC (permalink / raw)
To: ptxdist; +Cc: Ian Abbott
Thanks, applied as d08ea32b611a8fa472e00c3afc95c94a3ec39cfa.
Michael
[sent from post-receive hook]
On Sat, 15 Jun 2024 16:58:42 +0200, Ian Abbott <abbotti@mev.co.uk> wrote:
> genimage supports volume labels for ext2/ext3/ext4 filesystems, so let
> us allow it to be configured in PTXdist.
>
> Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
> Message-Id: <20240606170052.1255877-1-abbotti@mev.co.uk>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
>
> diff --git a/config/images/ext.config b/config/images/ext.config
> index 0ba21fc72e50..7082234b0440 100644
> --- a/config/images/ext.config
> +++ b/config/images/ext.config
> @@ -2,6 +2,7 @@
> image @IMAGE@ {
> @EXT_TYPE@ {
> use-mke2fs = true
> + label = "@LABEL@"
> }
> size = @SIZE@
> mountpoint = "/"
> diff --git a/platforms/image-root-ext.in b/platforms/image-root-ext.in
> index 950a59c41a89..a2c2b32cddc9 100644
> --- a/platforms/image-root-ext.in
> +++ b/platforms/image-root-ext.in
> @@ -44,4 +44,12 @@ config IMAGE_ROOT_EXT_TYPE
> default "ext3" if IMAGE_ROOT_EXT_EXT3
> default "ext4" if IMAGE_ROOT_EXT_EXT4
>
> +config IMAGE_ROOT_EXT_LABEL
> + string
> + default ""
> + prompt "volume label"
> + help
> + Set a volume label for the file system if the specified label
> + is non-empty. The maximum length of the label is 16 bytes.
> +
> endif
> diff --git a/rules/image-root-ext.make b/rules/image-root-ext.make
> index 4a51efa5c132..e8eec464fd52 100644
> --- a/rules/image-root-ext.make
> +++ b/rules/image-root-ext.make
> @@ -20,6 +20,7 @@ IMAGE_ROOT_EXT_IMAGE := $(IMAGEDIR)/root.ext2
> IMAGE_ROOT_EXT_FILES := $(IMAGEDIR)/root.tgz
> IMAGE_ROOT_EXT_CONFIG := ext.config
> IMAGE_ROOT_EXT_SIZE := $(call remove_quotes,$(PTXCONF_IMAGE_ROOT_EXT_SIZE))
> +IMAGE_ROOT_EXT_LABEL := $(call remove_quotes,$(PTXCONF_IMAGE_ROOT_EXT_LABEL))
>
> # ----------------------------------------------------------------------------
> # Image
> @@ -35,6 +36,7 @@ IMAGE_ROOT_EXT_SCALE := $(subst %,,$(IMAGE_ROOT_EXT_SIZE))/80/1024/1024
> IMAGE_ROOT_EXT_ENV += \
> SIZE="$(shell echo $$(($(IMAGE_ROOT_EXT_BASE)*$(IMAGE_ROOT_EXT_SCALE)+1))M)"
> endif
> +IMAGE_ROOT_EXT_ENV += LABEL="$(IMAGE_ROOT_EXT_LABEL)"
>
> ifdef PTXCONF_IMAGE_ROOT_EXT
> $(IMAGE_ROOT_EXT_IMAGE):
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-06-15 14:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-06-06 17:00 [ptxdist] [PATCH] image-root-ext: add configuration option to set a volume label Ian Abbott
2024-06-15 14:58 ` [ptxdist] [APPLIED] " Michael Olbrich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox