-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
mtkennerly.madamiru version 0.1.0 #198673
mtkennerly.madamiru version 0.1.0 #198673
Conversation
/AzurePipelines run |
Hmm, the errors don't sound like anything related to my changes specifically:
I was expecting a runtime error if GStreamer wasn't available on the test system (missing DLL), but not sure what this is. |
Manual Validation ended with:
(Automated response - build 904.) |
@stephengillie Could you please advise how I can resolve the security check? For what it's worth, Madamiru does get a clean report on VirusTotal: https://www.virustotal.com/gui/url/48658a8c56573aa8aee2bb98fdf26ab95d5f31c01608d7d75b6f1839cd9f247a |
Yes, Madamiru depends on GStreamer ( https://gstreamer.freedesktop.org ), which the user is expected to install separately. I did specify the dependency in the manifest, but I think that's just for documentation purposes and not automatically installed, right? Is there any way to set up the test environment for this or bypass it? Dependencies:
PackageDependencies:
- PackageIdentifier: gstreamerproject.gstreamer |
The dependency is automatically checked, and installed if missing. The process of manually checking that field and installing was automated around I believe, the 1.5 release. The dependency seems to depend on So I added it as another dependency on this manifest, and it works now:
But there was no change in overall outcome. It seems like the dependency isn't being found by the main application in my testing VMs. This might be unique to the VMs, so if |
It looks like the GStreamer installer doesn't automatically add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
manifests/m/mtkennerly/madamiru/0.1.0/mtkennerly.madamiru.installer.yaml
Outdated
Show resolved
Hide resolved
Head branch was pushed to by a user without write access
/AzurePipelines run |
8306446
into
microsoft:master
Thanks for your patience with this PR, and for helping me to validate it correctly. |
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>
?Microsoft Reviewers: Open in CodeFlow