Bubble up child process errors in IndexCreationTool#6261
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| "Input": "%BUILD_SOURCESDIRECTORY%/src/AppInstallerCLIE2ETests/TestData/AppInstallerTestFont.ttf", | ||
| "HashToken": "<FONTHASH>" | ||
| "HashToken": "<FONTHASH>", | ||
| "SkipSignature": true |
There was a problem hiding this comment.
signtool can't sign fonts without a SIP and font-specific key formats. my change bubbled up the error and caused test failures, but I believe this is the only one. couldn't get e2e tests fully working locally though
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Signed-off-by: Tom Plant <tom.plant@devicie.com>
pl4nty
force-pushed
the
bubble-IndexCreationTool-errors
branch
from
c9951d1 to
2dc8196
Compare
Not supported by signtool without SIP etc
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
I've spent a while trying to reproduce these E2E test failures locally, but haven't been able to. Could they just be flaky? They seem stable in the ADO reports, but I was surprised that x86 failed when x64 succeeded. Maybe it's worth retrying after #6354 is merged? |
It appears to be the ADO image version; x86 ran with the newer one and x64 the older one. |
3 participants
📖 Description
IndexCreationTool can run several child processes. If any of them fails, index creation has partially failed and I expect an exception, but IndexCreationTool exits 0. This PR throws an exception with the exit code and command/args to assist with debugging. I also updated
makeappx unpackto use the sharedRunCommandfunction.I suspect IndexCreationTool isn't a stable/supported dependency, but thought this was worth submitting anyway.
🔗 References
Example SignTool error: https://github.com/pl4nty/winget-extras/actions/runs/26671649037/job/78615909376
🔍 Validation
I don't have a VS toolchain on hand, so I'd love if someone can run the ADO tests on this PR. Then I'll validate in my CI from the built IndexCreationTool.
✅ Checklist
📋 Issue Type
Microsoft Reviewers: Open in CodeFlow