diff options
author | Joel Galenson <jgalenson@google.com> | 2021-09-30 22:47:36 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-09-30 22:47:36 +0000 |
commit | 819ca2ffc0b9e7e8da08b09f46351df70ec83981 (patch) | |
tree | cdbd4239b9eac16f2f342915200eb27428d03a80 | |
parent | 416484d1cc52bc04ad36cec44e6271c51de49603 (diff) | |
parent | 5f7bfba63cc3e17ac851c356b72511e9f525ce18 (diff) | |
download | external_updater-819ca2ffc0b9e7e8da08b09f46351df70ec83981.tar.gz |
Print a nice error if we cannot find any paths. am: 3564658445 am: eb4fd4e7b9 am: 3e4cd7c4c6 am: 5f7bfba63c
Original change: https://android-review.googlesource.com/c/platform/tools/external_updater/+/1842639
Change-Id: Ie3bcbc792b88e9e87da7726c74de8dcd7846b973
-rw-r--r-- | external_updater.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/external_updater.py b/external_updater.py index 7518380..c15e92d 100644 --- a/external_updater.py +++ b/external_updater.py @@ -202,8 +202,11 @@ def get_paths(paths: List[str]) -> List[str]: # We want to use glob to get all the paths, so we first convert to absolute. abs_paths = [fileutils.get_absolute_project_path(Path(path)) for path in paths] - return [path for abs_path in abs_paths - for path in sorted(glob.glob(str(abs_path)))] + result = [path for abs_path in abs_paths + for path in sorted(glob.glob(str(abs_path)))] + if paths and not result: + print('Could not find any valid paths in %s' % str(paths)) + return result def write_json(json_file: str, results: Dict[str, Dict[str, str]]) -> List[str]: |