Skip to content

[main] Source code updates from dotnet/runtime#4727

Merged
ViktorHofer merged 6 commits intomainfrom
darc-main-0d46b718-7792-4b8c-b50c-8f9b703de37e
Feb 11, 2026
Merged

[main] Source code updates from dotnet/runtime#4727
ViktorHofer merged 6 commits intomainfrom
darc-main-0d46b718-7792-4b8c-b50c-8f9b703de37e

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Feb 9, 2026

Note

This is a codeflow update. It may contain both source code changes from
the source repo
as well as dependency updates. Learn more here.

This pull request brings the following source code changes

From https://github.com/dotnet/runtime

Diff the source with this PR branch
darc vmr diff --name-only https://github.com/dotnet/runtime:0b91113ce10b7b868f0df88a1c87ae8f2f04bbf0..https://github.com/dotnet/dotnet:darc-main-0d46b718-7792-4b8c-b50c-8f9b703de37e

@dotnet-policy-service dotnet-policy-service bot requested a review from a team February 9, 2026 02:05
@ViktorHofer
Copy link
Member

@steveisok the build with ninja by default und Linux and Mac needs a reaction here. Looks like the images don't have Ninja installed.

@steveisok
Copy link
Member

@steveisok the build with ninja by default und Linux and Mac needs a reaction here. Looks like the images don't have Ninja installed.

dotnet/dotnet-buildtools-prereqs-docker#1587

@ViktorHofer
Copy link
Member

What needs to be done for Mac?

@steveisok
Copy link
Member

What needs to be done for Mac?

Is there a failure on the mac I'm not seeing?

@ViktorHofer
Copy link
Member

The iossimulator job which uses macos is passing so apparently not. Does that job provide enough validation?

@steveisok
Copy link
Member

The iossimulator job which uses macos is passing so apparently not. Does that job provide enough validation?

It'll check for ninja and fail, so yes.

@ViktorHofer
Copy link
Member

ViktorHofer commented Feb 9, 2026

I meant if we think that's sufficient given that we use another machine pool (arm64) internally. But we can find out post-merge.

@dotnet-maestro
Copy link
Contributor Author

Note

PRs from original repository included in this codeflow update:

💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance.

@akoeplinger
Copy link
Member

src/aspnetcore/src/HealthChecks/HealthChecks/src/DefaultHealthCheckService.cs(225,10): error SYSLIB1025: (NETCORE_ENGINEERING_TELEMETRY=Build) Multiple logging methods are using event name HealthCheckEnd in class Log (https://learn.microsoft.com/dotnet/fundamentals/syslib-diagnostics/syslib1025)
src/aspnetcore/src/HealthChecks/HealthChecks/src/DefaultHealthCheckService.cs(228,10): error SYSLIB1025: (NETCORE_ENGINEERING_TELEMETRY=Build) Multiple logging methods are using event name HealthCheckEnd in class Log (https://learn.microsoft.com/dotnet/fundamentals/syslib-diagnostics/syslib1025)

Most likely caused by dotnet/runtime#124077 but I don't understand why this even gets raised because the warning is suppressed in code:

#pragma warning disable SYSLIB1006
#pragma warning disable SYSLIB1025
[LoggerMessage(EventIds.HealthCheckEndId, LogLevel.Debug, HealthCheckEndText, EventName = EventIds.HealthCheckEndName)]
private static partial void HealthCheckEndHealthy(ILogger logger, string HealthCheckName, HealthStatus HealthStatus, double ElapsedMilliseconds, string? HealthCheckDescription);
[LoggerMessage(EventIds.HealthCheckEndId, LogLevel.Warning, HealthCheckEndText, EventName = EventIds.HealthCheckEndName)]
private static partial void HealthCheckEndDegraded(ILogger logger, string HealthCheckName, HealthStatus HealthStatus, double ElapsedMilliseconds, string? HealthCheckDescription, Exception? exception);
[LoggerMessage(EventIds.HealthCheckEndId, LogLevel.Error, HealthCheckEndText, EventName = EventIds.HealthCheckEndName)]
private static partial void HealthCheckEndUnhealthy(ILogger logger, string HealthCheckName, HealthStatus HealthStatus, double ElapsedMilliseconds, string? HealthCheckDescription, Exception? exception);
#pragma warning restore SYSLIB1025
#pragma warning restore SYSLIB1006

@akoeplinger
Copy link
Member

oh I see, the PR has a notice:

Pragma Suppression Behavioral Note
In the test .targets and .WithDiagnostics.cs files, the PR adds project-level for SYSLIB1002;SYSLIB1013;SYSLIB1018;8795 and 8795. This is because DiagnosticInfo.GetTrimmedLocation creates locations without SyntaxTree references, breaking #pragma warning disable matching. While this only affects the test project directly, the same DiagnosticInfo pattern is used in the production Roslyn 4.0 generator path, meaning end-users of the LoggerMessageGenerator will lose the ability to suppress diagnostics via #pragma directives. This is a behavioral regression. The existing System.Text.Json generator already has this limitation too, so the Logging source generator will start to have the same behavior as System.Text.Json.

@dotnet-policy-service dotnet-policy-service bot requested a review from a team February 10, 2026 15:14
@ViktorHofer ViktorHofer merged commit 87581bf into main Feb 11, 2026
16 checks passed
@ViktorHofer ViktorHofer deleted the darc-main-0d46b718-7792-4b8c-b50c-8f9b703de37e branch February 11, 2026 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants