Skip to content

[runtime] Remove remnant LinuxTracepoints references#2925

Merged
mdh1418 merged 1 commit into
dotnet:mainfrom
mdh1418:remove_remnant_linux_tracepoints_usage
Oct 24, 2025
Merged

[runtime] Remove remnant LinuxTracepoints references#2925
mdh1418 merged 1 commit into
dotnet:mainfrom
mdh1418:remove_remnant_linux_tracepoints_usage

Conversation

@mdh1418

@mdh1418 mdh1418 commented Oct 15, 2025

Copy link
Copy Markdown
Member

In dotnet/runtime, LinuxTracepoints references may be completely removed through dotnet/runtime#120720. Once that is merged and flowed into this repo, there are some more dangling references to LinuxTracepoints that should no longer be needed.

@mdh1418 mdh1418 requested review from jkotas and mmitche October 15, 2025 15:56
@mdh1418 mdh1418 requested review from a team as code owners October 15, 2025 15:56
@mdh1418 mdh1418 added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Oct 15, 2025
Copilot AI review requested due to automatic review settings October 15, 2025 15:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes remaining references to LinuxTracepoints following its complete removal from dotnet/runtime in PR #120720. The changes clean up TODO comments and binary file allowlist entries that are no longer needed.

  • Removed a TODO comment referencing LinuxTracepoints from the build configuration
  • Removed LinuxTracepoints test data file from the allowed VMR binaries list

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
repo-projects/runtime.proj Removed TODO comment about LinuxTracepoints from build arguments configuration
eng/allowed-vmr-binaries.txt Removed LinuxTracepoints test output file from the allowed binaries list

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mdh1418 mdh1418 removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Oct 23, 2025
@mdh1418 mdh1418 force-pushed the remove_remnant_linux_tracepoints_usage branch from ae29005 to 60823d1 Compare October 23, 2025 19:46
@mdh1418 mdh1418 merged commit f5afaa5 into dotnet:main Oct 24, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants