From 996e1bfc45d006bb8d0bd7f082a2ae65c20087d5 Mon Sep 17 00:00:00 2001
From: Jarek Kobus <jaroslaw.kobus@qt.io>
Date: Fri, 2 Sep 2022 14:58:50 +0200
Subject: [PATCH] VcsBase: Reuse vcsFullySynchronousExec() in
 vcsSynchronousExec()

Both implementations look the same nowadays.

Change-Id: I00503f625395db3354eefa8d55326ee22f452c2e
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
---
 src/plugins/vcsbase/vcsbaseclient.cpp | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/plugins/vcsbase/vcsbaseclient.cpp b/src/plugins/vcsbase/vcsbaseclient.cpp
index 12bdd51e538..eb078677fc9 100644
--- a/src/plugins/vcsbase/vcsbaseclient.cpp
+++ b/src/plugins/vcsbase/vcsbaseclient.cpp
@@ -172,11 +172,7 @@ CommandResult VcsBaseClientImpl::vcsSynchronousExec(const FilePath &workingDir,
                                                     unsigned flags,
                                                     QTextCodec *outputCodec) const
 {
-    Environment env = processEnvironment();
-    VcsCommand command(workingDir, env.isValid() ? env : Environment::systemEnvironment());
-    command.addFlags(flags);
-    command.setCodec(outputCodec);
-    return command.runCommand({vcsBinary(), args}, vcsTimeoutS());
+    return vcsFullySynchronousExec(workingDir, args, flags, vcsTimeoutS(), outputCodec);
 }
 
 int VcsBaseClientImpl::vcsTimeoutS() const