Skip to content

Conversation

@antkmsft
Copy link
Member

@antkmsft
Copy link
Member Author

antkmsft commented Oct 15, 2025

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:[email protected]...
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
Contributor

dg0yt commented Oct 15, 2025

New vcpkg tool bug.

@BillyONeal
Copy link
Member

microsoft/vcpkg-tool#1817 bug fix

@BillyONeal 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
BillyONeal previously approved these changes Oct 16, 2025
@BillyONeal 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
Member Author

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

@antkmsft antkmsft marked this pull request as draft October 16, 2025 17:11
@BillyONeal 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
@BillyONeal BillyONeal closed this Oct 16, 2025
@BillyONeal BillyONeal reopened this Oct 16, 2025
@antkmsft
Copy link
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.

@antkmsft antkmsft marked this pull request as ready for review October 17, 2025 00:23
@antkmsft antkmsft requested a review from BillyONeal October 17, 2025 00:28
@BillyONeal BillyONeal enabled auto-merge (squash) October 17, 2025 17:55
@BillyONeal 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