The architecture is no longer passed in at build time:
main.go and globals: the Buildarch variable and field are removed, not left empty.
logger.Identify and the listen log line in server/http.go report runtime.GOARCH under the key arch in place of buildarch.
backend/script/build drops -X main.Buildarch=$(uname -m) and its header's mention of the architecture; the Dockerfile comment quoting the build command drops it too.
Worth knowing: the log key changes from buildarch to arch, and the value changes from uname -m naming (x86_64, aarch64) to Go naming (amd64, arm64). Anything that filters logs on the old key or values would need updating; nothing in this repo does.
Deviation: backend/README.md is not in the issue's file list, but it said the build stamps in the architecture, which the definition of done rules out, so that one sentence is changed.
Judgement call: the globals package comment still describes build-time values; it stays true for Appname and Version, so it is unchanged.
Model: opus-5-5
Implements https://git.eeqj.de/sneak/netwatch/issues/83, following https://git.eeqj.de/sneak/prompts/issues/66.
The architecture is no longer passed in at build time:
- `main.go` and `globals`: the `Buildarch` variable and field are removed, not left empty.
- `logger.Identify` and the listen log line in `server/http.go` report `runtime.GOARCH` under the key `arch` in place of `buildarch`.
- `backend/script/build` drops `-X main.Buildarch=$(uname -m)` and its header's mention of the architecture; the `Dockerfile` comment quoting the build command drops it too.
Worth knowing: the log key changes from `buildarch` to `arch`, and the value changes from `uname -m` naming (`x86_64`, `aarch64`) to Go naming (`amd64`, `arm64`). Anything that filters logs on the old key or values would need updating; nothing in this repo does.
Deviation: `backend/README.md` is not in the issue's file list, but it said the build stamps in the architecture, which the definition of done rules out, so that one sentence is changed.
Judgement call: the `globals` package comment still describes build-time values; it stays true for `Appname` and `Version`, so it is unchanged.
Model: opus-5-5
The architecture is no longer passed in at build time. The Buildarch
variable and field are gone from main and globals, script/build no
longer stamps it in with -X, and the startup and listen log lines
report runtime.GOARCH under the key "arch". The Dockerfile comment and
backend/README.md no longer describe an architecture being stamped in.
Model: opus-5-5
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Implements #83, following sneak/prompts#66.
The architecture is no longer passed in at build time:
main.goandglobals: theBuildarchvariable and field are removed, not left empty.logger.Identifyand the listen log line inserver/http.goreportruntime.GOARCHunder the keyarchin place ofbuildarch.backend/script/builddrops-X main.Buildarch=$(uname -m)and its header's mention of the architecture; theDockerfilecomment quoting the build command drops it too.Worth knowing: the log key changes from
buildarchtoarch, and the value changes fromuname -mnaming (x86_64,aarch64) to Go naming (amd64,arm64). Anything that filters logs on the old key or values would need updating; nothing in this repo does.Deviation:
backend/README.mdis not in the issue's file list, but it said the build stamps in the architecture, which the definition of done rules out, so that one sentence is changed.Judgement call: the
globalspackage comment still describes build-time values; it stays true forAppnameandVersion, so it is unchanged.Model: opus-5-5
Review passed.
Model: opus-5-5