From f617df3e0f5226b1503a4c61e48cd691dde7965e Mon Sep 17 00:00:00 2001 From: Chet Husk Date: Tue, 28 Feb 2023 18:08:08 -0600 Subject: [PATCH] Update usage of AllowEmptyTelemetry based on changes to the task in the SDK In https://github.com/dotnet/sdk/pull/30269 the API of the AllowEmptyTelemetry task changed to allow for more granular hashing of the collected telemetry properties. This change was against a servicing branch and flowed into SDK main. Sometime after that change, these targets moved from SDK to Runtime, and so the change was lost. This re-applies the change to Runtime main to address breaks seen in source-build. --- .../Crossgen2Tasks/Microsoft.NET.CrossGen.targets | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/tasks/Crossgen2Tasks/Microsoft.NET.CrossGen.targets b/src/tasks/Crossgen2Tasks/Microsoft.NET.CrossGen.targets index 251336540b5bb3..a2912cc0822849 100644 --- a/src/tasks/Crossgen2Tasks/Microsoft.NET.CrossGen.targets +++ b/src/tasks/Crossgen2Tasks/Microsoft.NET.CrossGen.targets @@ -346,7 +346,13 @@ Copyright (c) .NET Foundation. All rights reserved. _CreateR2RImages; _CreateR2RSymbols"> - + + <_R2RCrossgenTelemetry Include="PublishReadyToRunUseCrossgen2" Value="$(PublishReadyToRunUseCrossgen2)" /> + <_R2RCrossgenTelemetry Include="Crossgen2PackVersion" Value="%(ResolvedCrossgen2Pack.NuGetPackageVersion)" /> + <_R2RCrossgenTelemetry Include="CompileListCount" Value="@(_ReadyToRunCompileList->Count())" /> + <_R2RCrossgenTelemetry Include="FailedCount" Value="@(_ReadyToRunCompilationFailures->Count())" /> + + @@ -519,4 +525,4 @@ Copyright (c) .NET Foundation. All rights reserved. Include="@(_ReadyToRunSymbolsCompileList)" /> - \ No newline at end of file +