This is an automated email from the ASF dual-hosted git repository. joemcdonnell pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 5d5f33115523dce3b4e29adfa4e921a027e70c90 Author: Joe McDonnell <[email protected]> AuthorDate: Thu Apr 27 17:28:29 2023 -0700 IMPALA-9627 (followup): Use -q for bootstrap_toolchain.py's wget calls The original fix for IMPALA-9627 removed the local check_output() function for bin/bootstrap_toolchain.py and replaced it with subprocess.check_call(). The local check_output() function was preventing output from the wget call from going to the commandline. The subprocess.check_call() doesn't do this. Since we don't want this output anyway, this modifies the wget call to add "-q" to avoid the extra output. Testing: - Verified the extra output was gone Change-Id: If5b81f502a73551d269a0379ed0106ba4d3c8363 Reviewed-on: http://gerrit.cloudera.org:8080/19814 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- bin/bootstrap_toolchain.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/bootstrap_toolchain.py b/bin/bootstrap_toolchain.py index 3817efb42..78b493d54 100755 --- a/bin/bootstrap_toolchain.py +++ b/bin/bootstrap_toolchain.py @@ -123,7 +123,7 @@ def wget_and_unpack_package(download_path, file_name, destination, wget_no_clobb download_path, destination, file_name, attempt)) # --no-clobber avoids downloading the file if a file with the name already exists try: - cmd = ["wget", download_path, + cmd = ["wget", "-q", download_path, "--output-document={0}/{1}".format(destination, file_name)] if wget_no_clobber: cmd.append("--no-clobber")
