From b3c6512be6634eb0221d3ccd8cd2ce3cd45ca2ef Mon Sep 17 00:00:00 2001 From: Chris Blume Date: Fri, 18 Feb 2022 22:23:08 +0000 Subject: [PATCH] Correct trailing path separator In [1], I landed a change that contained a bug. It meant to trim off a trailing path separator. Instead of trimming the end, it trimmed everything but the first char (since the path separator is 1 char long). This CL fixes that bug. [1] https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3451982 Change-Id: Ie4aebfc1042908437bd603da1c20681f19c50f73 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3473285 Auto-Submit: Chris Blume Reviewed-by: Bruce Dawson Commit-Queue: Bruce Dawson --- win_toolchain/package_from_installed.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/win_toolchain/package_from_installed.py b/win_toolchain/package_from_installed.py index 0666b5790..52aebbfca 100644 --- a/win_toolchain/package_from_installed.py +++ b/win_toolchain/package_from_installed.py @@ -188,7 +188,7 @@ def BuildFileList(override_dir, include_arm): # Strip off a trailing slash if present if sdk_path.endswith(os.path.sep): - sdk_path = sdk_path[:len(os.path.sep)] + sdk_path = sdk_path[:-len(os.path.sep)] debuggers_path = os.path.join(sdk_path, 'Debuggers') if not os.path.exists(debuggers_path):