Skip to content

Commit 0bae4bb

Browse files
committed
Merged PR 714300: Add Dependency Comment to BXL.Processes.dsc
Adding a dependency comment in BXL.Processes.dsc to ensure dependencies are not added into BXL.Processes now that it's being consumed by MSBuild
1 parent 4be0334 commit 0bae4bb

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Public/Src/Engine/Processes/BuildXL.Processes.dsc

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ namespace Processes {
1414
sources: globR(d`.`, "*.cs"),
1515
generateLogs: true,
1616
references: [
17+
// IMPORTANT!!! Do not add non-bxl dependencies or any bxl projects apart from Native and Utilities.Core into this project
18+
// This is consumed by MSBuild and adding any additional reference will cause it to break! BXL specific change
19+
// should go to 'ProcessPipExecutor' instead.
20+
1721
...addIfLazy(!BuildXLSdk.isDotNetCore, () => [
1822
importFrom("System.Text.Json").withQualifier({targetFramework: "netstandard2.0"}).pkg,
1923
importFrom("System.Memory").withQualifier({ targetFramework: "netstandard2.0" }).pkg,
@@ -22,7 +26,6 @@ namespace Processes {
2226
...addIf(BuildXLSdk.isFullFramework,
2327
BuildXLSdk.NetFx.System.IO.Compression.dll,
2428
BuildXLSdk.NetFx.System.Management.dll,
25-
BuildXLSdk.NetFx.System.Net.Http.dll,
2629
NetFx.Netstandard.dll
2730
),
2831
...addIf(BuildXLSdk.isDotNetCoreOrStandard,

0 commit comments

Comments
 (0)