Skip to content

Conversation

@VSadov
Copy link
Member

@VSadov VSadov commented Apr 27, 2023

@ghost ghost assigned VSadov Apr 27, 2023
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 27, 2023
@VSadov VSadov added source-build Issues relating to dotnet/source-build and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Apr 27, 2023
@VSadov VSadov requested review from am11 and jkotas April 27, 2023 17:28
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thanks

Copy link
Member

@am11 am11 Apr 27, 2023

Choose a reason for hiding this comment

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

This is a wrong file (for HP libunwind). This PR should use src/native/external/llvm-libunwind-version.txt

Also, please keep the format consistent: Apply <link>

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I suspected that something is wrong, since the other content in the file was referring to libunwind

@VSadov
Copy link
Member Author

VSadov commented Apr 27, 2023

Thanks!!

@jkotas jkotas merged commit 375cc6d into dotnet:main Apr 28, 2023
@VSadov VSadov deleted the arm-build-break branch April 28, 2023 07:08
@ghost ghost locked as resolved and limited conversation to collaborators May 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

source-build Issues relating to dotnet/source-build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants