Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/6.0] Fix build for musl 1.2.3 (backports #67772) #75529

Merged

Conversation

ayakael
Copy link
Contributor

@ayakael ayakael commented Sep 13, 2022

Backports #67772

Not a functional change.
This is to fix the build with musl 1.2.3 , which was released 7 Apr 2022

Most of the change is switching NULL -> nullptr in some PAL functions to match musl changes.

Customer Impact

No impact, this is an infra/build level of a fix.

Risk

Low, it is a build-related change that should be transparent outside of dotnet/runtime.

Testing

Regular tests.

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Sep 13, 2022
@carlossanlop
Copy link
Member

@janvorli can you help fill out the template? This needs to go through Tactics, so once it's ready, please add the servicing-consider label.

@VSadov VSadov added the Servicing-consider Issue for next servicing release review label Sep 14, 2022
@VSadov VSadov requested a review from jeffschwMSFT September 14, 2022 19:04
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Sep 14, 2022
@carlossanlop
Copy link
Member

CI is green. Approved. Signed off. No OOB package authoring changes since this is native code. :shipit:

@carlossanlop carlossanlop merged commit 8c6bcad into dotnet:release/6.0 Sep 14, 2022
vseanreesermsft pushed a commit to vseanreesermsft/runtime that referenced this pull request Oct 11, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Oct 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-PAL-coreclr community-contribution Indicates that the PR has been added by a community member Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants