Am Donnerstag, dem 08.08.2024 um 12:16 +0000 schrieb Peter Kjellerstedt:
> > -----Original Message-----
> > From: [email protected] 
> > <[email protected]> On
> > Behalf Of Enrico Jörns
> > Sent: den 8 augusti 2024 13:04
> > To: [email protected]
> > Cc: [email protected]
> > Subject: [OE-core] [PATCH] archiver.bbclass: fix 
> > BB_GENERATE_MIRROR_TARBALLS checking
> > 
> > The variable 'have_mirror_tarballs' is used as a boolean while it is
> > actually the result of 'd.getVar('BB_GENERATE_MIRROR_TARBALLS')' and
> > thus a string.
> > 
> > Fix this by converting it into a boolean before using it.
> > 
> > Signed-off-by: Enrico Jörns <[email protected]>
> > ---
> >  meta/classes/archiver.bbclass | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass
> > index 9d286224d6..9c8b0daf31 100644
> > --- a/meta/classes/archiver.bbclass
> > +++ b/meta/classes/archiver.bbclass
> > @@ -339,7 +339,7 @@ python do_ar_mirror() {
> >      dl_dir = d.getVar('DL_DIR')
> >      mirror_exclusions = (d.getVar('ARCHIVER_MIRROR_EXCLUDE') or '').split()
> >      mirror_mode = d.getVarFlag('ARCHIVER_MODE', 'mirror')
> > -    have_mirror_tarballs = d.getVar('BB_GENERATE_MIRROR_TARBALLS')
> > +    have_mirror_tarballs = d.getVar('BB_GENERATE_MIRROR_TARBALLS') == "1"
> 
> May I suggest:
> 
>     have_mirror_tarballs = 
> oe.types.boolean(d.getVar('BB_GENERATE_MIRROR_TARBALLS'))
> 
> instead?

Oh, did not know this helper exists. Thank you for the hint!
Will send a v2.

Regards, Enrico

> > 
> >      if mirror_mode == 'combined':
> >          destdir = d.getVar('ARCHIVER_COMBINED_MIRRORDIR')
> > --
> > 2.39.2
> 
> //Peter
> 

-- 
Pengutronix e.K.                           | Enrico Jörns                |
Embedded Linux Consulting & Support        | https://www.pengutronix.de/ |
Steuerwalder Str. 21                       | Phone: +49-5121-206917-180  |
31137 Hildesheim, Germany                  | Fax:   +49-5121-206917-9    |
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#203159): 
https://lists.openembedded.org/g/openembedded-core/message/203159
Mute This Topic: https://lists.openembedded.org/mt/107787362/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to