diff options
| author | Bob Haarman <inglorion@chromium.org> | 2021-08-06 21:08:43 +0000 |
|---|---|---|
| committer | Bob Haarman <inglorion@chromium.org> | 2021-08-09 21:06:29 +0000 |
| commit | 2adf813f6ab7a3f9af6d61f6274ad58d449f2999 (patch) | |
| tree | 6c7cbb96ec6479a1049d08d8d208d55fb8fdaff2 | |
| parent | 046d57a5e2567e6c6b9880f8c2be476c9a5fc433 (diff) | |
| download | toolchain-utils-2adf813f6ab7a3f9af6d61f6274ad58d449f2999.tar.gz | |
llvm_tools: more informative message when checking out llvm-project-copy
We keep a clone of the LLVM repository, which we create if it did not
already exist. This takes a while, so we inform the user. However, the
message did not specify where we are cloning to or why. This change
makes the message take the form:
Checking out LLVM to '<path>'
so that we can map between commit hashes and revision numbers.
This may take a while, but only has to be done once.
BUG=b:192699914
TEST=ran llvm_tools/get_llvm_hash.py with a few parameters
Change-Id: I86d83f03ab293c5cfb3014e80175c674740a2818
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3078689
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
| -rwxr-xr-x | llvm_tools/get_llvm_hash.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm_tools/get_llvm_hash.py b/llvm_tools/get_llvm_hash.py index 5f0ef58b..83b5ae76 100755 --- a/llvm_tools/get_llvm_hash.py +++ b/llvm_tools/get_llvm_hash.py @@ -190,8 +190,9 @@ def GetAndUpdateLLVMProjectInLLVMTools(): if not os.path.isdir(abs_path_to_llvm_project_dir): print( - 'Checking out LLVM from scratch. This could take a while...\n' - '(This should only need to be done once, though.)', + (f'Checking out LLVM to {abs_path_to_llvm_project_dir}\n' + 'so that we can map between commit hashes and revision numbers.\n' + 'This may take a while, but only has to be done once.'), file=sys.stderr) os.mkdir(abs_path_to_llvm_project_dir) |