Skip to content

Commit 735e420

Browse files
committed
C#: Don't use the NuGet CLI when private registries are configured.
1 parent fd2bfc8 commit 735e420

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ internal class PackagesConfigRestoreFactory
3535
{
3636
public static IPackagesConfigRestore Create(IFileProvider fileProvider, DependencyDirectory packageDirectory, Semmle.Util.Logging.ILogger logger, FeedManager feedManager)
3737
{
38-
if (SystemBuildActions.Instance.IsWindows() || SystemBuildActions.Instance.IsMonoInstalled())
38+
if ((SystemBuildActions.Instance.IsWindows() || SystemBuildActions.Instance.IsMonoInstalled()) && !feedManager.HasPrivateRegistryFeeds)
3939
{
4040
return new NugetExeWrapper(fileProvider, packageDirectory, logger, feedManager);
4141
}
@@ -252,7 +252,7 @@ public int InstallPackages()
252252
{
253253
if (PackageCount > 0)
254254
{
255-
logger.LogInfo("Found packages.config files, but nuget.exe cannot be used to restore packages on this platform. Skipping restore of packages.config files.");
255+
logger.LogInfo("Found packages.config files, but nuget.exe cannot be used to restore packages on this platform or if private registry feeds are configured. Skipping restore of packages.config files.");
256256
}
257257
return 0;
258258
}

0 commit comments

Comments
 (0)