Skip to content

Remove etw --exclude-feature tests. - #89107

Merged
vitek-karas merged 1 commit into
dotnet:mainfrom
Unity-Technologies:linker-remove-etw-removal-tests
Jul 21, 2023
Merged

Remove etw --exclude-feature tests.#89107
vitek-karas merged 1 commit into
dotnet:mainfrom
Unity-Technologies:linker-remove-etw-removal-tests

Conversation

@mrvoorhe

Copy link
Copy Markdown
Contributor

--exclude-feature logic was removed. If there is no intent to support this with coreclr bcl then we won't worry about it either.

While we do need UnityLinker to continue to handle mono class libraries for awhile longer, the --exclude-feature functionality is a niche thing we can drop now.

`--exclude-feature` logic was removed.  If there is no intent to support this with coreclr bcl then we won't worry about it either.

While we do need UnityLinker to continue to handle mono class libraries for awhile longer, the `--exclude-feature` functionality is a niche thing we can drop now.
@mrvoorhe
mrvoorhe requested a review from marek-safar as a code owner July 18, 2023 16:51
@ghost ghost added linkable-framework Issues associated with delivering a linker friendly framework needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners community-contribution Indicates that the PR has been added by a community member labels Jul 18, 2023
@ghost

ghost commented Jul 18, 2023

Copy link
Copy Markdown

Tagging subscribers to 'linkable-framework': @eerhardt, @vitek-karas, @LakshanF, @sbomer, @joperezr, @marek-safar
See info in area-owners.md if you want to be subscribed.

Issue Details

--exclude-feature logic was removed. If there is no intent to support this with coreclr bcl then we won't worry about it either.

While we do need UnityLinker to continue to handle mono class libraries for awhile longer, the --exclude-feature functionality is a niche thing we can drop now.

Author: mrvoorhe
Assignees: -
Labels:

linkable-framework, needs-area-label

Milestone: -

@mrvoorhe

Copy link
Copy Markdown
Contributor Author

@marek-safar or @vitek-karas is this PR ok?

@vitek-karas

Copy link
Copy Markdown
Member

Failures are unrelated.

@vitek-karas
vitek-karas merged commit 51af14c into dotnet:main Jul 21, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Aug 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community-contribution Indicates that the PR has been added by a community member linkable-framework Issues associated with delivering a linker friendly framework needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants