Skip to content

New package: StreetPea.chiaki-ng version 1.9.1#190343

Merged
microsoft-github-policy-service[bot] merged 4 commits intomicrosoft:masterfrom
streetpea:StreetPea.chiaki-ng-1.9.1-4a3b95aa-27a7-4708-bb43-0b54a74b30bd
Dec 11, 2024
Merged

New package: StreetPea.chiaki-ng version 1.9.1#190343
microsoft-github-policy-service[bot] merged 4 commits intomicrosoft:masterfrom
streetpea:StreetPea.chiaki-ng-1.9.1-4a3b95aa-27a7-4708-bb43-0b54a74b30bd

Conversation

@streetpea
Copy link
Copy Markdown
Contributor

@streetpea streetpea commented Nov 11, 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.6 schema?

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


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Copy Markdown
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@streetpea
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@wingetbot wingetbot added the Validation-Unattended-Failed During installation testing the installer was blocked on user input and no switches were provided. label Nov 11, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Nov 11, 2024
@wingetbot
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Author-Feedback This needs a response from the author. Validation-Unattended-Failed During installation testing the installer was blocked on user input and no switches were provided. labels Nov 11, 2024
@wingetbot wingetbot added New-Package Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels Nov 11, 2024
Copy link
Copy Markdown
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.

Manual Validation ended with:
image

@stephengillie stephengillie added the Needs-Author-Feedback This needs a response from the author. label Nov 11, 2024
@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 Nov 11, 2024
auto-merge was automatically disabled November 11, 2024 18:56

Head branch was pushed to by a user without write access

@wingetbot
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@streetpea
Copy link
Copy Markdown
Contributor Author

streetpea commented Nov 12, 2024

@stephengillie it seems that that dependency is installed with graphics drivers which is probably why I haven't heard anybody complain about it. I added it. Thank you for your patience.

@wingetbot
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@wingetbot wingetbot added New-Package Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels Nov 12, 2024
@stephengillie
Copy link
Copy Markdown
Collaborator

stephengillie commented Nov 13, 2024

Manual Validation ended with:

Faulting application name: chiaki.exe, version: 1.9.1.0, time stamp: 0x6731a7cb
Faulting module name: chiaki.exe, version: 1.9.1.0, time stamp: 0x6731a7cb
Exception code: 0xc0000005
Fault offset: 0x000000000005db0e
Faulting process id: 0x25bc
Faulting application start time: 0x01db3561e649b654
Faulting application path: C:\Program Files\chiaki-ng\chiaki.exe
Faulting module path: C:\Program Files\chiaki-ng\chiaki.exe
Report Id: b294148e-305f-420c-a82f-559afcc7e905
Faulting package full name:
Faulting package-relative application ID:

Fault bucket 1571553288640585701, type 4
Event Name: APPCRASH
Response: Not available
Cab Id: 0

Problem signature:
P1: chiaki.exe
P2: 1.9.1.0
P3: 6731a7cb
P4: chiaki.exe
P5: 1.9.1.0
P6: 6731a7cb
P7: c0000005
P8: 000000000005db0e
P9:
P10:

Attached files:
\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WERFA4D.tmp.dmp
\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WERFB19.tmp.WERInternalMetadata.xml
\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WERFB2A.tmp.xml
\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WERFB3F.tmp.csv
\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WERFB50.tmp.txt

These files may be available here:
\?\C:\ProgramData\Microsoft\Windows\WER\ReportArchive\AppCrash_chiaki.exe_441459e35264f7e28f20536c483284c5ba995bd8_0e021de8_6a93495e-5199-434c-9724-e5b7ccdbbeeb

Analysis symbol:
Rechecking for solution: 0
Report Id: b294148e-305f-420c-a82f-559afcc7e905
Report Status: 268435456
Hashed bucket: c0989a27d960fa0c35cf4763bb04dfe5
Cab Guid: 0

Edit: formatting.

@stephengillie stephengillie added the Needs-Author-Feedback This needs a response from the author. label Nov 13, 2024
@stephengillie
Copy link
Copy Markdown
Collaborator

@ stephengillie it seems that that dependency is installed with graphics drivers which is probably why I haven't heard anybody complain about it. I added it. Thank you for your patience.

It's my pleasure to help. The dependency might be missing from the VMs as they have inferior graphics drivers - if it's not necessary in most PCs then it should probably be removed.

Manual Validation ended with the errors above being logged in the Event Log. The application window didn't open - I don't have more info about why, but if you have any troubleshooting suggestions, I'd be happy to try them.

@streetpea
Copy link
Copy Markdown
Contributor Author

@stephengillie is it possible to run debugview https://learn.microsoft.com/en-us/sysinternals/downloads/debugview on the system and see if any logs pop up in there?

@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 Nov 13, 2024
@streetpea
Copy link
Copy Markdown
Contributor Author

streetpea commented Nov 14, 2024

@stephengillie according to Khronos who makes Vulkan, you shouldn’t include Vulkan-1.dll with your application and should install allow the drivers to update it or users to manually update it if their drivers are too old. https://github.com/KhronosGroup/Vulkan-Loader/blob/main/docs/LoaderApplicationInterface.md#bundling-the-loader-with-an-application so I should probably take that added dependency out.

@stephengillie
Copy link
Copy Markdown
Collaborator

Apologies for the delay in responding - I'm dealing with capacity issues. Yes, feel free to remove the dependency. I am unsure about getting debugview onto the VM - does it have a PackageIdentifier in the repo?

@streetpea
Copy link
Copy Markdown
Contributor Author

streetpea commented Nov 15, 2024

@stephengillie you can download the exe for debugview directly from sysinternals using this link https://live.sysinternals.com/Dbgview.exe with curl or wget. A zip file with all debug view files is here: https://download.sysinternals.com/files/DebugView.zip

it’s also available via chocolatey via

choco install dbgview

https://community.chocolatey.org/packages/dbgview

There are winget packages for many of the sysinternals under manifests/m/Microsoft/SysInternals/ but debug view doesn’t appear to be one of them

@streetpea
Copy link
Copy Markdown
Contributor Author

@stephengillie did you ever get a chance to try out debugview?

@stephengillie
Copy link
Copy Markdown
Collaborator

Unfortunately, it didn't find any logs on the VM.
image

@streetpea
Copy link
Copy Markdown
Contributor Author

streetpea commented Dec 10, 2024

Hmm. Maybe it needs administrative access to run based on that popup. If you start debug view first and then leave it open when launching chiaki-ng various logs should appear in debug view but I never got that popup window so it's possible it doesn't have the necessary access on your system.

@streetpea
Copy link
Copy Markdown
Contributor Author

@stephengillie did you open DebugView first and then leave it open when launching chiaki-ng?

@stephengillie
Copy link
Copy Markdown
Collaborator

I haven't used DebugView before, sorry. This seems to be a problem with the VM driver:
image

@stephengillie stephengillie removed the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 11, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot merged commit 81958b6 into microsoft:master Dec 11, 2024
@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 11, 2024
PckgrTom added a commit to PckgrTom/winget-pkgs that referenced this pull request Dec 11, 2024
streetpea added a commit to streetpea/winget-pkgs that referenced this pull request Dec 14, 2024
Notable changes:
- take out vulkan RT dependency as it should be installed at the correct version by graphics drivers (see microsoft#190343 (comment))
- update docs url to correct link
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 Validation-Completed Validation passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants