* [PATCH 0/1] oe-init-build-env: Error out when failed to locate cwd
@ 2019-03-11 10:16 Robert Yang
2019-03-11 10:16 ` [PATCH 1/1] " Robert Yang
0 siblings, 1 reply; 2+ messages in thread
From: Robert Yang @ 2019-03-11 10:16 UTC (permalink / raw)
To: openembedded-core
The following changes since commit 28e631d6dbc0a126253c0a072b8f39ff683bfa3a:
python: time.tzset missing (2019-03-09 14:41:20 +0000)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib rbt/init
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/init
Robert Yang (1):
oe-init-build-env: Error out when failed to locate cwd
oe-init-build-env | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
--
2.7.4
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 1/1] oe-init-build-env: Error out when failed to locate cwd
2019-03-11 10:16 [PATCH 0/1] oe-init-build-env: Error out when failed to locate cwd Robert Yang
@ 2019-03-11 10:16 ` Robert Yang
0 siblings, 0 replies; 2+ messages in thread
From: Robert Yang @ 2019-03-11 10:16 UTC (permalink / raw)
To: openembedded-core
Ubuntu's /bin/sh symlinks to /bin/dash by default, so
subprocess.check_call(oe-init-build-env, cwd=builddir) would be failed since
pwd is builddir, and there is no $builddir/oe-init-build-env, this would
lead to other confusing errors, check and error it out earlier to make it
easier to locate the problem.
We don't meet the problem when manually run ". oe-init-build-env" is because
Ubuntu's default login shell is bash, but subprocess.check_call() doesn't
respect to login shell, so the error only happens in situations like
subprocess.check_call().
And also print errors to stderr as oe-buildenv-internal does.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
oe-init-build-env | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/oe-init-build-env b/oe-init-build-env
index e813230..861c3e0 100755
--- a/oe-init-build-env
+++ b/oe-init-build-env
@@ -31,13 +31,18 @@ elif [ -n "$ZSH_NAME" ]; then
THIS_SCRIPT=$0
else
THIS_SCRIPT="$(pwd)/oe-init-build-env"
+ if [ ! -e "$THIS_SCRIPT" ]; then
+ echo "Error: $THIS_SCRIPT doesn't exist!" >&2
+ echo "Please run this script in oe-init-build-env's directory." >&2
+ exit 1
+ fi
fi
if [ -n "$BBSERVER" ]; then
unset BBSERVER
fi
if [ -z "$ZSH_NAME" ] && [ "$0" = "$THIS_SCRIPT" ]; then
- echo "Error: This script needs to be sourced. Please run as '. $THIS_SCRIPT'"
+ echo "Error: This script needs to be sourced. Please run as '. $THIS_SCRIPT'" >&2
exit 1
fi
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-11 9:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-11 10:16 [PATCH 0/1] oe-init-build-env: Error out when failed to locate cwd Robert Yang
2019-03-11 10:16 ` [PATCH 1/1] " Robert Yang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox