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

logitech-g-hub 2024.9.649333 #195559

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

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-cask-pr.

@BrewTestBot BrewTestBot added the bump-cask-pr PR was created using `brew bump-cask-pr` label Dec 17, 2024
@krehel krehel self-assigned this Dec 17, 2024
@krehel
Copy link
Member

krehel commented Dec 17, 2024

I'll fix this in a couple hours unless someone else gets there first

@krehel krehel removed their assignment Dec 17, 2024
@bevanjkay
Copy link
Member

There's some odd things that the installer does for this app, after it finishes installing, it continues working in the background and renames some things etc...
This can cause the uninstall to fail if it is run too quickly after the install.

@krehel
Copy link
Member

krehel commented Dec 18, 2024

Yeah, I had it stuck in an infinite loop within my VM.

I think the uninstaller may have issues on ARM, it is only an x64 build. I have seen on other applications where x64 uninstallers on ARM have failures, there was a stacktrace. I am going to try ripping it out and just doing it manually.

@krehel
Copy link
Member

krehel commented Dec 18, 2024

Does this even install correctly? It fails CI and I can't get it to behave properly in my VM either. I suspect if it was broken more individuals would file issues, but I am thinking if installer: manual is a way out of this one.

@khipp
Copy link
Member

khipp commented Dec 18, 2024

Related to #194572

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-cask-pr PR was created using `brew bump-cask-pr`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants