search

ltdrdata / ComfyUI-Manager

ComfyUI-Manager is an extension designed to enhance the usability of ComfyUI. It offers management functions to install, remove, disable, and enable various custom nodes of ComfyUI. Furthermore, this extension provides a hub feature and convenience functions to access a wide range of information within ComfyUI.
GNU General Public License v3.0
6.81k stars 885 forks source link

Cmd('git') failed due to: exit code(129) when updating node #921

Open irrikrlla opened 3 months ago

irrikrlla commented 3 months ago

屏幕截图 2024-08-01 224219 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 3 months ago

Please show the results of executing the following commands in cmd:

git diff --cached
git --version
irrikrlla commented 3 months ago

屏幕截图 2024-08-02 083055 Here is the result~ git version is the latest, git diff --cached command gives nothing

AlexanderDash commented 1 month ago

+1 image

AlexanderDash commented 1 month ago

image 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 1 month ago

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.

That directory isn't on a network drive, is it?

AlexanderDash commented 1 month ago

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.

That directory isn't on a network drive, is it?

“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 1 month ago

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.

That directory isn't on a network drive, is it?

“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..

What about permission?

AlexanderDash commented 1 month ago

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.

That directory isn't on a network drive, is it?

“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..

What about permission?

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.