Skip to content

Simplify install.sh: md5 checksum + python3 site-based install dir#1

Draft
Copilot wants to merge 8 commits intomainfrom
copilot/add-prebuilt-ursh-binaries
Draft

Simplify install.sh: md5 checksum + python3 site-based install dir#1
Copilot wants to merge 8 commits intomainfrom
copilot/add-prebuilt-ursh-binaries

Conversation

Copy link

Copilot AI commented Mar 24, 2026

Two over-engineered functions in install.sh replaced with simpler, more reliable alternatives.

Changes

  • verify_checksum — replaced sha256sumshasum -a 256 fallback chain with a straight md5sum / md5 -q call, consistent with crypto/md5 already used in main.go
  • detect_install_dir — replaced a 9-line heuristic probing ~/.local/bin / /usr/local/bin with the canonical single-liner:
    INSTALL_DIR="$(python3 -msite --user-base)/bin"
    This is OS/distro-agnostic and correct on both Linux and macOS without any directory-existence checks.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits March 24, 2026 01:52
Co-authored-by: kristopolous <231761+kristopolous@users.noreply.github.com>
Agent-Logs-Url: https://github.com/day50-dev/ursh/sessions/ea5ecfa4-d217-4bd2-a7a6-164b80c2302f
Copilot AI changed the title [WIP] Add prebuilt ursh binaries via GitHub Releases Ship prebuilt ursh binaries via GitHub Releases; replace deprecated bash installer Mar 24, 2026
Copilot AI requested a review from kristopolous March 24, 2026 01:59
Copilot stopped work on behalf of kristopolous due to an error March 24, 2026 02:26
Copilot AI changed the title Ship prebuilt ursh binaries via GitHub Releases; replace deprecated bash installer Simplify install URL to curl -sSL day50.dev/ursh | bash Mar 24, 2026
…ler from GitHub

Co-authored-by: kristopolous <231761+kristopolous@users.noreply.github.com>
Agent-Logs-Url: https://github.com/day50-dev/ursh/sessions/107ebc3c-6773-4876-bea7-a6e860aeeb09
Copilot AI changed the title Simplify install URL to curl -sSL day50.dev/ursh | bash Simplify install to curl -sSL day50.dev/ursh | bash Mar 24, 2026
…sh (real logic)

Co-authored-by: kristopolous <231761+kristopolous@users.noreply.github.com>
Agent-Logs-Url: https://github.com/day50-dev/ursh/sessions/9bb1f07b-38ce-45d7-b3b2-18bec9dd40a3
Copilot AI changed the title Simplify install to curl -sSL day50.dev/ursh | bash Split install into bootstrap + install; serve via day50.dev/ursh Mar 24, 2026
Copilot AI changed the title Split install into bootstrap + install; serve via day50.dev/ursh Rename install scripts: bootstrap.sh (stable) + install.sh (full installer) Mar 24, 2026
Co-authored-by: kristopolous <231761+kristopolous@users.noreply.github.com>
Agent-Logs-Url: https://github.com/day50-dev/ursh/sessions/f288df07-59f4-4fb9-bb0c-e2c76291a452
Copilot AI changed the title Rename install scripts: bootstrap.sh (stable) + install.sh (full installer) Remove unintended README changes from PR Mar 24, 2026
Co-authored-by: kristopolous <231761+kristopolous@users.noreply.github.com>
Agent-Logs-Url: https://github.com/day50-dev/ursh/sessions/bd31f29b-17c3-4131-8a2d-f1c0a609948a
Copilot AI changed the title Remove unintended README changes from PR Simplify install.sh checksum verification to md5 only; revert README Mar 24, 2026
Copilot AI changed the title Simplify install.sh checksum verification to md5 only; revert README Simplify install.sh: md5 checksum + python3 site-based install dir Mar 24, 2026
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