Skip to content

Commit

Permalink
Moved "Installing Dependencies" message to only print if there are de…
Browse files Browse the repository at this point in the history
…pendencies to install. (microsoft#1851)
  • Loading branch information
jedieaston authored Jan 13, 2022
1 parent 5f3f9d4 commit ae586be
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/AppInstallerCLICore/Workflows/DependenciesFlow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -155,8 +155,6 @@ namespace AppInstaller::CLI::Workflow
return;
}

info << Resource::String::DependenciesFlowInstall << std::endl;

context << OpenDependencySource;
if (context.IsTerminated())
{
Expand Down Expand Up @@ -238,6 +236,11 @@ namespace AppInstaller::CLI::Workflow
}
}

if (!dependencyPackageContexts.empty())
{
info << Resource::String::DependenciesFlowInstall << std::endl;
}

// Install dependencies in the correct order
context.Add<Execution::Data::PackagesToInstall>(std::move(dependencyPackageContexts));
context << Workflow::InstallMultiplePackages(m_dependencyReportMessage, APPINSTALLER_CLI_ERROR_INSTALL_DEPENDENCIES, {}, false, true);
Expand Down

0 comments on commit ae586be

Please sign in to comment.