[PATCH] classes/image-postproc: Fix unchangeable VARIAN_VERSION value

3 views
Skip to first unread message

Anton Mikanovich

unread,
Apr 26, 2025, 7:42:22 AM4/26/25
to isar-...@googlegroups.com, Anton Mikanovich
During custom /etc/os-release generation in update_etc_os_release()
VARIAN_VERSION was set to PV without taking in account --version option
provided from outside.

Fixes: 78b24427a050 ("classes/image-postproc: Add image version")

Signed-off-by: Anton Mikanovich <ami...@ilbers.de>
---
meta/classes/image-postproc-extension.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/image-postproc-extension.bbclass b/meta/classes/image-postproc-extension.bbclass
index f7f762c4..991bac4c 100644
--- a/meta/classes/image-postproc-extension.bbclass
+++ b/meta/classes/image-postproc-extension.bbclass
@@ -28,7 +28,7 @@ update_etc_os_release() {
fi
if [ -n "${OS_RELEASE_VARIANT_VERSION}" ]; then
sudo sed -i '/^VARIANT_VERSION=.*/d' '${IMAGE_ROOTFS}/etc/os-release'
- echo "VARIANT_VERSION=\"${PV}\"" | \
+ echo "VARIANT_VERSION=\"${OS_RELEASE_VARIANT_VERSION}\"" | \
sudo tee -a '${IMAGE_ROOTFS}/etc/os-release'
fi
}
--
2.34.1

Jan Kiszka

unread,
Apr 27, 2025, 1:46:08 PM4/27/25
to Anton Mikanovich, isar-...@googlegroups.com, Quirin Gylstorff
On 26.04.25 13:42, Anton Mikanovich wrote:
> During custom /etc/os-release generation in update_etc_os_release()
> VARIAN_VERSION was set to PV without taking in account --version option

"VARIANT" - here and in the title.

Jan

> provided from outside.
>
> Fixes: 78b24427a050 ("classes/image-postproc: Add image version")
>
> Signed-off-by: Anton Mikanovich <ami...@ilbers.de>
> ---
> meta/classes/image-postproc-extension.bbclass | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/image-postproc-extension.bbclass b/meta/classes/image-postproc-extension.bbclass
> index f7f762c4..991bac4c 100644
> --- a/meta/classes/image-postproc-extension.bbclass
> +++ b/meta/classes/image-postproc-extension.bbclass
> @@ -28,7 +28,7 @@ update_etc_os_release() {
> fi
> if [ -n "${OS_RELEASE_VARIANT_VERSION}" ]; then
> sudo sed -i '/^VARIANT_VERSION=.*/d' '${IMAGE_ROOTFS}/etc/os-release'
> - echo "VARIANT_VERSION=\"${PV}\"" | \
> + echo "VARIANT_VERSION=\"${OS_RELEASE_VARIANT_VERSION}\"" | \
> sudo tee -a '${IMAGE_ROOTFS}/etc/os-release'
> fi
> }

--
Siemens AG, Foundational Technologies
Linux Expert Center

Anton Mikanovich

unread,
Apr 28, 2025, 2:11:22 AM4/28/25
to Jan Kiszka, isar-...@googlegroups.com, Uladzimir Bely
27/04/2025 20:46, Jan Kiszka wrote:
> On 26.04.25 13:42, Anton Mikanovich wrote:
>> During custom /etc/os-release generation in update_etc_os_release()
>> VARIAN_VERSION was set to PV without taking in account --version option
> "VARIANT" - here and in the title.
>
> Jan

Thanks, will fix during the merge if no other changes needed.

Baurzhan Ismagulov

unread,
May 11, 2025, 10:23:35 AM5/11/25
to isar-...@googlegroups.com
On 2025-04-26 14:42, Anton Mikanovich wrote:
> During custom /etc/os-release generation in update_etc_os_release()
> VARIAN_VERSION was set to PV without taking in account --version option
> provided from outside.
>
> Fixes: 78b24427a050 ("classes/image-postproc: Add image version")

Changed to VARIANT_VERSION on two occasions, applied to next, thanks.

With kind regards,
Baurzhan
Reply all
Reply to author
Forward
0 new messages