Skip to content

Builder: a game folder it can't write to stops at once with the reason - #1849

Merged
NotRexed merged 2 commits into
mainfrom
fix/builder-unwritable-folder
Nov 10, 2026
Merged

NotRexed merged 2 commits into
mainfrom
fix/builder-unwritable-folder

Conversation

@NotRexed

@NotRexed NotRexed commented Nov 10, 2026 •

Copy link
Copy Markdown
Owner

Fixes #1847.

A player's v2.0.14 build (game folder under C:\Program Files (x86)\...) stopped minutes in, inside nup.py, with a bare PermissionError: [WinError 5] Access denied: '...\.game'.

make_game.py now writes a small check file into the game folder right after creating it, before any reading. If that is refused, it stops with:

The game folder C:\Program Files (x86)... can't be written to (Access is denied). Windows protects folders under Program Files and Windows, and only lets administrators write there. In Hookshot Builder, choose a game folder of your own, for example C:\Games\Hookshot, and build again.

The Program Files sentence appears only when the folder is under Program Files or Windows.

Tests:

  • New game-free test tests/test_make_game_writable.py covers three cases:
    • a writable folder passes and is left empty;
    • a refused folder names the folder and the reason;
    • Program Files is named as the reason when it applies.
  • The other make_game tests pass.
  • Checked against a real C:\Program Files (x86) folder from a non-admin shell: it gives the message above.

🤖 Generated with Claude Code

NotRexed and others added 2 commits November 10, 2026 13:19
A game folder under Program Files stopped the build minutes in, inside nup.py, with a bare "PermissionError: [WinError 5]"
on its .game folder (a player on Discord, v2.0.14). make_game now writes a small check file into the folder before anything
is read; refused, it names the folder, says Program Files needs administrator rights when the folder is under it, and asks
for a folder of the player's own.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@NotRexed
NotRexed merged commit 75052a3 into main Nov 10, 2026
11 checks passed
@NotRexed
NotRexed deleted the fix/builder-unwritable-folder branch November 10, 2026 19:48
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.

Builder: clear message when the game folder isn't writable, e.g. Program Files

1 participant