Skip to content
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

Conversation

mtkennerly
Copy link
Contributor

@mtkennerly mtkennerly commented Dec 13, 2024

Checklist for Pull Requests

Manifests

  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • This PR only modifies one (1) manifest
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>?
  • Does your manifest conform to the 1.9 schema?
Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@mtkennerly
Copy link
Contributor Author

Hmm, the errors don't sound like anything related to my changes specifically:

##[error]1ES PT Error: Job 'ContentValidation' uses the release tasks ["AzureFunction@1","AzureFunction@1","AzureFunction@1"] but is not marked as a release job. Release tasks will soon be disallowed outside of release jobs. See [https://aka.***/1espt/custom-release-job](https://aka.%2A%2A%2A/1espt/custom-release-job)

I was expecting a runtime error if GStreamer wasn't available on the test system (missing DLL), but not sure what this is.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Dec 13, 2024
@stephengillie
Copy link
Collaborator

Manual Validation ended with:

2024-12-13 12:57:27.461 [FAIL] Installer failed security check. Url: https://github.com/mtkennerly/madamiru/releases/download/v0.1.0/madamiru-v0.1.0-win64.zip Result: 0x80004005 Property(S): ErrorDialog = ErrorDlg MSI (s) (B8:E8) [12:57:16:077]: Windows Installer installed the product. Product Name: GStreamer 1.0. Product Version: 1.24.10. Product Language: 1033. Manufacturer: GStreamer Project. Installation success or error status: 0.

(Automated response - build 904.)

@mtkennerly
Copy link
Contributor Author

@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

@stephengillie stephengillie added Validation-Completed Validation passed and removed Validation-Installation-Error Error installing application Validation-Executable-Error Needs-Attention This work item needs to be reviewed by a member of the core team. labels Dec 16, 2024
@stephengillie
Copy link
Collaborator

The security check didn't recur.

Is a dependency missing?
image

@stephengillie stephengillie added the Needs-Author-Feedback This needs a response from the author. label Dec 16, 2024
@mtkennerly
Copy link
Contributor Author

mtkennerly commented Dec 16, 2024

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

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Dec 16, 2024
@stephengillie
Copy link
Collaborator

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 VCRedist:
image

So I added it as another dependency on this manifest, and it works now:
image

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
this isn't happening on other systems, then I'd be happy to merge it.

Verify while here:
image

@stephengillie stephengillie added Needs-Author-Feedback This needs a response from the author. and removed Needs-Attention This work item needs to be reviewed by a member of the core team. labels Dec 17, 2024
@mtkennerly
Copy link
Contributor Author

It looks like the GStreamer installer doesn't automatically add C:\gstreamer\1.0\msvc_x86_64\bin to the PATH environment variable. If you add that, then it should find the DLLs.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Dec 18, 2024
Copy link
Collaborator

@stephengillie stephengillie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for helping me to fix that issue. Now, it needs VCRedist:
image

@stephengillie stephengillie added Needs-Author-Feedback This needs a response from the author. and removed Validation-Completed Validation passed Needs-Attention This work item needs to be reviewed by a member of the core team. labels Dec 19, 2024
auto-merge was automatically disabled December 20, 2024 04:46

Head branch was pushed to by a user without write access

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Author-Feedback This needs a response from the author. Changes-Requested Changes Requested labels Dec 20, 2024
@wingetbot wingetbot added New-Package Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels Dec 20, 2024
@stephengillie
Copy link
Collaborator

Verify:
image
(With the workaround described above of manually adding gstreamer to the path.)

@microsoft-github-policy-service microsoft-github-policy-service bot merged commit 8306446 into microsoft:master Dec 20, 2024
2 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Dec 20, 2024
@stephengillie
Copy link
Collaborator

@mtkennerly,

Thanks for your patience with this PR, and for helping me to validate it correctly.

@mtkennerly mtkennerly deleted the mtkennerly.madamiru-0.1.0 branch December 20, 2024 19:57
@wingetbot
Copy link
Collaborator

Publish pipeline succeeded for this Pull Request. Once you refresh your index, this change should be present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Moderator-Approved One of the Moderators has reviewed and approved this PR New-Package Publish-Pipeline-Succeeded Validation-Completed Validation passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants