Skip to content

Commit d635eaf

Browse files
committed
Remove gnu subdirectory
Get rid of the extra subdirectory for the gnu toolchains and let them sit at the same level as the llvm toolchain. This avoids making the SDK incompatible with previous versions. Signed-off-by: Keith Packard <keithp@keithp.com>
1 parent 33ce134 commit d635eaf

File tree

5 files changed

+13
-20
lines changed

5 files changed

+13
-20
lines changed

.github/workflows/ci.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1292,7 +1292,7 @@ jobs:
12921292
fi
12931293
12941294
# Create bundle directory
1295-
mkdir ${ARCHIVE_DIR}
1295+
mkdir -p ${ARCHIVE_DIR}
12961296
pushd ${ARCHIVE_DIR}
12971297
12981298
# Stage version file
@@ -1351,8 +1351,7 @@ jobs:
13511351
fi
13521352
13531353
# Stage GNU toolchains
1354-
mkdir ${ARCHIVE_DIR}/gnu
1355-
pushd ${ARCHIVE_DIR}/gnu
1354+
pushd ${ARCHIVE_DIR}
13561355
13571356
## Generate target list from the workflow matrix
13581357
TARGETS=$(echo '${{ needs.setup.outputs.targets }}' | jq -cr '.[]')
@@ -1381,8 +1380,10 @@ jobs:
13811380
7z a -t7z -l ${ARCHIVE_NAME}_gnu.${EXT} ${ARCHIVE_DIR}
13821381
fi
13831382
1384-
# Unstage GNU toolchain directory
1385-
rm -rf ${ARCHIVE_DIR}/gnu
1383+
# Unstage GNU toolchain directories
1384+
for TARGET in ${TARGETS}; do
1385+
rm -rf ${ARCHIVE_DIR}/${TARGET}
1386+
done
13861387

13871388
# Stage LLVM toolchains
13881389
pushd ${ARCHIVE_DIR}

cmake/zephyr/gnu/generic.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# SPDX-License-Identifier: Apache-2.0
22

3-
set(TOOLCHAIN_HOME ${ZEPHYR_SDK_INSTALL_DIR}/gnu)
3+
set(TOOLCHAIN_HOME ${ZEPHYR_SDK_INSTALL_DIR})
44

55
set(COMPILER gcc)
66
set(LINKER ld)

cmake/zephyr/gnu/target.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ set(CROSS_COMPILE_TARGET_sparc sparc-zephyr-elf)
2626
set(CROSS_COMPILE_TARGET ${CROSS_COMPILE_TARGET_${ARCH}})
2727
set(SYSROOT_TARGET ${CROSS_COMPILE_TARGET})
2828

29-
set(SYSROOT_DIR ${ZEPHYR_SDK_INSTALL_DIR}/gnu/${SYSROOT_TARGET}/${SYSROOT_TARGET})
30-
set(CROSS_COMPILE ${ZEPHYR_SDK_INSTALL_DIR}/gnu/${CROSS_COMPILE_TARGET}/bin/${CROSS_COMPILE_TARGET}-)
29+
set(SYSROOT_DIR ${ZEPHYR_SDK_INSTALL_DIR}/${SYSROOT_TARGET}/${SYSROOT_TARGET})
30+
set(CROSS_COMPILE ${ZEPHYR_SDK_INSTALL_DIR}/${CROSS_COMPILE_TARGET}/bin/${CROSS_COMPILE_TARGET}-)
3131

3232
if("${ARCH}" STREQUAL "x86")
3333
if(CONFIG_X86_64)

scripts/template_setup_posix

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ check_command()
5252
check_full_gnu_sdk()
5353
{
5454
for toolchain in ${gnu_toolchains[@]}; do
55-
if [ ! -d "gnu/${toolchain}" ]; then
55+
if [ ! -d "${toolchain}" ]; then
5656
return 1
5757
fi
5858
done
@@ -100,7 +100,7 @@ user_prompt()
100100
inst_gnu_toolchains=(${gnu_toolchains[*]})
101101
else
102102
for toolchain in ${gnu_toolchains[@]}; do
103-
if [ ! -d "gnu/${toolchain}" ]; then
103+
if [ ! -d "${toolchain}" ]; then
104104
ask_yn "Install '${toolchain}' GNU toolchain" && inst_gnu_toolchains+=("${toolchain}")
105105
fi
106106
done
@@ -217,9 +217,6 @@ fi
217217

218218
# Install GNU toolchains
219219
if [ "${do_gnu_toolchain}" = "y" ]; then
220-
mkdir -p gnu
221-
pushd gnu
222-
223220
for toolchain in ${inst_gnu_toolchains[@]}; do
224221
eval toolchain_filename="${dl_gnu_toolchain_filename}"
225222
toolchain_uri="${dl_rel_base}/${toolchain_filename}"
@@ -248,7 +245,6 @@ if [ "${do_gnu_toolchain}" = "y" ]; then
248245

249246
echo
250247
done
251-
popd
252248
fi
253249

254250
# Install LLVM toolchain

scripts/template_setup_win

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ echo.
107107

108108
REM # Check installation type
109109
for %%t in (%GNU_TOOLCHAINS%) do (
110-
if not exist gnu\%%t\ (
110+
if not exist %%t\ (
111111
set IS_PARTIAL_GNU_SDK=y
112112
)
113113
)
@@ -122,7 +122,7 @@ if [!ERRORLEVEL!] equ [1] (
122122
set INST_GNU_TOOLCHAINS=%GNU_TOOLCHAINS%
123123
) else (
124124
for %%t in (%GNU_TOOLCHAINS%) do (
125-
if not exist gnu\%%t\ (
125+
if not exist %%t\ (
126126
choice /c:yn /m:"Install '%%t' GNU toolchain"
127127
if [!ERRORLEVEL!] equ [1] (
128128
set INST_GNU_TOOLCHAINS=!INST_GNU_TOOLCHAINS! %%t
@@ -147,9 +147,6 @@ echo.
147147
:process
148148
REM # Install GNU toolchains
149149
if [%DO_GNU_TOOLCHAIN%] neq [] (
150-
if not exist gnu\ mkdir gnu
151-
pushd gnu
152-
153150
for %%t in (%INST_GNU_TOOLCHAINS%) do (
154151
set TOOLCHAIN_FILENAME=%DL_GNU_TOOLCHAIN_FILENAME%
155152
set TOOLCHAIN_URI=%DL_REL_BASE%/!TOOLCHAIN_FILENAME!
@@ -178,7 +175,6 @@ if [%DO_GNU_TOOLCHAIN%] neq [] (
178175
del /q !TOOLCHAIN_FILENAME!
179176
)
180177
)
181-
popd
182178
)
183179

184180
REM # Install LLVM toolchain

0 commit comments

Comments
 (0)