commit c91e2aaa9ca9d19484ad4c9d04e7a6bec0609339
author Aravind Vasudevan <aravindvasudev@google.com> Thu Aug 10 23:42:11 2023 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Aug 10 23:42:11 2023 +0000
tree 2bc05714c4c80557db73d2ade681641331fa3b41
parent 25410068111f6a8dba36495ea146fa5cd4eb78f3

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 <ddoman@chromium.org>
Auto-Submit: Aravind Vasudevan <aravindvasudev@google.com>
Reviewed-by: Scott Lee <ddoman@chromium.org>
Commit-Queue: Aravind Vasudevan <aravindvasudev@google.com>

gclient.py

diff --git a/gclient.py b/gclient.py
index 5dffe75..64a9ccf 100755
--- a/gclient.py
+++ b/gclient.py
@@ -966,7 +966,7 @@
 
     # 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():