diff mbox series

[isar-cip-core,v3,2/2] conf/distro/cip-core-common.inc: Set dosfstools as build dependency

Message ID 20230926164021.30185-3-venkata.pyla@toshiba-tsip.com (mailing list archive)
State Superseded
Headers show
Series Add dosfstools recipe | expand

Commit Message

Venkata Pyla Sept. 26, 2023, 4:40 p.m. UTC
From: venkata pyla <venkata.pyla@toshiba-tsip.com>

custom dosfstools is included with fix that will help create DOS images
reproducibly.

Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
---
 conf/distro/cip-core-common.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Jan Kiszka Sept. 27, 2023, 7:04 a.m. UTC | #1
On 26.09.23 18:40, venkata.pyla@toshiba-tsip.com wrote:
> From: venkata pyla <venkata.pyla@toshiba-tsip.com>
> 
> custom dosfstools is included with fix that will help create DOS images
> reproducibly.
> 
> Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
> ---
>  conf/distro/cip-core-common.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/conf/distro/cip-core-common.inc b/conf/distro/cip-core-common.inc
> index 5d3ce10..5a158f2 100644
> --- a/conf/distro/cip-core-common.inc
> +++ b/conf/distro/cip-core-common.inc
> @@ -14,3 +14,4 @@ KERNEL_NAME ?= "cip"
>  WKS_FILE ?= "${MACHINE}.wks"
>  
>  CIP_IMAGE_OPTIONS ?= ""
> +IMAGER_BUILD_DEPS ?= "dosfstools"

Unfortunately, the patch does not apply to the buster version of
dosfstools, see e.g.
https://gitlab.com/cip-project/cip-core/isar-cip-core/-/jobs/5172706817.
We either need an adjusted patch for that release or need to skip buster.

Jan
Venkata Pyla Sept. 27, 2023, 7:16 a.m. UTC | #2
> -----Original Message-----
> From: Jan Kiszka <jan.kiszka@siemens.com>
> Sent: Wednesday, September 27, 2023 12:34 PM
> To: pyla venkata(TSIP TMIEC ODG Porting) <Venkata.Pyla@toshiba-
> tsip.com>; cip-dev@lists.cip-project.org
> Cc: dinesh kumar(TSIP TMIEC ODG Porting) <dinesh.kumar@toshiba-
> tsip.com>; hayashi kazuhiro(林 和宏 DME ○DIG□MPS○MP4)
> <kazuhiro3.hayashi@toshiba.co.jp>
> Subject: Re: [isar-cip-core v3 2/2] conf/distro/cip-core-common.inc: Set
> dosfstools as build dependency
> 
> On 26.09.23 18:40, venkata.pyla@toshiba-tsip.com wrote:
> > From: venkata pyla <venkata.pyla@toshiba-tsip.com>
> >
> > custom dosfstools is included with fix that will help create DOS
> > images reproducibly.
> >
> > Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
> > ---
> >  conf/distro/cip-core-common.inc | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/conf/distro/cip-core-common.inc
> > b/conf/distro/cip-core-common.inc index 5d3ce10..5a158f2 100644
> > --- a/conf/distro/cip-core-common.inc
> > +++ b/conf/distro/cip-core-common.inc
> > @@ -14,3 +14,4 @@ KERNEL_NAME ?= "cip"
> >  WKS_FILE ?= "${MACHINE}.wks"
> >
> >  CIP_IMAGE_OPTIONS ?= ""
> > +IMAGER_BUILD_DEPS ?= "dosfstools"
> 
> Unfortunately, the patch does not apply to the buster version of dosfstools,
> see e.g.
> https://gitlab.com/cip-project/cip-core/isar-cip-core/-/jobs/5172706817.
> We either need an adjusted patch for that release or need to skip buster.

sorry I didn't check this in buster before sending this patch, 
I think it is better to skip for buster, as reproducible builds are not achievable in buster and so it is not important.

Let me skip this in v4.

> 
> Jan
> 
> --
> Siemens AG, Technology
> Linux Expert Center
diff mbox series

Patch

diff --git a/conf/distro/cip-core-common.inc b/conf/distro/cip-core-common.inc
index 5d3ce10..5a158f2 100644
--- a/conf/distro/cip-core-common.inc
+++ b/conf/distro/cip-core-common.inc
@@ -14,3 +14,4 @@  KERNEL_NAME ?= "cip"
 WKS_FILE ?= "${MACHINE}.wks"
 
 CIP_IMAGE_OPTIONS ?= ""
+IMAGER_BUILD_DEPS ?= "dosfstools"