Fix WKT generation

Because of bufbuild/makego#281, we're now dealing with a symlink;
update the handling to match.

Still not quite right as we're manually updating the PROTOC_VERSION here
to match what we already have in this repo, overriding makego's managed
version. We should get in bufbuild/makego#295.

Author: stefanvanburen

make/buf/all.mk

@@ -74,7 +74,7 @@ include make/go/buf.mk
 installtest:: $(PROTOC) $(PROTOC_GEN_GO)
 
 .PHONY: godata
-godata: installwkt-go-data installbuf-legacyfederation-go-data $(PROTOC)
+godata: installwkt-go-data installbuf-legacyfederation-go-data $(PROTOC) $(PROTOC_INCLUDE)
 	rm -rf private/gen/data/datawkt
 	mkdir -p private/gen/data/datawkt
 	wkt-go-data "$(CACHE_INCLUDE)" --package datawkt --protobuf-version "$(PROTOC_VERSION)" > private/gen/data/datawkt/datawkt.gen.go

make/go/dep_protoc.mk

@@ -10,9 +10,9 @@ $(call _assert_var,CACHE_INCLUDE)
 $(call _assert_var,CACHE_BIN)
 
 # Settable
-# https://github.com/protocolbuffers/protobuf/releases 20250528 checked 20250603
+# https://github.com/protocolbuffers/protobuf/releases 20250225 checked 20250310
 # NOTE: Set to version compatible with genproto source code (only used in tests).
-PROTOC_VERSION ?= 31.1
+PROTOC_VERSION ?= 33.3
 
 # Google adds a dash to release candidate versions in the name of the
 # release artifact, i.e. v27.0-rc1 -> v27.0-rc-1

private/buf/bufwkt/cmd/wkt-go-data/main.go

@@ -106,7 +106,7 @@ func run(ctx context.Context, container appext.Container, flags *flags) error {
 	if protobufVersion == "" {
 		return appcmd.NewInvalidArgumentErrorf("--%s is required", protobufVersionFlagName)
 	}
-	readWriteBucket, err := storageos.NewProvider(storageos.ProviderWithSymlinks()).NewReadWriteBucket(dirPath)
+	readWriteBucket, err := storageos.NewProvider(storageos.ProviderWithSymlinks()).NewReadWriteBucket(dirPath, storageos.ReadWriteBucketWithSymlinksIfSupported())
 	if err != nil {
 		return err
 	}