Skip to content

Don't add the filled in template file to FileWries#121080

Merged
jkoritzinsky merged 1 commit intomainfrom
jkoritzinsky-patch-1
Oct 24, 2025
Merged

Don't add the filled in template file to FileWries#121080
jkoritzinsky merged 1 commit intomainfrom
jkoritzinsky-patch-1

Conversation

@jkoritzinsky
Copy link
Member

MSBuild cleans up intermediate file writes from previous builds that weren't also written in the current build after the build but before running the Pack task. This breaks every-other build (as we were actively using the cached copy here if it didn't change).

Fixes #121068

MSBuild cleans up intermediate file writes from previous builds that weren't also written in the current build after the build but before running the Pack task. This breaks every-other build (as we were actively using the cached copy here if it didn't change).

Fixes #121068
Copy link
Contributor

Copilot AI left a comment

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 fixes an issue where MSBuild's cleanup of intermediate files was breaking incremental builds. The problem occurred because a generated template file was being tracked as a file write, causing MSBuild to delete it between builds when it wasn't regenerated, breaking every-other-build scenarios.

Key Changes:

  • Removed the output tracking of the generated IL targets file to prevent MSBuild from cleaning it up between builds

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 24, 2025
@jkoritzinsky jkoritzinsky enabled auto-merge (squash) October 24, 2025 21:52
@jkoritzinsky jkoritzinsky merged commit 67379fd into main Oct 24, 2025
97 checks passed
@ViktorHofer ViktorHofer deleted the jkoritzinsky-patch-1 branch October 27, 2025 14:29
@github-actions github-actions bot locked and limited conversation to collaborators Nov 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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.

Every other build of clr component fails due to Microsoft.NET.Sdk.IL.targets not found

2 participants