]> code.ossystems Code Review - openembedded-core.git/commitdiff
insane.bbclass: print full path on invalid LICENSE_FILES_CHKSUM
authorDarren Hart <dvhart@linux.intel.com>
Tue, 18 Oct 2011 19:00:56 +0000 (12:00 -0700)
committerSaul Wold <sgw@linux.intel.com>
Thu, 20 Oct 2011 02:58:45 +0000 (19:58 -0700)
Currently only the basename is printed when os.path.isfile() returns a failure
for the license file. If the file is present, but in the wrong directory, this
can be non-obvious to debug. Use the full path instead.

Make a minor grammatical correction in the error message while we're at it.

Signed-off-by: Darren Hart <dvhart@linux.intel.com>
meta/classes/insane.bbclass

index b861e85321892af242f2bc3ee2aa90f38a2df3a2..017f7bedc68cacfa6ead83092d75abb3d684bd96 100644 (file)
@@ -330,7 +330,7 @@ def package_qa_check_license(workdir, d):
         (type, host, path, user, pswd, parm) = bb.decodeurl(url)
         srclicfile = os.path.join(srcdir, path)
         if not os.path.isfile(srclicfile):
-            raise bb.build.FuncFailed( pn + ": LIC_FILES_CHKSUM points to invalid file: " + path)
+            raise bb.build.FuncFailed( pn + ": LIC_FILES_CHKSUM points to an invalid file: " + srclicfile)
 
         if 'md5' not in parm:
             bb.error(pn + ": md5 checksum is not specified for ", url)