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 +``` 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