Skip to content

Commit c8db959

Browse files
committed
xtensa-build-zephyr: pure comment/link to west config simplification
Also fix the name of the west config class, see https://docs.zephyrproject.org/4.2.0/develop/west/west-apis.html #configuration-api Fixes 5a82b0a ("xtensa-build-zephyr.py: add new rimage_west_configuration()") Signed-off-by: Marc Herbert <marc.herbert@intel.com>
1 parent 8d44f26 commit c8db959

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

scripts/xtensa-build-zephyr.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -688,9 +688,13 @@ def clean_staging(platform):
688688
def rimage_west_configuration(platform_dict, dest_dir):
689689
"""Configure rimage in a new file `dest_dir/westconfig.ini`, starting
690690
from the workspace .west/config.
691-
Returns a tuple (west ConfigFile, pathlib.Path to that new file).
691+
Returns a tuple: (west.configuration.Configuration, pathlib.Path to that new file).
692692
"""
693693

694+
# This complex and painful saving and copying can be dropped and
695+
# greatly simplified once any of these alternatives gets finally
696+
# implemented in `west config`:
697+
# https://github.com/zephyrproject-rtos/west/issues/429 or 849, 867,..
694698
saved_local_var = os.environ.get('WEST_CONFIG_LOCAL')
695699
workspace_west_config_path = os.environ.get('WEST_CONFIG_LOCAL',
696700
str(west_top / ".west" / "config"))

0 commit comments

Comments
 (0)