From c91e2aaa9ca9d19484ad4c9d04e7a6bec0609339 Mon Sep 17 00:00:00 2001 From: Aravind Vasudevan Date: Thu, 10 Aug 2023 23:42:11 +0000 Subject: [PATCH] Fix filepath issue within ParseGitSubmodules() The filepath already has cwd appended so the command doesn't need cwd. Change-Id: I20c6fea27aee963e90cdb8a15cb087771dafa0fa Bug: 1472143 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4771942 Commit-Queue: Scott Lee Auto-Submit: Aravind Vasudevan Reviewed-by: Scott Lee Commit-Queue: Aravind Vasudevan --- gclient.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gclient.py b/gclient.py index 5dffe75ba..64a9ccf08 100755 --- a/gclient.py +++ b/gclient.py @@ -966,7 +966,7 @@ class Dependency(gclient_utils.WorkItem, DependencySettings): # Get .gitmodules fields gitmodules_entries = subprocess2.check_output( - ['git', 'config', '--file', filepath, '-l'], cwd=cwd).decode('utf-8') + ['git', 'config', '--file', filepath, '-l']).decode('utf-8') gitmodules = {} for entry in gitmodules_entries.splitlines():