-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
New package: Nvidia.ProfileInspector version 2.4.0.4 #162241
New package: Nvidia.ProfileInspector version 2.4.0.4 #162241
Conversation
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi @devCKVargas, This package installed normally, but gave this error when launched: Many of nVidia's applications require their hardware to operate, but the error message is usually much more clear. It seems more like a missing dependency or an installer error. What do you think? |
# Created with komac v2.3.0 | ||
# yaml-language-server: $schema=https://aka.ms/winget-manifest.installer.1.6.0.schema.json | ||
|
||
PackageIdentifier: Nvidia.ProfileInspector |
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.
PackageIdentifier: Nvidia.ProfileInspector | |
PackageIdentifier: Orbmu2k.NvidiaProfileInspector |
Best practice for package identifiers is Publisher.PackageName
to avoid confusion. After updating this, you will need to also update the folder and name of the manifest files
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.
Thanks, I'll keep that in mind and follow it up in my future PRs.
My hands are too full right now, but I'll get back to it when I'm a little freer.
# Created with komac v2.3.0 | ||
# yaml-language-server: $schema=https://aka.ms/winget-manifest.defaultLocale.1.6.0.schema.json | ||
|
||
PackageIdentifier: Nvidia.ProfileInspector |
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.
PackageIdentifier: Nvidia.ProfileInspector | |
PackageIdentifier: Orbmu2k.NvidiaProfileInspector |
# Created with komac v2.3.0 | ||
# yaml-language-server: $schema=https://aka.ms/winget-manifest.version.1.6.0.schema.json | ||
|
||
PackageIdentifier: Nvidia.ProfileInspector |
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.
PackageIdentifier: Nvidia.ProfileInspector | |
PackageIdentifier: Orbmu2k.NvidiaProfileInspector |
PackageName: Nvidia Profile Inspector | ||
License: MIT | ||
LicenseUrl: https://github.com/Orbmu2k/nvidiaProfileInspector/blob/master/LICENSE | ||
Copyright: Copyright (c) 2016 Orbmu2k |
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.
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.
Hmm, not quite sure how to resolve this since the LICENSE from the repo hasn't been modified for more than 8 years now.
Pull request was closed
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.
Good stuff, thank you.
Pull request has been created with komac v2.3.0 🚀
Microsoft Reviewers: Open in CodeFlow