Skip to content

[WIP] Modernize Splat multi-TFM via OS-aware MSBuild#1401

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-1400
Closed

[WIP] Modernize Splat multi-TFM via OS-aware MSBuild#1401
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-1400

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 7, 2025

  • Analyze current Directory.Build.props and project structure
  • Create centralized TFM buckets with OS-aware conditions in src/Directory.Build.props
  • Add System.Net.Http package reference fix for net462/net472
  • Configure tests/benchmarks to use modern .NET targets automatically
  • Test build functionality on current environment
  • Verify all projects can reference centralized properties
  • Update any project files that need to consume new centralized properties
  • Add contributor guidelines for the new TFM approach

Key Changes Planned:

  • Centralize all TFM logic in src/Directory.Build.props using $([MSBuild]::IsOsPlatform(...)) conditions
  • Create property buckets: SplatModernTargets, SplatCoreTargets, SplatNetFxTargets, etc.
  • OS-aware composition for Windows (.NET Framework + Windows-specific), macOS (Apple TFMs), Linux (Core + Android)
  • Automatic test/benchmark TFM detection to use modern .NET instead of netstandard
  • System.Net.Http version pinning for net462/net472 to resolve compile conflicts

Environment Note: Current SDK is .NET 8.0.119, so will use net8.0 for modern targets until .NET 9.0 SDK is available.

Fixes #1400.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 22, 2025
@glennawatson glennawatson deleted the copilot/fix-1400 branch September 23, 2025 02:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modernize Splat multi-TFM via OS-aware MSBuild

2 participants