espressif/esp-idf
1
0
Fork 1
mirror of https://github.com/espressif/esp-idf.git synced 2025-08-02 12:14:32 +02:00
Code Issues Packages Projects Releases Wiki Activity

find_apps.py: app with no supported targets will be skipped correctly

Browse Source 
- fix deprecated logging.warn()
This commit is contained in:
Fu Hanxi
2020-12-08 11:54:39 +08:00
parent 636f136296
commit af2220f5f8
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
tools/find_apps.py
View File

@@ -146,9 +146,9 @@ def find_apps(build_system_class, path, recursive, exclude_list, target):
logging.debug("Looking for {} apps in {}{}".format(build_system_name, path, " recursively" if recursive else "")) logging.debug("Looking for {} apps in {}{}".format(build_system_name, path, " recursively" if recursive else ""))
if not recursive: if not recursive:
if exclude_list: if exclude_list:
logging.warn("--exclude option is ignored when used without --recursive") logging.warning("--exclude option is ignored when used without --recursive")
if not build_system_class.is_app(path): if not build_system_class.is_app(path):
logging.warn("Path {} specified without --recursive flag, but no {} app found there".format( logging.warning("Path {} specified without --recursive flag, but no {} app found there".format(
path, build_system_name)) path, build_system_name))
return [] return []
return [path] return [path]
@@ -168,11 +168,14 @@ def find_apps(build_system_class, path, recursive, exclude_list, target):
del dirs[:] del dirs[:]
supported_targets = build_system_class.supported_targets(root) supported_targets = build_system_class.supported_targets(root)
if supported_targets and target not in supported_targets: if supported_targets and (target in supported_targets):
logging.debug("Skipping, app only supports targets: " + ", ".join(supported_targets))
continue
apps_found.append(root) apps_found.append(root)
else:
if supported_targets:
logging.debug("Skipping, app only supports targets: " + ", ".join(supported_targets))
else:
logging.debug("Skipping, app has no supported targets")
continue
return apps_found return apps_found