Commit 8daf2442 authored by Takuto Ikuta's avatar Takuto Ikuta Committed by LUCI CQ

gsutil: use shorter path for temp dir

Temp dir used to download gsutil seems somehow hit max path length limit
on Windows builders.

Bug: 1264242
Change-Id: Ib924932542dbbe68f4ef58ca604724dcb645330f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3247566
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Junji Watanabe <jwata@google.com>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Owners-Override: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: 's avatarJunji Watanabe <jwata@google.com>
Reviewed-by: 's avatarFumitoshi Ukai <ukai@google.com>
parent 673e8ede
......@@ -79,7 +79,7 @@ def download_gsutil(version, target_dir):
@contextlib.contextmanager
def temporary_directory(base):
tmpdir = tempfile.mkdtemp(prefix='gsutil_py', dir=base)
tmpdir = tempfile.mkdtemp(prefix='t', dir=base)
try:
yield tmpdir
finally:
......@@ -122,7 +122,7 @@ def ensure_gsutil(version, target, clean):
if cleanup_path:
shutil.rmtree(cleanup_path)
download_dir = os.path.join(instance_dir, 'download')
download_dir = os.path.join(instance_dir, 'd')
target_zip_filename = download_gsutil(version, instance_dir)
with zipfile.ZipFile(target_zip_filename, 'r') as target_zip:
target_zip.extractall(download_dir)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment