fix(public/install.sh): fail early if req buildtools are missing #49
Conversation
Would fix some of the issues some people have been reporting in: openclaw/openclaw#5462 [AI assisted via Codex 5.2]
|
@knocte is attempting to deploy a commit to the Amantus Machina Team on Vercel. A member of the Team first needs to authorize it. |
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
| local skip_onboard=false | ||
|
|
||
| # Fail early if required build tools are missing (before any npm commands) | ||
| check_build_tools |
There was a problem hiding this comment.
check_build_tools() is called too early before Homebrew installation on macOS, and unconditionally for npm installs which don't need build tools
There was a problem hiding this comment.
check_build_tools() is called too early before Homebrew installation on macOS
Better to fail eariler than installing anything? AFAIU homebrew wouldn't install make or cmake
unconditionally for npm installs which don't need build tools
How would we know which npm installs wouldn't need make or cmake? AFAIU there is no way to know
There was a problem hiding this comment.
You make good points about early failure being preferable to cryptic npm errors, and you're right that npm installs might also need build tools depending on prebuilt binary availability.
However, there's still a practical issue with the current placement: On macOS systems without Xcode CLT, this check will fail and exit before install_homebrew runs. Since brew install make cmake can provide these tools, it would be better to let Homebrew be installed first.
The ideal flow would be:
- Install Homebrew (macOS only)
- Then check for build tools (with option to auto-install via brew on macOS)
- Then proceed with Node.js and openclaw installation
This way, users on macOS get a better experience - the installer can automatically provide the missing tools via Homebrew rather than just failing. Linux users still get the early clear error message.
The current approach works, but it's not as user-friendly as it could be on macOS.
1 participant
Would fix some of the issues some people have been reporting in: openclaw/openclaw#5462
[AI assisted via Codex 5.2]