[oe-commits] Bruce Ashfield : kernel-yocto: use cat-file instead of git-show
git at git.openembedded.org
git at git.openembedded.org
Sat Aug 30 09:17:54 UTC 2014
Module: openembedded-core.git
Branch: master-next
Commit: 1bc296cec9b5279f3dae845d7411f58943f73e9b
URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=1bc296cec9b5279f3dae845d7411f58943f73e9b
Author: Bruce Ashfield <bruce.ashfield at windriver.com>
Date: Mon Mar 17 14:00:15 2014 -0400
kernel-yocto: use cat-file instead of git-show
Parsing the output of git show is error prone, since it changes based on
the type of issue with bad comit IDs. Since the output is no longer used
in the case of a valid ref, we can switch to git-cat-file and simply
check the return code.
Signed-off-by: Darren Hart <dvhart at linux.intel.com>
Signed-off-by: Bruce Ashfield <bruce.ashfield at windriver.com>
---
meta/classes/kernel-yocto.bbclass | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/classes/kernel-yocto.bbclass b/meta/classes/kernel-yocto.bbclass
index 6c92427..38c886b 100644
--- a/meta/classes/kernel-yocto.bbclass
+++ b/meta/classes/kernel-yocto.bbclass
@@ -321,8 +321,8 @@ do_validate_branches() {
fi
fi
- ref=`git show ${machine_srcrev} 2>&1 | head -n1 || true`
- if [ "$ref" = "fatal: bad object ${target_meta_head}" ]; then
+ git cat-file -t ${machine_srcrev} > /dev/null
+ if [ if $? -ne 0 ]; then
echo "ERROR ${machine_srcrev} is not a valid commit ID."
echo "The kernel source tree may be out of sync"
exit 1
@@ -358,8 +358,8 @@ do_validate_branches() {
git show-ref --quiet --verify -- "refs/heads/${KMETA}"
if [ $? -eq 0 ] && [ "${target_meta_head}" != "AUTOINC" ]; then
if [ "$meta_head" != "$target_meta_head" ]; then
- ref=`git show ${target_meta_head} 2>&1 | head -n1 || true`
- if [ "$ref" = "fatal: bad object ${target_meta_head}" ]; then
+ git cat-file -t ${target_meta_head} > /dev/null
+ if [ $? -ne 0 ]; then
echo "ERROR ${target_meta_head} is not a valid commit ID"
echo "The kernel source tree may be out of sync"
exit 1
More information about the Openembedded-commits
mailing list