README says that make install needs nix (closes #200) #202

Merged
clawbot merged 1 commits from issue-200-install-needs-nix into next 2026-10-16 01:40:20 +02:00
Collaborator

Fixes #200.

make install runs make build-bin, which runs bun through nix-shell -p bun. On a machine without nix it stops with nix-shell: not found, and the README named no such prerequisite.

  • Getting Started now opens with what its commands need (git, node 22 or later, yarn) and says that make install, which builds the single binary the cron section uses, also needs nix, linked to https://nixos.org/download.
  • "Running the backup from cron" says the same where it introduces make install: what Getting Started needs plus nix, even where bun is already installed, because the Makefile always gets bun from nix-shell. The next sentence now names ~/bin/quak instead of "it", since "it" would otherwise point at make install.
  • TODO.md: a Completed Steps entry, and Next Step names the four bugs that are open (#196, #197, #198, #199) instead of saying no work is open.

The Makefile is unchanged, as the plan on the issue decided.

Judgement call: "node 22 or later" is the engines entry in package.json; the README did not state a node version before.

Model: opus-5-5

Fixes https://git.eeqj.de/sneak/quak/issues/200. `make install` runs `make build-bin`, which runs bun through `nix-shell -p bun`. On a machine without nix it stops with `nix-shell: not found`, and the README named no such prerequisite. - Getting Started now opens with what its commands need (git, node 22 or later, yarn) and says that `make install`, which builds the single binary the cron section uses, also needs nix, linked to https://nixos.org/download. - "Running the backup from cron" says the same where it introduces `make install`: what Getting Started needs plus nix, even where bun is already installed, because the `Makefile` always gets bun from `nix-shell`. The next sentence now names `~/bin/quak` instead of "it", since "it" would otherwise point at `make install`. - `TODO.md`: a Completed Steps entry, and Next Step names the four bugs that are open (https://git.eeqj.de/sneak/quak/issues/196, https://git.eeqj.de/sneak/quak/issues/197, https://git.eeqj.de/sneak/quak/issues/198, https://git.eeqj.de/sneak/quak/issues/199) instead of saying no work is open. The `Makefile` is unchanged, as the plan on the issue decided. Judgement call: "node 22 or later" is the `engines` entry in `package.json`; the README did not state a node version before. Model: opus-5-5
clawbot added the needs-review label 2026-10-16 01:07:20 +02:00
clawbot added 1 commit 2026-10-16 01:07:20 +02:00
`make install` runs `make build-bin`, which runs bun through
`nix-shell -p bun`, so on a machine without nix it stops with
`nix-shell: not found`, and the README named no such prerequisite.
Getting Started now says what its commands need and that `make install`
also needs nix, with a link to its download page. "Running the backup
from cron" says the same where it introduces `make install`, and that a
bun already installed does not help. The Makefile is unchanged, as the
plan on the issue decided. TODO.md's Next Step names the four bugs that
are open instead of saying none is.

Judgement call: "node 22 or later" is the `engines` entry in package.json.

Model: opus-5-5
clawbot self-assigned this 2026-10-16 01:07:27 +02:00
Author
Collaborator

Review passed.

Model: opus-5-5

Review passed. Model: opus-5-5
clawbot merged commit 5a940d4e4e into next 2026-10-16 01:40:19 +02:00
clawbot deleted branch issue-200-install-needs-nix 2026-10-16 01:40:20 +02:00
Sign in to join this conversation.