[OE-core] [PATCH 1/3] sanity.bbclass: Check for the known broken version of make
Mark Hatle
mark.hatle at windriver.com
Tue Jun 25 02:06:11 UTC 2013
On 6/24/13 5:09 PM, Saul Wold wrote:
> On 06/24/2013 07:45 AM, Mark Hatle wrote:
>> See GNU Savannah bug 30612 -- make 3.82 is known to be broken.
>>
>> A number of vendors are providing a modified version, so checking
>> for just the version string is not enough. We also need to check
>> if the patch for the issue has been applied. We use a modified
>> version of the reproduced to check for the issue.
>>
>> Signed-off-by: Mark Hatle <mark.hatle at windriver.com>
>> ---
>> meta/classes/sanity.bbclass | 39 +++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 39 insertions(+)
>>
>
> Seems this needs a rebase since RP also changed sanity.bbclass recently.
This is from Friday morning. I'll start a rebase and send new code when I have
it ready.
--Mark
> Sau!
>
>> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
>> index 7f95f31..dc251a1 100644
>> --- a/meta/classes/sanity.bbclass
>> +++ b/meta/classes/sanity.bbclass
>> @@ -336,6 +336,41 @@ def check_gcc_march(sanity_data):
>>
>> return result
>>
>> +# Unpatched versions of make 3.82 are known to be broken. See GNU Savannah Bug 30612.
>> +# Use a modified reproducer from http://savannah.gnu.org/bugs/?30612 to validate.
>> +def check_make_version(sanity_data, loosever):
>> + status, result = oe.utils.getstatusoutput("make --version")
>> + if status != 0:
>> + return "Unable to execute make --version, exit code %s\n" % status
>> + version = result.split()[2]
>> + if loosever(version) == loosever("3.82"):
>> + # Construct a test file
>> + f = open("makefile_test", "w")
>> + f.write("makefile_test.a: makefile_test_a.c makefile_test_b.c makefile_test.a( makefile_test_a.c makefile_test_b.c)\n")
>> + f.write("\n")
>> + f.write("makefile_test_a.c:\n")
>> + f.write(" touch $@\n")
>> + f.write("\n")
>> + f.write("makefile_test_b.c:\n")
>> + f.write(" touch $@\n")
>> + f.close()
>> +
>> + # Check if make 3.82 has been patched
>> + status,result = oe.utils.getstatusoutput("make -f makefile_test")
>> +
>> + os.remove("makefile_test")
>> + if os.path.exists("makefile_test_a.c"):
>> + os.remove("makefile_test_a.c")
>> + if os.path.exists("makefile_test_b.c"):
>> + os.remove("makefile_test_b.c")
>> + if os.path.exists("makefile_test.a"):
>> + os.remove("makefile_test.a")
>> +
>> + if status != 0:
>> + return "Your version of make 3.82 is broken. Please revert to 3.81 or install a patched version.\n"
>> + return None
>> +
>> +
>> # Tar version 1.24 and onwards handle overwriting symlinks correctly
>> # but earlier versions do not; this needs to work properly for sstate
>> def check_tar_version(sanity_data, loosever):
>> @@ -407,6 +442,10 @@ def check_sanity(sanity_data):
>> messages = messages + 'Please set a MACHINE in your local.conf or environment\n'
>> machinevalid = False
>>
>> + makemsg = check_make_version(sanity_data, LooseVersion)
>> + if makemsg:
>> + messages = messages + makemsg
>> +
>> tarmsg = check_tar_version(sanity_data, LooseVersion)
>> if tarmsg:
>> messages = messages + tarmsg
>>
More information about the Openembedded-core
mailing list