[OE-core] [PATCH] sanity.bbclass: Improved error message

[Bystricky, Juro]

No problem. Will send in patch V2 shortly.


> -----Original Message-----
> From: Richard Purdie [mailto:richard.purdie at linuxfoundation.org]
> Sent: Thursday, January 12, 2017 1:42 PM
> To: Bystricky, Juro <juro.bystricky at intel.com>; openembedded-
> core at lists.openembedded.org
> Cc: jurobystricky at hotmail.com
> Subject: Re: [PATCH] sanity.bbclass: Improved error message
> 
> On Thu, 2017-01-12 at 11:24 -0800, Juro Bystricky wrote:
> > When a non-existing MACHINE is specified, sanity check issues
> > the following message:
> >
> >     Please set a valid MACHINE in your local.conf or environment
> >
> > However, MACHINE can also be set in multiconfig .conf file(s).
> > Hence we may have several different MACHINE settings within one
> > (multiconfig) build, so the present error message is fairly
> > ambiguous.
> >
> > This patch remedies this by explicitly naming the offending MACHINE
> > and
> > adding conf/multiconfig/*.conf to the list of places where this
> > erroneous
> > MACHINE definition could have originated.
> >
> >     Invalid MACHINE=xyz. Please set a valid MACHINE in your
> > local.conf, conf/multiconfig/*.conf or environment
> 
> Can we change this to:
> 
> "MACHINE=xyz is invalid. Please set a valid MACHINE in your local.conf,
> environment or other configuration file."
> 
> I don't want to have to list all the possible places you could set
> MACHINE.
> 
> Cheers,
> 
> Richard