]> code.ossystems Code Review - openembedded-core.git/commitdiff
buildhistory: avoid quoting issues with the layer list for build-id
authorChristopher Larson <kergoth@gmail.com>
Fri, 23 Dec 2011 16:23:57 +0000 (09:23 -0700)
committerSaul Wold <sgw@linux.intel.com>
Tue, 3 Jan 2012 04:26:22 +0000 (20:26 -0800)
--4ef4ab1d_66334873_12d0c
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

The layer list is multiline, and includes quotes, so including it within
quotes is problematic, particularly if the revision includes characters which
are not valid outside of a quoted string in shell. To reproduce this failure:
do a build with an scm layer not on a branch.

Signed-off-by: Christopher Larson <chris_larson@mentor.com (mailto:chris_larson@mentor.com)>
meta/classes/buildhistory.bbclass

index 39f5ff6eac70f5279243b91c6b0db1cfdd04842f..6a08db467eabd925b5612cccc1f5bd4cdb9c5b94 100644 (file)
@@ -325,7 +325,10 @@ buildhistory_get_imageinfo() {
 
        # Add some configuration information
        echo "${MACHINE}: ${IMAGE_BASENAME} configured for ${DISTRO} ${DISTRO_VERSION}" > ${BUILDHISTORY_DIR_IMAGE}/build-id
-       echo "${@buildhistory_get_layers(d)}" >> ${BUILDHISTORY_DIR_IMAGE}/build-id
+
+       cat >> ${BUILDHISTORY_DIR_IMAGE}/build-id <<END
+${@buildhistory_get_layers(d)}
+END
 }
 
 # By prepending we get in before the removal of packaging files