Skip to content

fix Component Governance file can't find warning #5510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

qiutongMS
Copy link
Contributor

This PR fixes Component Governance warnings that were occurring due to inconsistent capitalization of the configuration parameter across the build system. The warnings showed paths like:

##[warning]Project path C:\__w\1\s\eng\PackageReference\IXP\IXP.TransportPackage.PackageReference.csproj specified by D:\a\_work\1\s\obj\release\x86\IXP.TransportPackage.PackageReference\project.assets.json does not exist.
##[warning]Project output path C:\__w\1\s\obj\Release\x86\IXP.TransportPackage.PackageReference\ specified by D:\a\_work\1\s\obj\release\x86\IXP.TransportPackage.PackageReference\project.assets.json does not exist.

Root Cause

The issue was caused by inconsistent configuration parameter casing:

  • BuildAll.ps1 defaults to "Release" (uppercase R)
  • Azure DevOps pipelines explicitly passed "release" (lowercase r)
  • build-nupkg.ps1 defaulted to "release" (lowercase r)

This caused project.assets.json files to reference different obj directory paths (obj\Release\ vs obj\release\), creating Component Governance warnings about non-existent paths.

@qiutongMS
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@qiutongMS qiutongMS force-pushed the user/qiutongshen/fixCGWarnings branch from 496e19d to 8582cc9 Compare June 16, 2025 07:58
@qiutongMS qiutongMS force-pushed the user/qiutongshen/fixCGWarnings branch from 404da72 to 907d52a Compare June 17, 2025 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant