Skip to content

Commit 4ddb72f

Browse files
committed
Restore current Zephyr compatibility
This patch restores the abilty to use this SDK without any changes to Zephyr. It moves the GNU toolchains back to the top level, places hosttools at the top level and moves the cmake/zephyr/gnu files back to cmake/zephyr. The goal is to change the sequencing of the restructuring to ease migration. Instead of requiring synchronous changes between SDK and Zephyr, we'll plan on updating Zephyr with the ability to handle either structure, then modify the SDK structure once Zephyr is ready. Signed-off-by: Keith Packard <[email protected]>
1 parent be94c5f commit 4ddb72f

File tree

5 files changed

+13
-31
lines changed

5 files changed

+13
-31
lines changed

.github/workflows/ci.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1463,8 +1463,7 @@ jobs:
14631463
fi
14641464
14651465
# Stage GNU toolchains
1466-
mkdir ${ARCHIVE_DIR}/gnu
1467-
pushd ${ARCHIVE_DIR}/gnu
1466+
pushd ${ARCHIVE_DIR}
14681467
14691468
## Generate target list from the workflow matrix
14701469
TARGETS=$(echo '${{ needs.setup.outputs.targets }}' | jq -cr '.[]')
@@ -1493,8 +1492,10 @@ jobs:
14931492
7z a -t7z -l ${ARCHIVE_NAME}_gnu.${EXT} ${ARCHIVE_DIR}
14941493
fi
14951494
1496-
# Unstage GNU toolchain directory
1497-
rm -rf ${ARCHIVE_DIR}/gnu
1495+
# Unstage GNU toolchain directories
1496+
for TARGET in ${TARGETS}; do
1497+
rm -rf ${ARCHIVE_DIR}/${TARGET}
1498+
done
14981499

14991500
# Stage LLVM toolchains
15001501
pushd ${ARCHIVE_DIR}

cmake/zephyr/gnu/generic.cmake renamed to cmake/zephyr/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 renamed to cmake/zephyr/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: 3 additions & 19 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
@@ -101,7 +101,7 @@ user_prompt()
101101
inst_gnu_toolchains=(${gnu_toolchains[*]})
102102
else
103103
for toolchain in ${gnu_toolchains[@]}; do
104-
if [ ! -d "gnu/${toolchain}" ]; then
104+
if [ ! -d "${toolchain}" ]; then
105105
ask_yn "Install '${toolchain}' GNU toolchain" && inst_gnu_toolchains+=("${toolchain}")
106106
fi
107107
done
@@ -118,8 +118,6 @@ user_prompt()
118118
# Environment Configurations
119119
ask_yn "Register Zephyr SDK CMake package" && do_cmake_pkg="y"
120120

121-
ask_yn "Create symbolic links for old Zephyr bisectability" && do_old_zephyr="y"
122-
123121
echo
124122
}
125123

@@ -235,8 +233,6 @@ fi
235233

236234
# Install GNU toolchains
237235
if [ "${do_gnu_toolchain}" = "y" ]; then
238-
mkdir -p gnu
239-
pushd gnu
240236

241237
for toolchain in ${inst_gnu_toolchains[@]}; do
242238
eval toolchain_filename="${dl_gnu_toolchain_filename}"
@@ -266,7 +262,6 @@ if [ "${do_gnu_toolchain}" = "y" ]; then
266262

267263
echo
268264
done
269-
popd
270265
fi
271266

272267
# Install LLVM toolchain
@@ -299,10 +294,8 @@ if [ "${do_hosttools}" = "y" ]; then
299294
echo "Installing host tools ..."
300295
case ${host} in
301296
linux-*)
302-
pushd hosttools
303-
./zephyr-sdk-${HOSTTYPE}-hosttools-standalone-0.10.sh -y -d . &> /dev/null
297+
./hosttools/zephyr-sdk-${HOSTTYPE}-hosttools-standalone-0.10.sh -y -d . &> /dev/null
304298
assert_rc "ERROR: Host tools installation failed" 30
305-
popd
306299
;;
307300
macos-*)
308301
echo "SKIPPED: macOS host tools are not available yet."
@@ -319,15 +312,6 @@ if [ "${do_cmake_pkg}" = "y" ]; then
319312
echo
320313
fi
321314

322-
# Create links for old Zephyr versions
323-
if [ "${do_old_zephyr}" = "y" ]; then
324-
echo "Creating links for old Zephyr bisectability ..."
325-
ln -sr gnu/* . || assert_rc "ERROR: Creating toolchain links" 50
326-
ln -sr cmake/zephyr/gnu/* cmake/zephyr || assert_rc "ERROR: Creating cmake links" 50
327-
ln -sr hosttools/* . || assert_rc "ERROR: Creating hosttools links" 50
328-
echo
329-
fi
330-
331315
echo "All done."
332316
if [ "${interactive}" = "y" ]; then
333317
read -n 1 -s -r -p "Press any key to exit ..."

scripts/template_setup_win

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ echo.
109109

110110
REM # Check installation type
111111
for %%t in (%GNU_TOOLCHAINS%) do (
112-
if not exist gnu\%%t\ (
112+
if not exist %%t\ (
113113
set IS_PARTIAL_GNU_SDK=y
114114
)
115115
)
@@ -124,7 +124,7 @@ if [!ERRORLEVEL!] equ [1] (
124124
set INST_GNU_TOOLCHAINS=%GNU_TOOLCHAINS%
125125
) else (
126126
for %%t in (%GNU_TOOLCHAINS%) do (
127-
if not exist gnu\%%t\ (
127+
if not exist %%t\ (
128128
choice /c:yn /m:"Install '%%t' GNU toolchain"
129129
if [!ERRORLEVEL!] equ [1] (
130130
set INST_GNU_TOOLCHAINS=!INST_GNU_TOOLCHAINS! %%t
@@ -149,8 +149,6 @@ echo.
149149
:process
150150
REM # Install GNU toolchains
151151
if [%DO_GNU_TOOLCHAIN%] neq [] (
152-
if not exist gnu\ mkdir gnu
153-
pushd gnu
154152

155153
for %%t in (%INST_GNU_TOOLCHAINS%) do (
156154
set TOOLCHAIN_FILENAME=%DL_GNU_TOOLCHAIN_FILENAME%
@@ -180,7 +178,6 @@ if [%DO_GNU_TOOLCHAIN%] neq [] (
180178
del /q !TOOLCHAIN_FILENAME!
181179
)
182180
)
183-
popd
184181
)
185182

186183
REM # Install LLVM toolchain

0 commit comments

Comments
 (0)