[oe] [RFC] A case of distro config overriding DISTRO var

Paul Sokolovsky pmiscml at gmail.com
Sat Feb 9 19:27:26 UTC 2008


Hello,

On Sat, 09 Feb 2008 19:04:33 +0000
Richard Purdie <rpurdie at rpsys.net> wrote:

> 
> On Sat, 2008-02-09 at 21:04 +0200, Paul Sokolovsky wrote:
> > On Sat, 09 Feb 2008 18:42:49 +0000
> > Richard Purdie <rpurdie at rpsys.net> wrote:
> > > On Sat, 2008-02-09 at 11:56 +0200, Paul Sokolovsky wrote:
> > > > ============================================================
> > > > --- classes/image.bbclass
> > > > 4d92381733b09bae07a07da35bd0024744924c3c +++
> > > > classes/image.bbclass
> > > > 6eeae6a01c36465bc36cc737d42ea6b8ab09b152 @@ -90,6 +90,7 @@
> > > > fakeroot do_rootfs () { ${IMAGE_PREPROCESS_COMMAND} 
> > > >  	export TOPDIR=${TOPDIR}
> > > > +	export DISTRO=${USERDISTRO}
> > > >  	export MACHINE=${MACHINE}
> > > 
> > > This could be rather confusing, can we not have:
> > > 
> > > USERDISTRO=${USERDISTRO}
> > > 
> > > then its obvious which variable we mean?
> > 
> > Not sure I understand. USERDISTRO is the variable which holds *real*
> > DISTRO value, and here, we just use that real value to set a
> > top-level parameter DISTRO for a tool which expects such top-level
> > parameter (bbimage in this case).
> 
> Can it not expect USERDISTRO though? Where exactly is this being used?
> 
> I just worry that people are going to get confused if we call this
> DISTRO since its not the value in the "DISTRO" variable...

Well, it *is* value of DISTRO variable. Secondly, how you imagine that
- bitbake accepts some notion as DISTRO variable, yet bbimage as
USERDISTRO variable? And people should not be confused or even
concerned with all this stuff - it's just internal OE workings made to
make external whitebox interface (DISTRO being a part of it) work as
it should.

Well, after all, bitbake is your area. Pragmatically, I of course would 
prefer to avoid waiting for next bitbake release to get this right
though. Ah, and why bbimage needs DISTRO set properly? It just loads
bitbake.conf and that references ${DISTRO}.conf.

> 
> Cheers,
> 
> Richard
> 
> 
> 



-- 
Best regards,
 Paul                          mailto:pmiscml at gmail.com




More information about the Openembedded-devel mailing list