patch: cmd/dist: stamp git commit into runtime.Version value

[bradfitz]

Summary: stamp the tailscale/go git commit into the toolchain
Upstream status: not applicable, probably. Haven't asked.
Importance: important for Tailscale to debug how binaries were built
Example prior commit: 26292c0 + 6a17f14 (the latter adds a thing for NixOS we should either merge into the former or just drop)

cc @danderson

[bradfitz]

Latest commit in our fork is b3dc9ccd7a5

But I don't see us using this anymore.

@danderson, @raggi, I saw it got removed in https://github.com/tailscale/corp/commit/ec4addf43a630d495cd842d0ed31bf97c3c91374

Do we still need this?

It might still be nice to ask a Tailscale-built binary exactly which git rev of https://github.com/tailscale/go it was built from, but we don't have that property today. We only know that it's e.g. "1.23.1" but not more granular than that.

[raggi]

I don't have a strong opininion either way. Certainly being able to identify it sounds nice, though I'd also quite like it if this actually worked:

% ~/.cache/tailscale-go/bin/go version -m ~/.cache/tailscale-go/bin/go
/Users/raggi/.cache/tailscale-go/bin/go: go1.23.1
        path    cmd/go
        build   -buildmode=exe
        build   -compiler=gc
        build   -gcflags=cmd/...=-dwarf=false
        build   -trimpath=true
        build   DefaultGODEBUG=asynctimerchan=1,gotypesalias=0,httplaxcontentlength=1,httpmuxgo121=1,httpservecontentkeepheaders=1,netedns0=0,panicnil=1,tls10server=1,tls3des=1,tlskyber=0,tlsrsakex=1,tlsunsafeekm=1,winreadlinkvolume=0,winsymlink=0,x509keypairleaf=0,x509negativeserial=1
        build   CGO_ENABLED=0
        build   GOARCH=arm64
        build   GOOS=darwin
        build   GOARM64=v8.0

Maybe something to raise with upstream.

In the meantime, is there a way to persuade the go build to include a .dSYM? I just realized I'm not even too sure how to match up the binaries without one.

[bradfitz]

ts1.23: bf15628
ts1.24: eb462ce (without the .github/workflows bit)