Skip to content

Commit e176b32

Browse files
committed
Remove environment dictionary passing
1 parent 648c06c commit e176b32

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

csharp/autobuilder/Semmle.Autobuild.CSharp/CSharpAutobuilder.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public override BuildScript GetBuildScript()
5353
attempt = DotNetRule.WithDotNet(this, ensureDotNetAvailable: true, (dotNetPath, env) =>
5454
{
5555
// No need to check that the extractor has been executed in buildless mode
56-
return new StandaloneBuildRule(dotNetPath, env).Analyse(this, false);
56+
return new StandaloneBuildRule(dotNetPath).Analyse(this, false);
5757
});
5858
break;
5959
case CSharpBuildStrategy.MSBuild:

csharp/autobuilder/Semmle.Autobuild.CSharp/StandaloneBuildRule.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,10 @@ namespace Semmle.Autobuild.CSharp
99
internal class StandaloneBuildRule : IBuildRule<CSharpAutobuildOptions>
1010
{
1111
private readonly string? dotNetPath;
12-
private readonly IDictionary<string, string>? env;
1312

14-
internal StandaloneBuildRule(string? dotNetPath, IDictionary<string, string>? env)
13+
internal StandaloneBuildRule(string? dotNetPath)
1514
{
1615
this.dotNetPath = dotNetPath;
17-
this.env = env;
1816
}
1917

2018
public BuildScript Analyse(IAutobuilder<CSharpAutobuildOptions> builder, bool auto)
@@ -27,7 +25,7 @@ public BuildScript Analyse(IAutobuilder<CSharpAutobuildOptions> builder, bool au
2725
}
2826

2927
var standalone = builder.Actions.PathCombine(builder.CodeQLExtractorLangRoot, "tools", builder.CodeQlPlatform, "Semmle.Extraction.CSharp.Standalone");
30-
var cmd = new CommandBuilder(builder.Actions, environment: this.env);
28+
var cmd = new CommandBuilder(builder.Actions);
3129
cmd.RunCommand(standalone);
3230

3331
if (!string.IsNullOrEmpty(this.dotNetPath))

0 commit comments

Comments
 (0)