Skip to content

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

Draft
kristopolous with Copilot wants to merge 8 commits into
mainfrom
copilot/add-prebuilt-ursh-binaries
Draft

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

Conversation

CopilotAI commented Mar 24, 2026

Copy link
Copy Markdown

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.

CopilotAIand 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
CopilotAI changed the title [WIP] Add prebuilt ursh binaries via GitHub ReleasesShip prebuilt ursh binaries via GitHub Releases; replace deprecated bash installerMar 24, 2026
CopilotAI requested a review from kristopolousMarch 24, 2026 01:59
Copilot stopped work on behalf of kristopolous due to an error March 24, 2026 02:26
CopilotAI changed the title Ship prebuilt ursh binaries via GitHub Releases; replace deprecated bash installerSimplify install URL to curl -sSL day50.dev/ursh | bashMar 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
CopilotAI changed the title Simplify install URL to curl -sSL day50.dev/ursh | bashSimplify install to curl -sSL day50.dev/ursh | bashMar 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
CopilotAI changed the title Simplify install to curl -sSL day50.dev/ursh | bashSplit install into bootstrap + install; serve via day50.dev/urshMar 24, 2026
CopilotAI changed the title Split install into bootstrap + install; serve via day50.dev/urshRename 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
CopilotAI changed the title Rename install scripts: bootstrap.sh (stable) + install.sh (full installer)Remove unintended README changes from PRMar 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
CopilotAI changed the title Remove unintended README changes from PRSimplify install.sh checksum verification to md5 only; revert READMEMar 24, 2026
CopilotAI changed the title Simplify install.sh checksum verification to md5 only; revert READMESimplify install.sh: md5 checksum + python3 site-based install dirMar 24, 2026
Sign up for freeto 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

@kristopolous