Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[mono] Linking statically ICU shim on mono #35790
[mono] Linking statically ICU shim on mono #35790
Changes from 35 commits
c996d5e
fc6ec08
da6f766
6f0bb5c
806ab54
d1cd362
63b31b3
9dbce14
e582815
a17b189
b570c23
2571f55
efa2cb5
2459b39
c233c3f
2d41932
13bd64d
5626e11
5b6820a
55f7fc9
78886b3
1e66f98
b07315c
a9ed865
3d5d97b
31200b3
2d25950
fbd44f2
d641726
2657cb6
1bebc32
2bf4f50
b483e41
376764a
8327179
5c06e56
1af719d
06460ff
1a884ad
dc17ef9
3bbdf1e
2e45cbe
d83a61c
11c7a3d
2737b96
d990080
d7eefa3
03bdce3
8da62b1
676b58e
e974e46
464656a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where are these warnings coming from? Not saying they look bad, just wondering why this specific list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should have a tracking item somewhere about revisiting this since we won't be using the libmono->libcoreclr rename hack forever.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commenting just to note that this is a significant behavioral change to ensure we default to RTLD_LOCAL on both OSX and Linux. This was the intention from the start and was discussed beforehand, but is worth bringing attention to.