Extend Makefile with protoc

.gitignore

@@ -1,3 +1,5 @@
+bin/
+
 *.swp
 .idea
 *.qlog

Makefile

@@ -1,11 +1,47 @@
 export GOBIN := $(abspath .)/bin
 export PATH := $(GOBIN):$(PATH)
 
-.PHONY: install
-install:
+ifeq ($(OS),Windows_NT)
+	UNAME_OS := windows
+	ifeq ($(PROCESSOR_ARCHITECTURE),AMD64)
+		UNAME_ARCH := x86_64
+	endif
+	ifeq ($(PROCESSOR_ARCHITECTURE),ARM64)
+		UNAME_ARCH := aarch64
+	endif
+	PROTOC_BUILD := win64
+
+	BIN_DIR := $(abspath .)/bin
+	export PATH := $(BIN_DIR);$(PATH)
+	TMP_PROTOC := $(TEMP)/protoc-$(RANDOM)
+else
+	UNAME_OS := $(shell uname -s)
+	UNAME_ARCH := $(shell uname -m)
+	PROTOC_BUILD := $(shell echo ${UNAME_OS}-${UNAME_ARCH} | tr '[:upper:]' '[:lower:]' | sed 's/darwin/osx/' | sed 's/arm64/aarch_64/' | sed 's/aarch64/aarch_64/')
+
+ 	BIN_DIR := $(abspath .)/bin
+ 	export PATH := $(BIN_DIR):$(PATH)
+ 	TMP_PROTOC := $(shell mktemp -d)
+endif
+
+.PHONY: install-protoc
+install-protoc: protoc-plugins
+	@mkdir -p $(BIN_DIR)
+ifeq ($(OS),Windows_NT)
+	@mkdir -p $(TMP_PROTOC)
+endif
+	curl -sSL https://github.com/protocolbuffers/protobuf/releases/download/v26.0/protoc-26.0-${PROTOC_BUILD}.zip -o $(TMP_PROTOC)/protoc.zip
+	@unzip $(TMP_PROTOC)/protoc.zip -d $(TMP_PROTOC)
+	@cp -f $(TMP_PROTOC)/bin/protoc $(BIN_DIR)/protoc
+	@chmod +x $(BIN_DIR)/protoc
+
+protoc-plugins:
 	go install google.golang.org/protobuf/cmd/protoc-gen-go@v1.34.0
+
+.PHONY: install
+install: install-protoc
 	go install go.uber.org/mock/mockgen@v0.4.0
 
 .PHONY: generate
-generate: install
+generate:
 	go generate ./...

core/network/mocks/network.go

@@ -1,8 +1,8 @@
 package mocknetwork
 
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_resource_manager.go github.com/libp2p/go-libp2p/core/network ResourceManager"
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_conn_management_scope.go github.com/libp2p/go-libp2p/core/network ConnManagementScope"
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_stream_management_scope.go github.com/libp2p/go-libp2p/core/network StreamManagementScope"
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_peer_scope.go github.com/libp2p/go-libp2p/core/network PeerScope"
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_protocol_scope.go github.com/libp2p/go-libp2p/core/network ProtocolScope"
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package mocknetwork -destination mock_resource_scope_span.go github.com/libp2p/go-libp2p/core/network ResourceScopeSpan"
+//go:generate mockgen -package mocknetwork -destination mock_resource_manager.go github.com/libp2p/go-libp2p/core/network ResourceManager
+//go:generate mockgen -package mocknetwork -destination mock_conn_management_scope.go github.com/libp2p/go-libp2p/core/network ConnManagementScope
+//go:generate mockgen -package mocknetwork -destination mock_stream_management_scope.go github.com/libp2p/go-libp2p/core/network StreamManagementScope
+//go:generate mockgen -package mocknetwork -destination mock_peer_scope.go github.com/libp2p/go-libp2p/core/network PeerScope
+//go:generate mockgen -package mocknetwork -destination mock_protocol_scope.go github.com/libp2p/go-libp2p/core/network ProtocolScope
+//go:generate mockgen -package mocknetwork -destination mock_resource_scope_span.go github.com/libp2p/go-libp2p/core/network ResourceScopeSpan

p2p/host/basic/mocks.go

@@ -1,6 +1,4 @@
-//go:build gomock || generate
-
 package basichost
 
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -build_flags=\"-tags=gomock\" -package basichost -destination mock_nat_test.go github.com/libp2p/go-libp2p/p2p/host/basic NAT"
+//go:generate mockgen -package basichost -destination mock_nat_test.go github.com/libp2p/go-libp2p/p2p/host/basic NAT
 type NAT nat

p2p/host/pstoremanager/pstoremanager_test.go

@@ -15,7 +15,7 @@ import (
 	"go.uber.org/mock/gomock"
 )
 
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package pstoremanager_test -destination mock_peerstore_test.go github.com/libp2p/go-libp2p/core/peerstore Peerstore"
+//go:generate mockgen -package pstoremanager_test -destination mock_peerstore_test.go github.com/libp2p/go-libp2p/core/peerstore Peerstore
 
 func TestGracePeriod(t *testing.T) {
 	t.Parallel()

p2p/net/nat/nat_test.go

@@ -13,7 +13,7 @@ import (
 	"go.uber.org/mock/gomock"
 )
 
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package nat -destination mock_nat_test.go github.com/libp2p/go-nat NAT"
+//go:generate mockgen -package nat -destination mock_nat_test.go github.com/libp2p/go-nat NAT
 
 func setupMockNAT(t *testing.T) (mockNAT *MockNAT, reset func()) {
 	t.Helper()

p2p/test/transport/gating_test.go

@@ -18,7 +18,7 @@ import (
 	"go.uber.org/mock/gomock"
 )
 
-//go:generate go run go.uber.org/mock/mockgen@v0.4.0 -package transport_integration -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater
+//go:generate mockgen -package transport_integration -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater
 
 func stripCertHash(addr ma.Multiaddr) ma.Multiaddr {
 	for {

p2p/test/transport/transport_test.go

@@ -455,7 +455,7 @@ func TestMoreStreamsThanOurLimits(t *testing.T) {
 						var err error
 						// maxRetries is an arbitrary retry amount if there's any error.
 						maxRetries := streamCount * 4
-						shouldRetry := func(err error) bool {
+						shouldRetry := func(error) bool {
 							didErr = true
 							sawFirstErr.Store(true)
 							maxRetries--

p2p/transport/quic/conn_test.go

@@ -27,7 +27,7 @@ import (
 	"go.uber.org/mock/gomock"
 )
 
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package libp2pquic -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater && go run golang.org/x/tools/cmd/goimports@v0.20.0 -w mock_connection_gater_test.go"
+//go:generate mockgen -package libp2pquic -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater
 
 type connTestCase struct {
 	Name    string

p2p/transport/webtransport/transport_test.go

@@ -360,7 +360,7 @@ func TestResourceManagerListening(t *testing.T) {
 }
 
 // TODO: unify somehow. We do the same in libp2pquic.
-//go:generate sh -c "go run go.uber.org/mock/mockgen@v0.4.0 -package libp2pwebtransport_test -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater && go run golang.org/x/tools/cmd/goimports@v0.20.0 -w mock_connection_gater_test.go"
+//go:generate mockgen -package libp2pwebtransport_test -destination mock_connection_gater_test.go github.com/libp2p/go-libp2p/core/connmgr ConnectionGater
 
 func TestConnectionGaterDialing(t *testing.T) {
 	ctrl := gomock.NewController(t)