Skip to content

[azure-*-cpp] Remove cgmanifest.json during installation - #47812

Merged
Billy O'Neal (BillyONeal) merged 2 commits into
microsoft:masterfrom
antkmsft:cg
Oct 17, 2025
Merged

[azure-*-cpp] Remove cgmanifest.json during installation#47812
Billy O'Neal (BillyONeal) merged 2 commits into
microsoft:masterfrom
antkmsft:cg

Conversation

@antkmsft

Copy link
Copy Markdown
Member

@antkmsft

Anton Kolesnyk (antkmsft) commented Oct 15, 2025

Copy link
Copy Markdown
Member Author

I have no idea why CI is failing. Is your CI broken?
It is failing for our another PR with unrelated changes (#34835) but we're not changing anything that should cause this. I think it has started to fail around October 8th.
I was trying to repro the failure locally, but I can't.

Feature Test [2/21] azure-core-cpp[core,winhttp]:x64-windows-static
Restored 1 package(s) from HTTP servers in 175 ms. Use --debug to see more details.
Removing 1/3 azure-core-cpp:x64-windows-static
Elapsed time to handle azure-core-cpp:x64-windows-static: 9.31 ms
Installing 2/3 wil:x64-windows-static@1.0.250325.1...
no such file or directory
D:\a\_work\1\s\scripts\azure-pipelines\test-modified-ports.ps1 : vcpkg feature testing failed; this is usually a bug in one of the features in the port(s) edited in this pull request. Check for failure logs attached to the run in Azure Pipelines.
At D:\a\_work\_temp\azureclitaskscript1760491660013_inlinescript.ps1:11 char:1
+ & scripts/azure-pipelines/test-modified-ports.ps1 -Triplet x64-window …
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,test-modified-ports.ps1

##[error]Script failed with exit code: 1
C:\Windows\system32\cmd.exe /D /S /C ""C:\Program Files\Microsoft SDKs\Azure\CLI2\wbin\az.cmd" account clear"
Finishing: *** Test Modified Ports

@dg0yt

Copy link
Copy Markdown
Contributor

New vcpkg tool bug.

@BillyONeal

Copy link
Copy Markdown
Member

microsoft/vcpkg-tool#1817 bug fix

@BillyONeal Billy O'Neal (BillyONeal) added the requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof label Oct 16, 2025
@BillyONeal Billy O'Neal (BillyONeal) added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Oct 16, 2025
@antkmsft

Copy link
Copy Markdown
Member Author

Please merge #47829 first, and then I'll update this PR.

@antkmsft
Anton Kolesnyk (antkmsft) marked this pull request as draft October 16, 2025 17:11
@BillyONeal Billy O'Neal (BillyONeal) removed the requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof label Oct 16, 2025
@antkmsft

Copy link
Copy Markdown
Member Author

Ok, I've now decoupled this change from #47829 - that PR now includes the same changes, but only for Storage libraries, and this PR has everything else. So now, both this PR or that PR can get merged in any order (plus, storage will be using port-version 0 which always looks a tiny bit nicer).

This PR can now be merged without waiting on any other PR.

@BillyONeal
Billy O'Neal (BillyONeal) merged commit 071f676 into microsoft:master Oct 17, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants