[OE-core] [PATCH 2/4] autotools.bbclass: Report the missing configure path

Khem Raj raj.khem at gmail.com
Wed Nov 30 22:48:56 UTC 2011


>>> -               bbfatal "no configure script found"
>>> +               bbfatal "no configure script found at $cfgscript"
>>
>> why not just do one line change something like
>>
>> bbfatal "configure script ${S}/configure not found"
>
> I prefer to use a variable when I need a value more than once. It avoids
> future bugs where one is changed and the others were missed. As this
> string is used 4 times, it seemed like the right call.

ok with variable still the cfgscript itself is the script and the
message "no configure script found at $cfgscript"
seems like $cfgscript is a location. you may want to make it
"$cfgscript configure script not found" or something




More information about the Openembedded-core mailing list