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

[Automated] Add w4po.ExplorerTabUtility version 2.2.0 #233771

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

w4po
Copy link
Contributor

@w4po w4po commented Mar 1, 2025

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?

Note: <path> is the directory's name containing the manifest you're submitting.


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Mar 1, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Mar 1, 2025
@w4po
Copy link
Contributor Author

w4po commented Mar 1, 2025

It works just fine, This is a standalone app (no installers);
Check for the system tray.

@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 Mar 1, 2025
@w4po
Copy link
Contributor Author

w4po commented Mar 1, 2025

Hello @stephengillie,

As always, the automatic validation fails.
Please let me know if it is something that I can fix for the feature PRs.
Or is it because it is a standalone app?

You're awesome. Thanks for the support. ♥️

PR 221627 PR 228779

@DuckDuckStudio
Copy link
Contributor

DuckDuckStudio commented Mar 1, 2025

Hello @w4po
Here's where the problem may be coming from:

/~https://github.com/w4po/winget-pkgs/blob/70a49640d59bf5bae334031b076aeeed0ff27160/manifests/w/w4po/ExplorerTabUtility/2.2.0/w4po.ExplorerTabUtility.installer.yaml#L11

image

expltab: The term 'expltab' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

The suggested fix is to remove this line.
WinGet only adds the directory to the Path, the filename is still the same.

@w4po
Copy link
Contributor Author

w4po commented Mar 1, 2025

Hello @DuckDuckStudio,

I was testing using an elevated shell, so I didn't face the issue before.
Turns out it is an issue in the CLI,

I don't know if it is good enough to remove the alias,
As the original name is too long to type ExplorerTabUtility

Thank you for your support ♥️

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Attention This work item needs to be reviewed by a member of the core team. Validation-Installation-Error Error installing application labels Mar 1, 2025
@wingetbot wingetbot added the Manifest-Validation-Error Manifest validation failed label Mar 1, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Mar 1, 2025
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Manifest-Validation-Error Manifest validation failed Needs-Author-Feedback This needs a response from the author. labels Mar 1, 2025
@DuckDuckStudio
Copy link
Contributor

I don't know if it is good enough to remove the alias, As the original name is too long to type ExplorerTabUtility

Personally I don't think this package should be opened by command, but that's still waiting for microsoft/winget-cli#2299 to implement it.

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Mar 1, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Mar 1, 2025
@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. Validation-Installation-Error Error installing application labels Mar 1, 2025
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Mar 1, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Mar 1, 2025
@w4po
Copy link
Contributor Author

w4po commented Mar 1, 2025

Personally I don't think this package should be opened by command, but that's still waiting for microsoft/winget-cli#2299 to implement it.

That would be much better ✨

Also, It seems PortableCommandAlias is not the issue, it must be something else.

@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 Mar 1, 2025
@stephengillie
Copy link
Collaborator

Automatic Validation ended with:

Signature Update Failed. Failure to parse win 10, attempting win 11 Installation failed with exit code -1978335216 2025-03-01 15:48:17.905 [CORE] Deployment operation #0: error 0x800700EA: Failed to load the extension DLL due to the following error: More data is available. 2025-03-01 15:48:17.905 [FAIL] C:__w\1\s\external\pkg\src\AppInstallerCommonCore\Deployment.cpp(54)\WindowsPackageManager.dll!61D9DF27: (caller: 61D9E606) Exception(1) tid(1e00) 800700EA More data is available. Msg:[Operation failed: error 0x800700EA: Failed to load the extension DLL due to the following error: More data is available. Msg:[C:__w\1\s\external\pkg\src\AppInstallerCommonCore\Deployment.cpp(54)\WindowsPackageManager.dll!61D9DF27: (caller: 61D9E606) Exception(1) tid(1e00) 800700EA More data is available. 2025-03-01 15:48:17.925 [REPO] Source add/update failed, waiting 6292 milliseconds and retrying: winget

Automated error analysis suggests -1978335216 may mean None of the installers are applicable for the current system

(Automated response - build 986.)

@w4po
Copy link
Contributor Author

w4po commented Mar 1, 2025

I'm puzzled by the automatic validation failure as this appears to be a recurring issue with ExplorerTabUtility packages that doesn't reflect any actual problems:

  1. The manifests are correctly structured and follow all Winget guidelines
  2. The .NET 9 dependency is properly specified and available in the Winget repository
  3. Previous versions of this package have been successfully merged after manual review despite similar automatic validation failures
  4. End users can install the package without any issues using the winget command

The error message about "Failed to load the extension DLL" with code 0x800700EA suggests this might be a limitation in the validation environment rather than an issue with the package itself.

Could this be a known issue with the validation pipeline for certain types of packages?

I appreciate your help in getting this merged despite the automatic validation issues. If there's anything I can do to help troubleshoot or resolve these recurring validation failures for future updates, please let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs-Attention This work item needs to be reviewed by a member of the core team. Validation-Installation-Error Error installing application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants