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

[irrikrlla]

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]

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

git diff --cached
git --version

[irrikrlla]

Here is the result~
git version is the latest, git diff --cached command gives nothing

[AlexanderDash]

+1

[AlexanderDash]

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]

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]

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]

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]

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.