Skip to content

Rerender 06-matrix-images to fix Python installs#18

Closed
ianpittwood wants to merge 1 commit intomainfrom
rerender-06-matrix-images
Closed

Rerender 06-matrix-images to fix Python installs#18
ianpittwood wants to merge 1 commit intomainfrom
rerender-06-matrix-images

Conversation

@ianpittwood
Copy link
Copy Markdown
Contributor

No description provided.

@ianpittwood ianpittwood requested a review from bschwedler as a code owner April 13, 2026 20:14
bschwedler added a commit that referenced this pull request Apr 22, 2026
Replace the tarball download of Quarto with an apt install from the
Posit Open repository (`dl.posit.co/public/open/`). This sidesteps the
`${TARGETARCH}` expansion bug in #18 — where the PR swapped
`linux-amd64` for `linux-${TARGETARCH}` but the final stage did not
declare `ARG TARGETARCH`, so the URL expanded to
`...-linux-.tar.gz` and returned 404.

Why apt instead of patching the tarball URL:

- `quarto` is published as a multi-arch deb (amd64 + arm64) to the
  Posit Open repo on both `jammy` and `noble`, so the install works
  uniformly without URL construction.
- The base image already runs `setup.deb.sh` for the Pro repo; adding
  the Open repo is a one-liner that mirrors that pattern.
- No tarball extraction, no `TARGETARCH` juggling, no stale mirror
  concerns.

The `quarto` deb installs the binary to `/opt/quarto/bin/quarto` and
symlinks it into `/usr/local/bin/quarto` via its postinst script, so
TinyTeX is installed via that path. `xz-utils` is installed
explicitly because the TinyTeX tarball is `.tar.xz` and the Ubuntu
24.04 base does not include it.

Goss tests now assert the apt-installed Quarto package at the pinned
version and the presence of the `/opt/quarto/bin/quarto` binary,
replacing the previous `/opt/quarto/${QUARTO_VERSION}` file check
that no longer applies.

Supersedes #18.
@bschwedler bschwedler closed this Apr 22, 2026
@bschwedler bschwedler deleted the rerender-06-matrix-images branch April 22, 2026 18:48
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