coreclr
coreclr copied to clipboard
[release/3.1] backports dotnet#67763: fix build with musl 1.2.3
Backports https://github.com/dotnet/runtime/pull/67772 to release/3.1 to fix build with musl 1.2.3
@nekopsykose I see that you backported the musl 123 fix on Alpine's side. (thanks for that!) You removed whatever modifications in src/pal/src/init/pal.cpp
. Why was that? Could this be the source of the failing tests on release/3.1 and release/5.0?
afaik those don't change any semantics (NULL == nullptr), it just failed to apply cleanly and i was lazy for a mere backport :)
afaik those don't change any semantics (NULL == nullptr), it just failed to apply cleanly and i was lazy for a mere backport :)
Gotcha, then for the upgrade to 3.1.418 I'll apply proper change - it's what upstream wants.
Confirmed good build in Alpine pipelines
@ayakael is this ready to merge? Can you get a sign-off from someone in your team?
Ah, it's missing the servicing consider
label. I added it. This needs to go through tactics.
@janvorli can you do a code review? Same as https://github.com/dotnet/runtime/pull/67772
@ayakael, could you update the PR body with a template we use to get servicing approval? Once I get it from you, I will get an approval. Please refer an example from https://github.com/dotnet/runtime/pull/68934 which includes:
- What is the issue
- What was the root cuase
- What was the fix
- Customer Impact
- Testing
- Risk (if it is low, why is it low?)
Removing servicing consider until I hear back from @ayakael and get code review from @janvorli. cc @mangod9.
@ayakael, I have a question to you. You put up servicing PRs for only .NET 3.1 and .NET 5. I don't see .NET 6 patch. What is your plan for .NET 6 patch?
When fixing issues, we need to fix all the supported releases between the lowest needed (in this case .NET 3.1) and the currently supported release (.NET 6). Since .NET 5 is going out of support, we will need to patch both .NET 3.1 and .NET 6 if you need to patch .NET 3.1.
@ayakael the PR has been inactive for a few months. I'll close it. Feel free to reopen if you can answer the questions above.