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

Use git submodules for installation/management #928

Open
Xyem opened this issue Aug 4, 2024 · 3 comments
Open

Use git submodules for installation/management #928

Xyem opened this issue Aug 4, 2024 · 3 comments

Comments

@Xyem
Copy link

Xyem commented Aug 4, 2024

For my own ComfyUI installation, I add custom nodes as git submodules, as this allows me to manage custom nodes as part of how I manage ComfyUI (on a personal branch).

Would this installation method be considered for CUI-Manager? I have it installed as it helps with some other things, but would really like to use it for managing custom nodes as well.

@ltdrdata
Copy link
Owner

ltdrdata commented Aug 5, 2024

Actually, using git submodules is not recommended.
At one point, Impact Pack also used a git submodule structure, but I found that git submodules are not supported in several systems depending on the environment.
Also, when updating the submodule parts, problems frequently occurred.

I am currently considering how to handle additional installations using submodules or similar methods.

@Xyem
Copy link
Author

Xyem commented Aug 11, 2024

I'd certainly be interested in hearing about these issues you experienced as I haven't had any (though I am using them to manage an installation, not a codebase).

Perhaps I have overlooked something, but submodules are, to my knowledge, the only way to manage compatible versions between ComfyUI and the custom nodes. If you just check out the latest of either of them, they could be incompatible.

@ltdrdata
Copy link
Owner

I'd certainly be interested in hearing about these issues you experienced as I haven't had any (though I am using them to manage an installation, not a codebase).

Perhaps I have overlooked something, but submodules are, to my knowledge, the only way to manage compatible versions between ComfyUI and the custom nodes. If you just check out the latest of either of them, they could be incompatible.

On systems with older Git installations, the submodule functionality doesn't work at all. Furthermore, when updating, there were frequent issues where changes that should have been applied to submodules weren't properly implemented.

If I make a mistake, I might need to reorganize the branch history. This often led to divergent branch issues. (This situation prevents me from delivering further updates, causing many problems.)

After encountering these issues several times, I reverted to using independent modules instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants