Skip to content

TOOLS: Handle several issue related to running scripts on Windows#127

Merged
MotoLegacy merged 2 commits into
mainfrom
cypress_windows_fixes
Feb 15, 2026
Merged

TOOLS: Handle several issue related to running scripts on Windows#127
MotoLegacy merged 2 commits into
mainfrom
cypress_windows_fixes

Conversation

@MotoLegacy
Copy link
Copy Markdown
Member

@MotoLegacy MotoLegacy commented Feb 15, 2026

Description of Changes


  1. Windows does not allow * in paths on disk full-stop. This meant that checking out the assets repository on Windows would fail due to water textures using this pattern. The workaround employed is to use $ in place of the asterisk, then during build copy away to /tmp/ on the docker container and restore the asterisk where Windows will not see it and complain. Added a test case to check for these asterisks too.
  2. Map compilation will no longer fail because TrenchBroom uses Windows backslashes to refer back to WAD paths by default. The map will be overwritten on build to swap to forward slashes only in the wad key. Added a test case to check for these too. Fixes Toolbox doesn't accept \ in WAD Directory #125

Visual Sample


N/A (see builds)

Checklist


  • I have thoroughly tested my changes to the best of my ability
  • I confirm I have not contributed anything that would impact Nazi Zombies: Portable's licensing and usage
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible

@MotoLegacy MotoLegacy force-pushed the cypress_windows_fixes branch from 6c34a6d to 0d325ef Compare February 15, 2026 01:27
@MotoLegacy MotoLegacy force-pushed the cypress_windows_fixes branch from 0d325ef to 26dac33 Compare February 15, 2026 01:39
@MotoLegacy MotoLegacy changed the title WAD: Fix issues with Windows failing to check out water textures TOOLS: Handle several issue related to running scripts on Windows Feb 15, 2026
@MotoLegacy MotoLegacy marked this pull request as ready for review February 15, 2026 21:23
@MotoLegacy MotoLegacy merged commit 4709735 into main Feb 15, 2026
10 checks passed
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.

Toolbox doesn't accept \ in WAD Directory

1 participant