From 864fe8fc4c199c94029ffce06086244887b15780 Mon Sep 17 00:00:00 2001 From: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com> Date: Mon, 8 Apr 2024 08:56:18 -0300 Subject: [PATCH 1/2] Fix UI Readme --- tools/config_editor/README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/config_editor/README.md b/tools/config_editor/README.md index c0ba0bc40..10cf8f8a0 100644 --- a/tools/config_editor/README.md +++ b/tools/config_editor/README.md @@ -26,6 +26,7 @@ If you are using MacOS and the application looks weird, check [this guide from T These command line arguments can be used to pre-configure the application: +``` Command line arguments: -t, --target Comma-separated list of targets to be compiled. Choose from: all, esp32, esp32s2, esp32s3, esp32c2, esp32c3, esp32c6, esp32h2. Default: all except esp32c2 @@ -36,3 +37,4 @@ Command line arguments: -i, --idf-commit Commit of the ESP-IDF repository to be used. Default: set by the build script -D, --debug-level Debug level to be set to ESP-IDF. Choose from: default, none, error, warning, info, debug, verbose. Default: default +``` From 9a0847bbedad8f0676c6b7bcabf24d5f7504215f Mon Sep 17 00:00:00 2001 From: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com> Date: Mon, 8 Apr 2024 09:22:22 -0300 Subject: [PATCH 2/2] Force targets to be lower case --- tools/config_editor/app.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/tools/config_editor/app.py b/tools/config_editor/app.py index 8a7b2ce24..9ab882c41 100755 --- a/tools/config_editor/app.py +++ b/tools/config_editor/app.py @@ -231,10 +231,13 @@ def main() -> None: args = parser.parse_args() - # Set the options in the app - if args.target.strip() == "default": + # Force targets to be lower case + args.target = args.target.lower() + + # Check if the target is valid + if args.target == "default": args.target = ",".join([x[0] for x in target_choices if x[1]]) - elif args.target.strip() == "all": + elif args.target == "all": args.target = ",".join([x[0] for x in target_choices]) app.supported_targets = [x[0] for x in target_choices] @@ -246,6 +249,7 @@ def main() -> None: app.setting_target = args.target + # Check if the Arduino path is valid if args.copy: if check_arduino_path(args.arduino_path): app.setting_enable_copy = True @@ -258,6 +262,7 @@ def main() -> None: else: app.setting_enable_copy = False + # Set the other options app.setting_arduino_path = os.path.abspath(args.arduino_path) app.setting_arduino_branch = args.arduino_branch app.setting_idf_branch = args.idf_branch