Use die() or bbfatal_log() where the log should definitely be printed
Change calls to bbfatal() to either die() or bbfatal_log() where we know we want the full log to be printed by the UI (calling bberror or bbfatal would otherwise suppress it since the change to connect these functions through to the UI.) bbfatal() is still fine to use where there is enough context information in the message such that the log isn't needed. (From OE-Core rev: 04ed9a19e1b08003329138b8ab83691d13c11fd9) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
757be608a5
commit
ddc1df3e80
|
@ -87,7 +87,7 @@ oe_runconf () {
|
|||
if [ "$?" != "0" ]; then
|
||||
echo "Configure failed. The contents of all config.log files follows to aid debugging"
|
||||
find ${S} -ignore_readdir_race -name config.log -print -exec cat {} \;
|
||||
bbfatal "oe_runconf failed"
|
||||
die "oe_runconf failed"
|
||||
fi
|
||||
set -e
|
||||
else
|
||||
|
@ -287,7 +287,7 @@ autotools_do_configure() {
|
|||
intltoolize --copy --force --automake
|
||||
fi
|
||||
bbnote Executing ACLOCAL=\"$ACLOCAL\" autoreconf --verbose --install --force ${EXTRA_AUTORECONF} $acpaths
|
||||
ACLOCAL="$ACLOCAL" autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || bbfatal "autoreconf execution failed."
|
||||
ACLOCAL="$ACLOCAL" autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || die "autoreconf execution failed."
|
||||
cd $olddir
|
||||
fi
|
||||
if [ -e ${S}/configure ]; then
|
||||
|
|
|
@ -120,7 +120,7 @@ do_kernel_metadata() {
|
|||
|
||||
createme -v -v ${createme_flags} ${ARCH} ${machine_branch}
|
||||
if [ $? -ne 0 ]; then
|
||||
bbfatal "Could not create ${machine_branch}"
|
||||
bbfatal_log "Could not create ${machine_branch}"
|
||||
fi
|
||||
|
||||
sccs="$sccs ${@" ".join(find_sccs(d))}"
|
||||
|
@ -152,7 +152,7 @@ do_kernel_metadata() {
|
|||
updateme ${updateme_flags} -DKDESC=${KMACHINE}:${LINUX_KERNEL_TYPE} \
|
||||
${includes} ${addon_features} ${ARCH} ${KMACHINE} ${sccs} ${patches}
|
||||
if [ $? -ne 0 ]; then
|
||||
bbfatal "Could not update ${machine_branch}"
|
||||
bbfatal_log "Could not update ${machine_branch}"
|
||||
fi
|
||||
}
|
||||
|
||||
|
@ -163,7 +163,7 @@ do_patch() {
|
|||
patchme ${KMACHINE}
|
||||
if [ $? -ne 0 ]; then
|
||||
bberror "Could not apply patches for ${KMACHINE}."
|
||||
bbfatal "Patch failures can be resolved in the linux source directory ${S})"
|
||||
bbfatal_log "Patch failures can be resolved in the linux source directory ${S})"
|
||||
fi
|
||||
|
||||
# check to see if the specified SRCREV is reachable from the final branch.
|
||||
|
@ -253,7 +253,7 @@ do_kernel_checkout() {
|
|||
if [ $? -eq 1 ]; then
|
||||
bberror "The branch '${KMETA}' is required and was not found"
|
||||
bberror "Ensure that the SRC_URI points to a valid linux-yocto"
|
||||
bbfatal "kernel repository"
|
||||
bbfatal_log "kernel repository"
|
||||
fi
|
||||
fi
|
||||
|
||||
|
@ -293,7 +293,7 @@ do_kernel_configme() {
|
|||
PATH=${PATH}:${S}/scripts/util
|
||||
configme ${configmeflags} --reconfig --output ${B} ${LINUX_KERNEL_TYPE} ${KMACHINE}
|
||||
if [ $? -ne 0 ]; then
|
||||
bbfatal "Could not configure ${KMACHINE}-${LINUX_KERNEL_TYPE}"
|
||||
bbfatal_log "Could not configure ${KMACHINE}-${LINUX_KERNEL_TYPE}"
|
||||
fi
|
||||
|
||||
echo "# Global settings from linux recipe" >> ${B}/.config
|
||||
|
@ -372,7 +372,7 @@ do_validate_branches() {
|
|||
git cat-file -t ${machine_srcrev} > /dev/null
|
||||
if [ $? -ne 0 ]; then
|
||||
bberror "${machine_srcrev} is not a valid commit ID."
|
||||
bbfatal "The kernel source tree may be out of sync"
|
||||
bbfatal_log "The kernel source tree may be out of sync"
|
||||
fi
|
||||
force_srcrev=${machine_srcrev}
|
||||
fi
|
||||
|
@ -387,14 +387,14 @@ do_validate_branches() {
|
|||
git cat-file -t ${target_meta_head} > /dev/null
|
||||
if [ $? -ne 0 ]; then
|
||||
bberror "${target_meta_head} is not a valid commit ID"
|
||||
bbfatal "The kernel source tree may be out of sync"
|
||||
bbfatal_log "The kernel source tree may be out of sync"
|
||||
fi
|
||||
if [ "$meta_head" != "$target_meta_head" ]; then
|
||||
bbnote "Setting branch ${KMETA} to ${target_meta_head}"
|
||||
git branch -m ${KMETA} ${KMETA}-orig
|
||||
git checkout -q -b ${KMETA} ${target_meta_head}
|
||||
if [ $? -ne 0 ];then
|
||||
bbfatal "Could not checkout ${KMETA} branch from known hash ${target_meta_head}"
|
||||
bbfatal_log "Could not checkout ${KMETA} branch from known hash ${target_meta_head}"
|
||||
fi
|
||||
fi
|
||||
fi
|
||||
|
|
|
@ -4,12 +4,12 @@ EXTRA_OESCONS ?= ""
|
|||
|
||||
scons_do_compile() {
|
||||
${STAGING_BINDIR_NATIVE}/scons ${PARALLEL_MAKE} PREFIX=${prefix} prefix=${prefix} ${EXTRA_OESCONS} || \
|
||||
bbfatal "scons build execution failed."
|
||||
die "scons build execution failed."
|
||||
}
|
||||
|
||||
scons_do_install() {
|
||||
${STAGING_BINDIR_NATIVE}/scons PREFIX=${D}${prefix} prefix=${D}${prefix} install ${EXTRA_OESCONS}|| \
|
||||
bbfatal "scons install execution failed."
|
||||
die "scons install execution failed."
|
||||
}
|
||||
|
||||
EXPORT_FUNCTIONS do_compile do_install
|
||||
|
|
|
@ -41,7 +41,7 @@ do_configure_append() {
|
|||
# generate gnulib configure script
|
||||
olddir=`pwd`
|
||||
cd ${S}/src/libs/gnulib/
|
||||
ACLOCAL="$ACLOCAL" autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || bbfatal "autoreconf execution failed."
|
||||
ACLOCAL="$ACLOCAL" autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || die "autoreconf execution failed."
|
||||
cd ${olddir}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue