Skip to content

bpf2c/nuget: Track script dependencies as CustomBuild inputs#5003

Open
mikeagun wants to merge 1 commit intomicrosoft:mainfrom
mikeagun:script-deps-incremental
Open

bpf2c/nuget: Track script dependencies as CustomBuild inputs#5003
mikeagun wants to merge 1 commit intomicrosoft:mainfrom
mikeagun:script-deps-incremental

Conversation

@mikeagun
Copy link
Contributor

@mikeagun mikeagun commented Feb 13, 2026

Description

Refactors bpf2c build steps to improve incremental build time for bpf2c and bpf programs.

Add AdditionalInputs to CustomBuild items in bpf2c.vcxproj and nuget.vcxproj so that changes to helper scripts and data files correctly trigger rebuilds:

  • bpf2c_dll.c / bpf2c_driver.c: track escape_text.ps1
  • bpf2c.exe.manifest.in: track Set-Version.ps1, Directory.Build.props, git_commit_id.h
  • ebpf-for-windows.nuspec.in: track Set-Version.ps1, Directory.Build.props, git_commit_id.h, ebpf-for-windows.props.in

Also fix two nuget incremental-build bugs:

  • Outputs mismatch: add missing $(Configuration) to match actual nupkg filename (fixes MSB8065 warning)
  • PreBuildEvent defeating incremental builds: remove del ebpf-for-windows.*.nupkg that unconditionally deleted the CustomBuild output every build

Part of #5024

Testing

Build only changes, and I locally validated that rebuilds happen (or don't) when files are touched.

Documentation

N/A

Installation

N/A

Add AdditionalInputs to CustomBuild items in bpf2c.vcxproj and nuget.vcxproj so that changes to helper scripts and
data files correctly trigger rebuilds:

 - bpf2c_dll.c / bpf2c_driver.c: track escape_text.ps1
 - bpf2c.exe.manifest.in: track Set-Version.ps1, Directory.Build.props, git_commit_id.h
 - ebpf-for-windows.nuspec.in: track Set-Version.ps1, Directory.Build.props, git_commit_id.h,
ebpf-for-windows.props.in

Also fix two nuget incremental-build bugs:

 - Outputs mismatch: add missing $(Configuration) to match actual nupkg filename (fixes MSB8065 warning)
 - PreBuildEvent defeating incremental builds: remove del ebpf-for-windows.*.nupkg that unconditionally deleted the
CustomBuild output every build
@mikeagun mikeagun enabled auto-merge February 13, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants