Skip to content

build.py: updated path normalization for .js files [windows]#613

Open
sfinktah wants to merge 1 commit intorgthree:mainfrom
sfinktah:sfink-simple-build-fix-windows
Open

build.py: updated path normalization for .js files [windows]#613
sfinktah wants to merge 1 commit intorgthree:mainfrom
sfinktah:sfink-simple-build-fix-windows

Conversation

@sfinktah
Copy link
Contributor

@sfinktah sfinktah commented Nov 8, 2025

This is the one-line fix for the issue with building on Windows. The one that created ../../../../../../../../app.js paths.

-   rel_path = file.replace(f'{DIR_WEB}/', "")
+   rel_path = os.path.relpath(file, DIR_WEB)

This avoids the excessive changes in #609.

The declarations in comfy/frontend do still need to be made exportable, but there is probably a more appropriate place to deal with that.

@sfinktah
Copy link
Contributor Author

@rgthree any change of expiditing this one? it's an absolute pain to have to patch manually for every PR.

... of course, you could probably do with less PRs. :)

r-vage added a commit to r-vage/rgthree-comfy that referenced this pull request Feb 9, 2026
- PR rgthree#613: Fix __build__.py path handling for cross-platform compatibility (use os.path.relpath)
- PR rgthree#465: Fix README.md typo (custom_modules → custom_nodes)
- PR rgthree#264: Add shift key support to keep LoRA menu open + fix callback return propagation
- PR rgthree#267: Add context menu option to fetch info for all LoRAs at once
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.

1 participant