From 38500c0be769da4f023b238498b7b8287462250c Mon Sep 17 00:00:00 2001 From: Michael Nebel Date: Fri, 19 Jun 2026 10:01:25 +0200 Subject: [PATCH] C#: NoOpPackageRestore only needs the PackageConfigs collection and not the entire FileProvider. --- .../PackagesConfigRestorer.cs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs index 68a0a746ca91..a4fca7e2c84d 100644 --- a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs +++ b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs @@ -40,7 +40,7 @@ public static IPackagesConfigRestore Create(FileProvider fileProvider, Dependenc return new NugetExeWrapper(fileProvider, packageDirectory, logger, useDefaultFeed); } - return new NoOpPackagesConfig(fileProvider, logger); + return new NoOpPackagesConfig(fileProvider.PackagesConfigs, logger); } /// @@ -343,15 +343,15 @@ public void Dispose() private class NoOpPackagesConfig : IPackagesConfigRestore { private readonly Semmle.Util.Logging.ILogger logger; - private readonly FileProvider fileProvider; + private readonly ICollection packagesConfigs; - public NoOpPackagesConfig(FileProvider fileProvider, Semmle.Util.Logging.ILogger logger) + public NoOpPackagesConfig(ICollection packagesConfigs, Semmle.Util.Logging.ILogger logger) { - this.fileProvider = fileProvider; + this.packagesConfigs = packagesConfigs; this.logger = logger; } - public int PackageCount => fileProvider.PackagesConfigs.Count; + public int PackageCount => packagesConfigs.Count; public int InstallPackages() {