diff options
| author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2012-05-04 14:36:01 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-05-04 15:48:25 +0100 |
| commit | 2086bb86885951d0a74342e30ff205c24cb93f0d (patch) | |
| tree | 1f999604d78ee17117f1efcd66338d83780b799f | |
| parent | 0be64cda0a20ca3aab75ee0cc360236d2dc40c95 (diff) | |
| download | openembedded-core-2086bb86885951d0a74342e30ff205c24cb93f0d.tar.gz openembedded-core-2086bb86885951d0a74342e30ff205c24cb93f0d.tar.bz2 openembedded-core-2086bb86885951d0a74342e30ff205c24cb93f0d.zip | |
buildhistory: fix multiple commit of images and packages at the same time
The echo line here was merging multiple lines into one, and the result
was that if both image and package changes had to be comitted then only
the image changes were being committed and the package changes could
potentially be merged into the next package change. Quoting the variable
reference fixes this.
Fixes [YOCTO #2411]
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/classes/buildhistory.bbclass | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/classes/buildhistory.bbclass b/meta/classes/buildhistory.bbclass index 8d5b0963f8..d2d19ff9cd 100644 --- a/meta/classes/buildhistory.bbclass +++ b/meta/classes/buildhistory.bbclass @@ -402,7 +402,7 @@ buildhistory_commit() { git add ${BUILDHISTORY_DIR}/* HOSTNAME=`hostname 2>/dev/null || echo unknown` # porcelain output looks like "?? packages/foo/bar" - for entry in `echo $repostatus | awk '{print $2}' | awk -F/ '{print $1}' | sort | uniq` ; do + for entry in `echo "$repostatus" | awk '{print $2}' | awk -F/ '{print $1}' | sort | uniq` ; do git commit ${BUILDHISTORY_DIR}/$entry -m "$entry: Build ${BUILDNAME} of ${DISTRO} ${DISTRO_VERSION} for machine ${MACHINE} on $HOSTNAME" --author "${BUILDHISTORY_COMMIT_AUTHOR}" > /dev/null done if [ "${BUILDHISTORY_PUSH_REPO}" != "" ] ; then |