Skip to content

Fix typo in directory name in provision script - #5

Closed
amgad-naiem wants to merge 2 commits into
vast-ai:mainfrom
amgad-naiem:patch-1
Closed

amgad-naiem wants to merge 2 commits into
vast-ai:mainfrom
amgad-naiem:patch-1

Conversation

@amgad-naiem

Copy link
Copy Markdown

No description provided.

@amgad-naiem amgad-naiem changed the title Fix type in directory name in provision script Fix typo in directory name in provision script Mar 13, 2025
@robballantyne

Copy link
Copy Markdown
Collaborator

Thank you — and apologies this sat so long. You were right, and the bug was still live on main until today: COMFYUI_DIR has no trailing slash, so ${COMFYUI_DIR}custom_nodes/ resolved to /workspace/ComfyUIcustom_nodes/, a sibling directory ComfyUI never scans. Every custom node a user listed was cloned somewhere it could not load from, and re-cloned on every provisioning run because the -d guard never matched.

Fixed in #236, which credits you as co-author and also covers hunyuan3d.sh (added after you opened this). Closing in favour of that — the fix is yours, the delay was ours.

robballantyne added a commit that referenced this pull request Aug 6, 2026
…e it (#236)

COMFYUI_DIR is ${WORKSPACE}/ComfyUI with no trailing slash, so
"${COMFYUI_DIR}custom_nodes/${dir}" resolved to /workspace/ComfyUIcustom_nodes/ —
a sibling directory ComfyUI never scans. Every custom node a user listed in NODES
was cloned somewhere it could not be loaded from, and because the [[ -d $path ]]
guard never matched, each provisioning run re-cloned the lot.

Remaining in default.sh, flux.sh and hunyuan3d.sh; the same line was already
correct in lora.sh, image-tools.sh, ltx-video.sh and text_to_video_wan.sh, which
is why it survived this long.

Reported independently in #5 (amgad-naiem) and #6 (nprabhakar) in March 2025.

Co-authored-by: amgad-naiem <amgad.naiem@gmail.com>
Co-authored-by: nprabhakar <nprabhakar@gmail.com>
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

Successfully merging this pull request may close these issues.

2 participants