Skip to content

Improvements to Protobuf.Makefile #356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Oct 30, 2025
Merged

Improvements to Protobuf.Makefile #356

Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
066d877
Rename a variable
dkovba Oct 24, 2025
f732d45
Improve the assignment
dkovba Oct 24, 2025
63a1caa
Fix a target definition
dkovba Oct 24, 2025
f7524bc
Merge similar targets
dkovba Oct 24, 2025
d49019a
Improve comments
dkovba Oct 24, 2025
909dd90
Add a target to clean protos
dkovba Oct 25, 2025
3bf0280
Add a license header
dkovba Oct 29, 2025
9faec98
Improve the target for cleaning proto tools
dkovba Oct 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 14 additions & 11 deletions Protobuf.Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
LOCAL_DIR := $(ROOT_DIR)/.local
LOCALBIN := $(LOCAL_DIR)/bin
LOCAL_BIN_DIR := $(LOCAL_DIR)/bin

## Versions
PROTOC_VERSION=26.1
# Versions
PROTOC_VERSION := 26.1

# protoc binary installation
PROTOC_ZIP = protoc-$(PROTOC_VERSION)-osx-universal_binary.zip
PROTOC = $(LOCALBIN)/protoc@$(PROTOC_VERSION)/protoc
# Protoc binary installation
PROTOC_ZIP := protoc-$(PROTOC_VERSION)-osx-universal_binary.zip
PROTOC := $(LOCAL_BIN_DIR)/protoc@$(PROTOC_VERSION)/protoc
$(PROTOC):
@echo Downloading protocol buffers...
@mkdir -p $(LOCAL_DIR)
Expand All @@ -16,14 +16,13 @@ $(PROTOC):
@unzip -o $(PROTOC_ZIP) 'include/*' -d $(dir $@)
@rm -f $(PROTOC_ZIP)

protoc_gen_grpc_swift:
swift build --product protoc-gen-grpc-swift

.PHONY: protoc-gen-swift
protoc-gen-swift:
swift build --product protoc-gen-swift
@$(SWIFT) build --product protoc-gen-swift
Copy link
Contributor

@jglogan jglogan Oct 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SWIFT has to be defined in the environment or the build fails?

I don't see any default definition.

Copy link
Contributor Author

@dkovba dkovba Oct 27, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Protobuf.Makefile is imported by Makefile, where the variables are defined. Other such variables outside changes in this PR - ROOT_DIR and BUILD_BIN_DIR.

@$(SWIFT) build --product protoc-gen-grpc-swift

.PHONY: protos
protos: $(PROTOC) protoc_gen_grpc_swift protoc-gen-swift
protos: $(PROTOC) protoc-gen-swift
@echo Generating protocol buffers source code...
@$(PROTOC) Sources/Containerization/SandboxContext/SandboxContext.proto \
--plugin=protoc-gen-grpc-swift=$(BUILD_BIN_DIR)/protoc-gen-grpc-swift \
Expand All @@ -35,3 +34,7 @@ protos: $(PROTOC) protoc_gen_grpc_swift protoc-gen-swift
--swift_opt=Visibility=Public \
-I.
@"$(MAKE)" update-licenses

.PHONY: clean-protos
clean-protos:
@rm -rf $(LOCAL_DIR)