Skip to content

fix(build): update Windows build.bat to match build.sh#111

Open
64johnlee wants to merge 26 commits into
RetroShare:masterfrom
64johnlee:ralph/RS-019
Open

fix(build): update Windows build.bat to match build.sh#111
64johnlee wants to merge 26 commits into
RetroShare:masterfrom
64johnlee:ralph/RS-019

Conversation

@64johnlee
Copy link
Copy Markdown

Summary

Update Windows build script to match the bash script functionality.

Changes

  • Added /y flag for copy commands to suppress overwrite prompts
  • Fixed file copy logic to properly copy index.html and styles.css
  • Updated app.js build to properly append JS modules (matching build.sh require.register pattern)
  • Fixed xcopy flags for assets folder (/s /e /i /y)
  • Changed variable expansion order for correct path handling
  • Clear and rebuild app.js at start (not after copy)

Files changed

  • webui-src/make-src/build.bat

64johnlee and others added 26 commits May 4, 2026 00:50
- Updated build.bat to properly copy files to destination folder
- Fixed /y flag for copy commands
- Added recursive globstar-style file processing matching build.sh
- Fixed app.js rebuild logic (was clearing after copy)
- Added /s /e /i /y flags for xcopy assets to properly mirror build.sh behavior

Fixes issue RetroShare#19 - Windows build script is outdated and does not build the webui correctly
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