From 28dd2e979b73feed485f644df0e13a6f767170e7 Mon Sep 17 00:00:00 2001
From: Alex Denisov <alexdenisov@github.com>
Date: Fri, 12 Jan 2024 12:34:13 +0100
Subject: [PATCH] Revert "Swift: separate installation of dependencies and
 autobuilding"

This reverts commit dd13ea3d0a41764b065f6e50efff40aa4fc8ffca.
---
 swift/swift-autobuilder/swift-autobuilder.cpp | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/swift/swift-autobuilder/swift-autobuilder.cpp b/swift/swift-autobuilder/swift-autobuilder.cpp
index 32cd52541a9..c6b2582ed2b 100644
--- a/swift/swift-autobuilder/swift-autobuilder.cpp
+++ b/swift/swift-autobuilder/swift-autobuilder.cpp
@@ -54,11 +54,6 @@ static bool buildSwiftPackages(const std::vector<std::filesystem::path>& swiftPa
   return any_successful;
 }
 
-static void installDependencies(const CLIArgs& args) {
-  auto structure = scanProjectStructure(args.workingDir);
-  installDependencies(structure, args.dryRun);
-}
-
 static bool autobuild(const CLIArgs& args) {
   auto structure = scanProjectStructure(args.workingDir);
   auto& xcodeTargets = structure.xcodeTargets;
@@ -87,6 +82,7 @@ static bool autobuild(const CLIArgs& args) {
     return false;
   } else if (!xcodeTargets.empty()) {
     LOG_INFO("Building Xcode target: {}", xcodeTargets.front());
+    installDependencies(structure, args.dryRun);
     auto buildSucceeded = buildXcodeTarget(xcodeTargets.front(), args.dryRun);
     // If build failed, try to build Swift packages
     if (!buildSucceeded && !swiftPackages.empty()) {
@@ -117,7 +113,6 @@ static CLIArgs parseCLIArgs(int argc, char** argv) {
 
 int main(int argc, char** argv) {
   auto args = parseCLIArgs(argc, argv);
-  installDependencies(args);
   auto success = autobuild(args);
   codeql::Log::flush();
   if (!success) {