mirror of
https://github.com/Comfy-Org/ComfyUI-Manager.git
synced 2026-11-05 07:47:25 +00:00
[GH-ISSUE #921] Cmd('git') failed due to: exit code(129) when updating node #400
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
Comfy-Org/ComfyUI-Manager#400
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @irrikrlla on GitHub (Aug 1, 2024).
Original GitHub issue: https://github.com/Comfy-Org/ComfyUI-Manager/issues/921
I tried to update a node, with the newest ComfyUImanager, but failed with the e Cmd('git') failed due to: exit code(129). I also tried to fix by modifying the code, but had no idea where is the bug. Is there any idea to solve this? It's the first time I meet the issue with manager.
@ltdrdata commented on GitHub (Aug 1, 2024):
Please show the results of executing the following commands in cmd:
@irrikrlla commented on GitHub (Aug 2, 2024):
Here is the result~
git version is the latest, git diff --cached command gives nothing
@cheezmil commented on GitHub (Sep 23, 2024):
+1

@cheezmil commented on GitHub (Sep 23, 2024):
After entering this command, I can click on "try updata" in Comfyui. It's really strange. Do I have to enter this command every time I update? It's too troublesome. What should I do.
@ltdrdata commented on GitHub (Sep 23, 2024):
That directory isn't on a network drive, is it?
@cheezmil commented on GitHub (Sep 23, 2024):
“That directory isn't on a network drive, is it?” No, it may not have anything to do with this.
I found that running cmd as an administrator can solve this problem, but I don't know if it's safe to do so..
@ltdrdata commented on GitHub (Sep 23, 2024):
What about permission?
@cheezmil commented on GitHub (Sep 24, 2024):
I have found a solution。
https://www.zhihu.com/tardis/bd/art/619858339
Running Comfyui's cmd as an administrator can also solve the problem.
In addition, there is also an issue with ComfyUI Manager. Even though it reports an error with exit code (129), it still shows that the update was successful. The exception should be caught instead of informing the user that the update was successful.
@n0valis commented on GitHub (Nov 24, 2024):
Yes can confirm. ComfyUI does not capture the fail instead claims success.
How is that a solutution? That's bloody chinese
@ltdrdata commented on GitHub (Nov 25, 2024):
In Explorer, check the properties of the directory in the permissions section to verify if the owner is set to your account.