From df4c691694e90455864e890936727b40787a8d49 Mon Sep 17 00:00:00 2001 From: Paul Eggleton Date: Thu, 24 Aug 2017 16:54:23 +1200 Subject: classes/license: drop erroneous sha256 parameter in LIC_FILES_CHKSUM In OE-Core commit a48fea275b08ff3d3dfc9a928aeb04768db35873, a check on the value of a "sha256" parameter was added, however there was no mention of this in the commit message and no corresponding code to actually verify the checksum as sha256 was added along with it either, so there's no point in getting the value. Additionally it was assuming that a sha256 value would be present without checking first, with the result that if you leave out the md5 value in a recipe intentionally in order to get it to tell you the correct value on the next build, you got a traceback instead of the appropriate error containing the information. Drop this entirely - if we want to implement this we need to do it properly. Signed-off-by: Paul Eggleton --- meta/classes/license.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass index 4f7842c3db..d353110464 100644 --- a/meta/classes/license.bbclass +++ b/meta/classes/license.bbclass @@ -486,7 +486,7 @@ def find_license_files(d): except bb.fetch.MalformedUrl: bb.fatal("%s: LIC_FILES_CHKSUM contains an invalid URL: %s" % (d.getVar('PF'), url)) # We want the license filename and path - chksum = parm['md5'] if 'md5' in parm else parm['sha256'] + chksum = parm.get('md5', None) beginline = parm.get('beginline') endline = parm.get('endline') lic_chksums[path] = (chksum, beginline, endline) -- cgit 1.2.3-korg