New version: Sigil-Ebook.PageEdit version 2.1.0#147897
New version: Sigil-Ebook.PageEdit version 2.1.0#147897stephengillie merged 3 commits intomicrosoft:masterfrom
Conversation
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Automatic Validation ended with:
(Automated response - build 876.) |
@denelon - This definitely seems to be a CLI issue and not a manifest issue |
|
This version of |
It's possible. Unless something changed in the CLI or pipelines recently though, I thought that the scope was omitted when searching for dependencies? I can very easily foresee many, many cases where a dependency is only available in machine scope but the requested package is available in both user or machine scope. I also know that many wix and burn packages have machine scope specified because the installers only ever write to HKLM even if it is a user scoped install. If the dependency flow takes a hard requirement when |
|
Scope should not matter in the installer verification pipeline with regard to dependencies in my opinion. The winget client can easily accommodate a machine scope dependency (in a user scope install) by prompting for admin elevation. This installer failure where dependencies are concerned is new behavior. I had the very same issue with the recent 2.1.0 Sigil-Ebook.Sigil update: #147284 My only recourse to get it merged was to remove the dependency from the user scope installer altogether. I don't find that to be a very pleasing workaround. |
|
A further note: the VCRedist.2015+.x64 dependency package in my manifests has never had a user scope installer available. And it never mattered to my manifests in the past. |
|
@wingetbot run |
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Automatic Validation ended with:
(Automated response - build 876.) |
|
Still pending investigation on the cli side |
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Automatic Validation ended with:
(Automated response - build 880.) |
|
Bump |
|
Publish pipeline succeeded for this Pull Request. Once you refresh your index, this change should be present. |
Checklist for Pull Requests
Manifests
winget validate --manifest <path>?winget install --manifest <path>?Note:
<path>is the name of the directory containing the manifest you're submitting.Microsoft Reviewers: Open in CodeFlow