Skip to content

feat: add download/extraction progress to install.sh, expand distro coverage in docs#4869

Merged
proggeramlug merged 3 commits into
PerryTS:mainfrom
nglmercer:install-progress
Jun 10, 2026
Merged

feat: add download/extraction progress to install.sh, expand distro coverage in docs#4869
proggeramlug merged 3 commits into
PerryTS:mainfrom
nglmercer:install-progress

Conversation

@nglmercer

Copy link
Copy Markdown
Contributor

Summary

Two improvements to Perry's Linux installation story:

1. Progress indicators in packaging/install.sh

  • Download: Uses curl --progress-bar on interactive terminals (real-time #### 45% 12 MB/s 00:05)
  • Extraction: Uses pv if installed for pipe-through progress; silent fallback
  • Elapsed time: Shows Done in Xs (size) after download
  • CI-safe: Detects non-TTY (piped) context and falls back to -fsS (silent, no messy bars)
  • Changed set -eset -eu for better error handling

2. Per-distro C toolchain instructions in docs

Added Linux linker install commands for Arch/Cachy OS/Manjaro (pacman -S base-devel gcc), Fedora/RHEL (dnf), openSUSE (zypper), Alpine (apk), and Void Linux (xbps-install) to both:

  • README.md Requirements section
  • docs/src/getting-started/installation.md Prerequisites section

Previously only Debian/Ubuntu (apt) and Alpine were documented.

@nglmercer

nglmercer commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author
imagen imagen

I recommend changing the installation instructions.

proggeramlug and others added 2 commits June 10, 2026 05:09
…ents

Without -f, an HTTP error (transient 5xx, proxy interception) writes the
error body into perry.tar.gz and exits 0, surfacing later as a confusing
tar/gzip failure. Also restores the comments documenting why the script
probes recent tags instead of releases/latest, and the multi-hop
%{http_code} parsing.
@proggeramlug proggeramlug merged commit ecc7ec6 into PerryTS:main Jun 10, 2026
1 check passed
@proggeramlug

Copy link
Copy Markdown
Contributor

Thanks @nglmercer - really appreciate the PR.

I added a few minor things and merged :)

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