Skip to content

fix(rokur): use deno binary release directly, not install-manual URL (closes #16)#20

Merged
hyperpolymath merged 1 commit into
mainfrom
fix-rokur-deno-install-url
May 3, 2026
Merged

fix(rokur): use deno binary release directly, not install-manual URL (closes #16)#20
hyperpolymath merged 1 commit into
mainfrom
fix-rokur-deno-install-url

Conversation

@hyperpolymath
Copy link
Copy Markdown
Owner

Summary

Replaces the broken deno.land/install-manual@vX.Y.Z.sh URL with the canonical GitHub-releases binary download. Closes #16.

Symptom

sh: syntax error: unexpected redirection
curl: (23) Failure writing output to destination

The install-manual@vX.Y.Z.sh URL pattern returns content that sh can't parse.

Change

Switch to GitHub-releases binary download. Same shape as the cerro-torre Alire fix — explicit-version URL is deterministic, no shell-script-piping involved.

Why this is safer

  • No shell injection surface: previous form piped curl output to sh
  • Deterministic: GitHub-releases URL pattern is stable across Deno versions
  • Smaller: just unzip a single binary, no install-script logic

Test plan

  • URL reachable (HTTP 302 → asset)
  • Asset is a single deno binary
  • Full podman build container-stack/rokur end-to-end (not run)

Cross-reference: idaptik commit hyperpolymath/idaptik@98f110ce.

Closes #16.

`https://deno.land/install-manual@v${DENO_VERSION}.sh` returns content
that sh cannot parse:

  sh: syntax error: unexpected redirection
  curl: (23) Failure writing output to destination, passed 16375 returned 0

Replaced with the explicit binary release URL pattern from
denoland/deno's GitHub releases. That path is the canonical,
deterministic download surface — no shell parsing involved, no
install-script redirection, and the URL pattern is stable across
Deno releases.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 2, 2026

@hyperpolymath hyperpolymath merged commit f845173 into main May 3, 2026
21 of 25 checks passed
@hyperpolymath hyperpolymath deleted the fix-rokur-deno-install-url branch May 3, 2026 08:56
hyperpolymath added a commit that referenced this pull request May 3, 2026
hyperpolymath added a commit that referenced this pull request May 3, 2026
…20) (#23)

Closes #16.

`https://deno.land/install-manual@v${DENO_VERSION}.sh` returns content
that sh cannot parse:

  sh: syntax error: unexpected redirection
  curl: (23) Failure writing output to destination, passed 16375 returned 0

Replaced with the explicit binary release URL pattern from
denoland/deno's GitHub releases. That path is the canonical,
deterministic download surface — no shell parsing involved, no
install-script redirection, and the URL pattern is stable across
Deno releases.
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.

[container-stack/rokur] Deno install URL 'install-manual@vX.Y.Z' returns content that breaks sh

1 participant