* [PATCH] debian-common: Add fdisk package as a dependency in wic
@ 2021-08-26 5:16 Srinuvasan A
2021-08-26 9:18 ` Henning Schild
2021-08-26 12:16 ` Srinuvasan A
0 siblings, 2 replies; 11+ messages in thread
From: Srinuvasan A @ 2021-08-26 5:16 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
When we build the bullseye images for de0-nano-soc target it's getting
failed in wic image creation step due to sfdisk required to build the
wic images.
We added --syste-id to WKS file and triggered the builds for buster as
well as bullseye, buster image got passed, might be that indirectly pull
the fdisk package but bullseye builds got failed hence added this
package as a dependency in wic imager install.
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
meta/conf/distro/debian-common.conf | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/conf/distro/debian-common.conf b/meta/conf/distro/debian-common.conf
index ba5262a..4c9b225 100644
--- a/meta/conf/distro/debian-common.conf
+++ b/meta/conf/distro/debian-common.conf
@@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
e2fsprogs \
python3 \
bmap-tools \
- tar"
+ tar \
+ fdisk"
GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] debian-common: Add fdisk package as a dependency in wic
2021-08-26 5:16 [PATCH] debian-common: Add fdisk package as a dependency in wic Srinuvasan A
@ 2021-08-26 9:18 ` Henning Schild
2021-08-26 12:16 ` Srinuvasan A
1 sibling, 0 replies; 11+ messages in thread
From: Henning Schild @ 2021-08-26 9:18 UTC (permalink / raw)
To: Srinuvasan A; +Cc: isar-users
Am Thu, 26 Aug 2021 10:46:03 +0530
schrieb Srinuvasan A <Srinuvasan_A@mentor.com>:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
>
> When we build the bullseye images for de0-nano-soc target it's getting
> failed in wic image creation step due to sfdisk required to build the
> wic images.
>
> We added --syste-id to WKS file and triggered the builds for buster as
--system-id
> well as bullseye, buster image got passed, might be that indirectly
> pull the fdisk package but bullseye builds got failed hence added this
> package as a dependency in wic imager install.
It is not might be, it is for sure. See
https://github.com/ilbers/isar/blob/03124cca669f50b682336a0bdf4ede5a4238e144/scripts/lib/wic/plugins/imager/direct.py#L592
So maybe change the wording here accordingly.
You could add
Fixes: c8eb30bfbff4 ("wic-img: import support for .wks.in files from oe-core")
So ack to the patch, but commit message should be improved.
Henning
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> ---
> meta/conf/distro/debian-common.conf | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/conf/distro/debian-common.conf
> b/meta/conf/distro/debian-common.conf index ba5262a..4c9b225 100644
> --- a/meta/conf/distro/debian-common.conf
> +++ b/meta/conf/distro/debian-common.conf
> @@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
> e2fsprogs \
> python3 \
> bmap-tools \
> - tar"
> + tar \
> + fdisk"
>
> GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
> GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH] debian-common: Add fdisk package as a dependency in wic
2021-08-26 5:16 [PATCH] debian-common: Add fdisk package as a dependency in wic Srinuvasan A
2021-08-26 9:18 ` Henning Schild
@ 2021-08-26 12:16 ` Srinuvasan A
2021-08-26 19:26 ` Henning Schild
` (3 more replies)
1 sibling, 4 replies; 11+ messages in thread
From: Srinuvasan A @ 2021-08-26 12:16 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
In downstream when we build the bullseye images for de0-nano-soc target it's getting
failed in wic image creation step due to sfdisk required to build the
wic images.
To reproduce this issue in upstream we triggered the de0-nano-soc build
with --system-id option enabled in wks.in file, this issue present in
upstream build hence added the dependency package fdisk to the wic
imager install.
Ref for de0-nano-soc.wks.in: https://github.com/ilbers/isar/commit/c8eb30bfbff4dc22c6d9633f33646e99590853d9
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
meta/conf/distro/debian-common.conf | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/conf/distro/debian-common.conf b/meta/conf/distro/debian-common.conf
index ba5262a..4c9b225 100644
--- a/meta/conf/distro/debian-common.conf
+++ b/meta/conf/distro/debian-common.conf
@@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
e2fsprogs \
python3 \
bmap-tools \
- tar"
+ tar \
+ fdisk"
GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] debian-common: Add fdisk package as a dependency in wic
2021-08-26 12:16 ` Srinuvasan A
@ 2021-08-26 19:26 ` Henning Schild
2021-08-27 13:05 ` [PATCH v3] " Srinuvasan A
` (2 subsequent siblings)
3 siblings, 0 replies; 11+ messages in thread
From: Henning Schild @ 2021-08-26 19:26 UTC (permalink / raw)
To: Srinuvasan A; +Cc: isar-users
Should be "v2" see "--subject-prefix" in "git send-email"
Am Thu, 26 Aug 2021 17:46:40 +0530
schrieb Srinuvasan A <Srinuvasan_A@mentor.com>:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
>
> In downstream when we build the bullseye images for de0-nano-soc
> target it's getting failed in wic image creation step due to sfdisk
> required to build the wic images.
>
> To reproduce this issue in upstream we triggered the de0-nano-soc
> build with --system-id option enabled in wks.in file, this issue
> present in upstream build hence added the dependency package fdisk to
> the wic imager install.
>
> Ref for de0-nano-soc.wks.in:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Filbers%2Fisar%2Fcommit%2Fc8eb30bfbff4dc22c6d9633f33646e99590853d9&data=04%7C01%7Cde173c00-e982-4fda-8644-47edf4671d63%40ad011.siemens.com%7C8d5cac41bb6b45120ee308d9688b6d6b%7C38ae3bcd95794fd4addab42e1495d55a%7C1%7C0%7C637655770766182740%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=2QWon3o9OYfdb%2BpQjlokZL8mkbbV4K8gT8MkuRa395c%3D&reserved=0
Forgive the f..ed up link. But naming a ref in the same repo is best
just done with that ref. So please just the sha maybe shortened to 8 or
12 leading chars.
gitk and other repo browsers will allow navigation when just using
the sha ... think of it as a much better relative link as opposed to
the absolute one you suggest here.
Henning
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> ---
> meta/conf/distro/debian-common.conf | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/conf/distro/debian-common.conf
> b/meta/conf/distro/debian-common.conf index ba5262a..4c9b225 100644
> --- a/meta/conf/distro/debian-common.conf
> +++ b/meta/conf/distro/debian-common.conf
> @@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
> e2fsprogs \
> python3 \
> bmap-tools \
> - tar"
> + tar \
> + fdisk"
>
> GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
> GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-08-26 12:16 ` Srinuvasan A
2021-08-26 19:26 ` Henning Schild
@ 2021-08-27 13:05 ` Srinuvasan A
2021-08-27 16:00 ` Henning Schild
2021-08-27 13:10 ` Srinuvasan A
2021-08-27 13:17 ` Srinuvasan A
3 siblings, 1 reply; 11+ messages in thread
From: Srinuvasan A @ 2021-08-27 13:05 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
In downstream when we build the bullseye images for de0-nano-soc target it's getting
failed in wic image creation step due to sfdisk required to build the
wic images.
To reproduce this issue in upstream we triggered the de0-nano-soc build
with --system-id option enabled in wks.in file, this issue present in
upstream build hence added the dependency package fdisk to the wic
imager install.
Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
meta/conf/distro/debian-common.conf | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/conf/distro/debian-common.conf b/meta/conf/distro/debian-common.conf
index ba5262a..4c9b225 100644
--- a/meta/conf/distro/debian-common.conf
+++ b/meta/conf/distro/debian-common.conf
@@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
e2fsprogs \
python3 \
bmap-tools \
- tar"
+ tar \
+ fdisk"
GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-08-27 13:05 ` [PATCH v3] " Srinuvasan A
@ 2021-08-27 16:00 ` Henning Schild
0 siblings, 0 replies; 11+ messages in thread
From: Henning Schild @ 2021-08-27 16:00 UTC (permalink / raw)
To: Srinuvasan A; +Cc: isar-users
Looking good now.
I guess we could switch the "default" distro of Isar from buster to
bullseye soon, that finding/fix will ease that step.
And introduce bookwork while at it.
Henning
Am Fri, 27 Aug 2021 18:35:37 +0530
schrieb Srinuvasan A <Srinuvasan_A@mentor.com>:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
>
> In downstream when we build the bullseye images for de0-nano-soc
> target it's getting failed in wic image creation step due to sfdisk
> required to build the wic images.
>
> To reproduce this issue in upstream we triggered the de0-nano-soc
> build with --system-id option enabled in wks.in file, this issue
> present in upstream build hence added the dependency package fdisk to
> the wic imager install.
>
> Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
>
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> ---
> meta/conf/distro/debian-common.conf | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/conf/distro/debian-common.conf
> b/meta/conf/distro/debian-common.conf index ba5262a..4c9b225 100644
> --- a/meta/conf/distro/debian-common.conf
> +++ b/meta/conf/distro/debian-common.conf
> @@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
> e2fsprogs \
> python3 \
> bmap-tools \
> - tar"
> + tar \
> + fdisk"
>
> GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
> GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-08-26 12:16 ` Srinuvasan A
2021-08-26 19:26 ` Henning Schild
2021-08-27 13:05 ` [PATCH v3] " Srinuvasan A
@ 2021-08-27 13:10 ` Srinuvasan A
2021-08-27 13:17 ` Srinuvasan A
3 siblings, 0 replies; 11+ messages in thread
From: Srinuvasan A @ 2021-08-27 13:10 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
In downstream when we build the bullseye images for de0-nano-soc target it's getting
failed in wic image creation step due to sfdisk required to build the
wic images.
To reproduce this issue in upstream we triggered the de0-nano-soc build
with --system-id option enabled in wks.in file, this issue present in
upstream build hence added the dependency package fdisk to the wic
imager install.
Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
meta/conf/distro/debian-common.conf | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/conf/distro/debian-common.conf b/meta/conf/distro/debian-common.conf
index ba5262a..4c9b225 100644
--- a/meta/conf/distro/debian-common.conf
+++ b/meta/conf/distro/debian-common.conf
@@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
e2fsprogs \
python3 \
bmap-tools \
- tar"
+ tar \
+ fdisk"
GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-08-26 12:16 ` Srinuvasan A
` (2 preceding siblings ...)
2021-08-27 13:10 ` Srinuvasan A
@ 2021-08-27 13:17 ` Srinuvasan A
2021-09-06 13:48 ` Anton Mikanovich
3 siblings, 1 reply; 11+ messages in thread
From: Srinuvasan A @ 2021-08-27 13:17 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
In downstream when we build the bullseye images for de0-nano-soc target it's getting
failed in wic image creation step due to sfdisk required to build the
wic images.
To reproduce this issue in upstream we triggered the de0-nano-soc build
with --system-id option enabled in wks.in file, this issue present in
upstream build hence added the dependency package fdisk to the wic
imager install.
Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
meta/conf/distro/debian-common.conf | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/conf/distro/debian-common.conf b/meta/conf/distro/debian-common.conf
index ba5262a..4c9b225 100644
--- a/meta/conf/distro/debian-common.conf
+++ b/meta/conf/distro/debian-common.conf
@@ -16,7 +16,8 @@ WIC_IMAGER_INSTALL = "parted \
e2fsprogs \
python3 \
bmap-tools \
- tar"
+ tar \
+ fdisk"
GRUB_BOOTLOADER_INSTALL_amd64 = "grub-efi-amd64-bin"
GRUB_BOOTLOADER_INSTALL_i386 = "grub-efi-ia32-bin"
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-08-27 13:17 ` Srinuvasan A
@ 2021-09-06 13:48 ` Anton Mikanovich
2021-09-06 18:41 ` Jan Kiszka
0 siblings, 1 reply; 11+ messages in thread
From: Anton Mikanovich @ 2021-09-06 13:48 UTC (permalink / raw)
To: Srinuvasan A, isar-users
27.08.2021 16:17, Srinuvasan A wrote:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
>
> In downstream when we build the bullseye images for de0-nano-soc target it's getting
> failed in wic image creation step due to sfdisk required to build the
> wic images.
>
> To reproduce this issue in upstream we triggered the de0-nano-soc build
> with --system-id option enabled in wks.in file, this issue present in
> upstream build hence added the dependency package fdisk to the wic
> imager install.
>
> Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
>
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
This will fail in stretch, because there is no such a package in it.
sfdisk was already included in util-linux, but moved to separate fdisk
package started from buster.
--
Anton Mikanovich
Promwad Ltd.
External service provider of ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn, Germany
+49 (89) 122 67 24-0
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-09-06 13:48 ` Anton Mikanovich
@ 2021-09-06 18:41 ` Jan Kiszka
2021-09-07 0:18 ` Srinuvasan Arjunan
0 siblings, 1 reply; 11+ messages in thread
From: Jan Kiszka @ 2021-09-06 18:41 UTC (permalink / raw)
To: Anton Mikanovich, Srinuvasan A, isar-users
On 06.09.21 15:48, Anton Mikanovich wrote:
> 27.08.2021 16:17, Srinuvasan A wrote:
>> From: Srinuvasan A <srinuvasan_a@mentor.com>
>>
>> In downstream when we build the bullseye images for de0-nano-soc
>> target it's getting
>> failed in wic image creation step due to sfdisk required to build the
>> wic images.
>>
>> To reproduce this issue in upstream we triggered the de0-nano-soc build
>> with --system-id option enabled in wks.in file, this issue present in
>> upstream build hence added the dependency package fdisk to the wic
>> imager install.
>>
>> Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
>>
>> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
>
> This will fail in stretch, because there is no such a package in it.
> sfdisk was already included in util-linux, but moved to separate fdisk
> package started from buster.
>
WIC_IMAGER_INSTALL_remove_stretch = "fdisk"
should fix that and can easily be dropped when we retire stretch.
Jan
--
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3] debian-common: Add fdisk package as a dependency in wic
2021-09-06 18:41 ` Jan Kiszka
@ 2021-09-07 0:18 ` Srinuvasan Arjunan
0 siblings, 0 replies; 11+ messages in thread
From: Srinuvasan Arjunan @ 2021-09-07 0:18 UTC (permalink / raw)
To: Jan Kiszka; +Cc: Anton Mikanovich, Srinuvasan A, isar-users
[-- Attachment #1: Type: text/plain, Size: 1624 bytes --]
Sure jan, will send next version of patch soon.
Many thanks,
Srinuvasan.A
On Tue, 7 Sep, 2021, 12:11 am Jan Kiszka, <jan.kiszka@siemens.com> wrote:
> On 06.09.21 15:48, Anton Mikanovich wrote:
> > 27.08.2021 16:17, Srinuvasan A wrote:
> >> From: Srinuvasan A <srinuvasan_a@mentor.com>
> >>
> >> In downstream when we build the bullseye images for de0-nano-soc
> >> target it's getting
> >> failed in wic image creation step due to sfdisk required to build the
> >> wic images.
> >>
> >> To reproduce this issue in upstream we triggered the de0-nano-soc build
> >> with --system-id option enabled in wks.in file, this issue present in
> >> upstream build hence added the dependency package fdisk to the wic
> >> imager install.
> >>
> >> Ref commit for de0-nano-soc.wks.in: c8eb30bfbff4
> >>
> >> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> >
> > This will fail in stretch, because there is no such a package in it.
> > sfdisk was already included in util-linux, but moved to separate fdisk
> > package started from buster.
> >
>
> WIC_IMAGER_INSTALL_remove_stretch = "fdisk"
>
> should fix that and can easily be dropped when we retire stretch.
>
> Jan
>
> --
> Siemens AG, T RDA IOT
> Corporate Competence Center Embedded Linux
>
> --
> You received this message because you are subscribed to the Google Groups
> "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to isar-users+unsubscribe@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/isar-users/f5dc1714-0eee-77e5-be30-035d82e0c2ed%40siemens.com
> .
>
[-- Attachment #2: Type: text/html, Size: 2771 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2021-09-07 0:18 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-26 5:16 [PATCH] debian-common: Add fdisk package as a dependency in wic Srinuvasan A
2021-08-26 9:18 ` Henning Schild
2021-08-26 12:16 ` Srinuvasan A
2021-08-26 19:26 ` Henning Schild
2021-08-27 13:05 ` [PATCH v3] " Srinuvasan A
2021-08-27 16:00 ` Henning Schild
2021-08-27 13:10 ` Srinuvasan A
2021-08-27 13:17 ` Srinuvasan A
2021-09-06 13:48 ` Anton Mikanovich
2021-09-06 18:41 ` Jan Kiszka
2021-09-07 0:18 ` Srinuvasan Arjunan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox