Hi Tuomas,
On Wed, 14 Feb 2018 09:50:17 +0100 Lukasz Majewski lukma@denx.de wrote:
On Wed, 14 Feb 2018 03:25:58 +0200 Tuomas Tynkkynen tuomas@tuxera.com wrote:
Hi,
On Tue, 13 Feb 2018 23:01:13 +0100 Lukasz Majewski lukma@denx.de wrote:
This commit fixes several issues:
- After moving env related code to ./env directory the
env_common.o file is no longer present in the system (has been replaced with built-in.o).
- Use ${OBJCOPY} if available, fallback to system default's
objcopy if not present.
- Extend the script to accept different build directory than
current one. It is extremely handy with OE usage, where source code is separated from build.
Signed-off-by: Lukasz Majewski lukma@denx.de
scripts/get_default_envs.sh | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/scripts/get_default_envs.sh b/scripts/get_default_envs.sh index 7955db60e5..bbb6d0a6ef 100755 --- a/scripts/get_default_envs.sh +++ b/scripts/get_default_envs.sh @@ -6,16 +6,24 @@ #
# This file extracts default envs from built u-boot -# usage: get_default_envs.sh > u-boot-env-default.txt +# usage: get_default_envs.sh [build dir] > u-boot-env-default.txt set -ue
-ENV_OBJ_FILE="env_common.o" +: "${OBJCOPY:=objcopy}"
Maybe "${CROSS_COMPILE}objcopy" would be a better default as that's what the build system uses.
Unfortunately not with OE builds. In OE, the OBJCOPY is set to arm specific objcopy.
With fallback (objcopy) from your local machine - it depends if it supports multi arch or not.
No, I meant that instead of having a fallback of "objcopy", have a fallback of "${CROSS_COMPILE}objcopy". I.e. squashing this change:
diff --git a/scripts/get_default_envs.sh b/scripts/get_default_envs.sh index 3e532d12c4..184cc19ab7 100755 --- a/scripts/get_default_envs.sh +++ b/scripts/get_default_envs.sh @@ -9,7 +9,7 @@ # usage: get_default_envs.sh [build dir] > u-boot-env-default.txt set -ue
-: "${OBJCOPY:=objcopy}" +: "${OBJCOPY:=${CROSS_COMPILE:-}objcopy}"
Yes, you are obviously right. I've misunderstood you, sorry.
I will prepare v3 immediately.
Thanks.
ENV_OBJ_FILE="built-in.o" ENV_OBJ_FILE_COPY="copy_${ENV_OBJ_FILE}"
That way the script works out-of-the-box for people who do the usual
export CROSS_COMPILE=aarch64-linux-gnu- (or whatever)
when building U-Boot.
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de