On Tue, 23 Mar 2021 at 21:53, Andre McCurdy <armccu...@gmail.com> wrote:
> > @@ -541,7 +541,10 @@ export BUILD_AR = "${BUILD_PREFIX}ar"
> >  export BUILD_AS = "${BUILD_PREFIX}as ${BUILD_AS_ARCH}"
> >  export BUILD_RANLIB = "${BUILD_PREFIX}ranlib"
> >  export BUILD_STRIP = "${BUILD_PREFIX}strip"
> > +export BUILD_OBJCOPY = "${BUILD_PREFIX}objcopy"
> > +export BUILD_OBJDUMP = "${BUILD_PREFIX}objdump"
> >  export BUILD_NM = "${BUILD_PREFIX}nm"
> > +export BUILD_READELF = "${BUILD_PREFIX}readelf"
>
> Do these new variables need to be exported?
>
> As far as I remember a few of the BUILD_xxx variables are "official"
> autotools variables which some autotools packages may expect to find
> in the environment (although googling now doesn't find any
> references).
>
> If so then the rest (i.e. most of them, and certainly these newly
> added ones) are OE specific and only used in OE recipes and classes...
> and therefore don't need to be exported here.

It was all in the name of consistency.

The Meson cross-file sets readelf, so to mirror the new native-file
also sets readelf.  There wasn't a BUILD_READELF and if someone is
changing BUILD_CC there's a good chance they'll be changing the entire
toochain, so I added any missing entries.

Yes, BUILD_* are effectively OE-specific so we could unexport the
entire set.  I'd prefer to do that than have some exported and some
not.

Ross
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149861): 
https://lists.openembedded.org/g/openembedded-core/message/149861
Mute This Topic: https://lists.openembedded.org/mt/81562047/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to