Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: revise method for detecting screeps install #17

Merged
merged 10 commits into from
Jun 23, 2024

Conversation

admon84
Copy link
Member

@admon84 admon84 commented Jun 23, 2024

  • Removed steam-game-path — its using registry-js and requires Python and C++ build tools on Windows (https://github.com/desktop/registry-js/blob/master/docs/index.md). If those are not installed node-gyp throws errors.

  • Revised method for finding the Screeps installation: getScreepsPath()

    • For MacOS, it checks for 'Steam' in the default location within the user's home directory.

    • For Linux, it checks for 'Steam' in several common locations within the user's home directory. It also has basic support for the Windows Subsystem for Linux (WSL), where it checks for 'Steam' in the default location on the Windows file system.

    • For Windows, it first tries to get the 'Steam' path from the Windows Registry using the lightweight winreg. If this fails, it falls back to checking for 'Steam' in the default location within the 'Program Files (x86)' directory.

  • Added rollup (was having issues with tsc importing from local files)

  • More readme updates: View

@CarsonBurke CarsonBurke merged commit 92259c3 into screepers:main Jun 23, 2024
3 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.

2 participants