-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Items returned by tasks not considered for EmbedInBinlog #7665
Copy link
Copy link
Closed
Labels
Area: DebuggabilityIssues impacting the diagnosability of builds, including logging and clearer error messages.Issues impacting the diagnosability of builds, including logging and clearer error messages.Area: LoggingGood First IssueSelf-contained issues good for first-time contributors.Self-contained issues good for first-time contributors.help wantedIssues that the core team doesn't plan to work on, but would accept a PR for. Comment to claim.Issues that the core team doesn't plan to work on, but would accept a PR for. Comment to claim.triaged
Metadata
Metadata
Assignees
Labels
Area: DebuggabilityIssues impacting the diagnosability of builds, including logging and clearer error messages.Issues impacting the diagnosability of builds, including logging and clearer error messages.Area: LoggingGood First IssueSelf-contained issues good for first-time contributors.Self-contained issues good for first-time contributors.help wantedIssues that the core team doesn't plan to work on, but would accept a PR for. Comment to claim.Issues that the core team doesn't plan to work on, but would accept a PR for. Comment to claim.triaged
Type
Fields
Give feedbackNo fields configured for issues without a type.
I'd expect this to embed
C:\temp\1.txtinto the binlog, but it doesn't:As a workaround to fix this, also need to add an explicit ItemGroup:
I think this line:
msbuild/src/Build/Logging/BinaryLogger/BuildEventArgsWriter.cs
Line 521 in f1dae6a
should also allow for
TaskParameterMessageKind.TaskOutput