Skip to content

fix: use POSIX character class in sed for macOS compatibility#2

Merged
codemedic merged 1 commit intomainfrom
fix/bsd-sed-compat
Mar 3, 2026
Merged

fix: use POSIX character class in sed for macOS compatibility#2
codemedic merged 1 commit intomainfrom
fix/bsd-sed-compat

Conversation

@oliver-cieliszak-natterbox
Copy link
Contributor

Summary

  • Replace GNU-only \s with [[:space:]] in two sed -E expressions in install.sh
  • Fixes bootstrap failure on macOS where BSD sed doesn't recognise \s, causing uv_version to resolve to empty and the download URL to 404

Test plan

  • Run install.sh on macOS (BSD sed) — uv downloads successfully
  • Run install.sh on Linux (GNU sed) — uv downloads successfully
  • Verify checksum extraction still works on both platforms

🤖 Generated with Claude Code

Replace GNU-only \s with [[:space:]] in sed expressions so that
uv_version extraction and checksum lookup work on BSD sed (macOS).
Without this, \s silently fails, uv_version resolves to empty, and
the download URL 404s.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@codemedic codemedic merged commit 129c1a0 into main Mar 3, 2026
4 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