Skip to content

Commit 8236f38

Browse files
marc-hbkv2019i
authored andcommitted
xtensa-build-all.sh: rename local variable COMPILER to build_dir_suffix
"COMPILER" is a very common name with a high risk of collision. Don't use it for a minor, local variable. This also reduces confusion. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
1 parent b8a4ddf commit 8236f38

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

scripts/xtensa-build-all.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -234,15 +234,15 @@ do
234234
export XTENSA_SYSTEM=$XTENSA_BUILDS_DIR/$XTENSA_CORE/config
235235
printf 'XTENSA_SYSTEM=%s\n' "${XTENSA_SYSTEM}"
236236
PATH=$XTENSA_TOOLS_DIR/XtensaTools/bin:$OLDPATH
237-
COMPILER="xcc"
237+
build_dir_suffix='xcc'
238238
else
239239
TOOLCHAIN=$HOST
240240
PATH=$SOF_TOP/../$HOST/bin:$OLDPATH
241-
COMPILER="gcc"
241+
build_dir_suffix='gcc'
242242
DEFCONFIG_PATCH=""
243243
fi
244244

245-
BUILD_DIR=build_${platform}_${COMPILER}
245+
BUILD_DIR=build_${platform}_${build_dir_suffix}
246246
printf "Build in %s\n" "$BUILD_DIR"
247247

248248
# only delete binary related to this build

0 commit comments

Comments
 (0)