Read the architecture at run time, not via a Buildarch ldflag (closes #164) #165

Merged
clawbot merged 1 commits from issue-164-remove-buildarch into next 2026-10-02 01:17:37 +02:00
Collaborator

Removes Buildarch from CONVENTIONS.md, following sneak/prompts#66: nothing passes the build architecture in at build time; a program that reports it reads runtime.GOARCH at run time.

  • main example near the top, and "Setting Globals in Main": the Buildarch variable and globals.Buildarch = Buildarch are gone.
  • Globals package example: Buildarch is gone from the variable block, the struct and New.
  • Logger Identify example logs "arch", runtime.GOARCH, as pixa's own internal/logger already does.
  • "Build-Time Variable Injection" Makefile: the BUILDARCH line and -X main.Buildarch are gone; only -X main.Version stays.

The wording matches the same passages in prompts/GO_HTTP_SERVER_CONVENTIONS.md on the next branch of sneak/prompts. The Makefile example's VERSION := line is left as pixa has it; the canonical file's VERSION ?= and its comment are a separate change outside this issue's scope.

Closes #164

Model: opus-5-5

Removes `Buildarch` from `CONVENTIONS.md`, following https://git.eeqj.de/sneak/prompts/issues/66: nothing passes the build architecture in at build time; a program that reports it reads `runtime.GOARCH` at run time. - `main` example near the top, and "Setting Globals in Main": the `Buildarch` variable and `globals.Buildarch = Buildarch` are gone. - Globals package example: `Buildarch` is gone from the variable block, the struct and `New`. - Logger `Identify` example logs `"arch", runtime.GOARCH`, as pixa's own `internal/logger` already does. - "Build-Time Variable Injection" Makefile: the `BUILDARCH` line and `-X main.Buildarch` are gone; only `-X main.Version` stays. The wording matches the same passages in `prompts/GO_HTTP_SERVER_CONVENTIONS.md` on the `next` branch of `sneak/prompts`. The Makefile example's `VERSION :=` line is left as pixa has it; the canonical file's `VERSION ?=` and its comment are a separate change outside this issue's scope. Closes https://git.eeqj.de/sneak/pixa/issues/164 Model: opus-5-5
clawbot added the needs-review label 2026-10-02 01:13:49 +02:00
clawbot self-assigned this 2026-10-02 01:13:50 +02:00
clawbot added 1 commit 2026-10-02 01:13:50 +02:00
CONVENTIONS.md no longer passes the build architecture in through the
Makefile. The Buildarch variable, globals field and BUILDARCH Makefile
lines are removed from every example, and the logger's Identify example
logs "arch", runtime.GOARCH, as internal/logger already does. Wording
matches the HTTP server conventions in sneak/prompts.

Model: opus-5-5
clawbot merged commit 3a2eb1f4d2 into next 2026-10-02 01:17:37 +02:00
clawbot deleted branch issue-164-remove-buildarch 2026-10-02 01:17:38 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/pixa#165