Skip to content

Conversation

@glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Jul 3, 2020

Motivation:

#869 adds a plugin containing the
protoc binaries for CocoaPods. It relies on having a zip of the binaries
in the tagged release.

Modifications:

  • Add a script to generate a zip of the protoc binaries.

Result:

We can generate a bundle of plugins which may be pulled in by a
CocoaPod.


This change is Reviewable

Motivation:

grpc#869 adds a plugin containing the
protoc binaries for CocoaPods. It relies on having a zip of the binaries
in the tagged release.

Modifications:

- Add a script to generate a zip of the protoc binaries.

Result:

We can generate a bundle of plugins which may be pulled in by a
CocoaPod.
@glbrntt glbrntt added nio semver/none No version bump required. labels Jul 3, 2020
@glbrntt glbrntt requested a review from MrMage July 3, 2020 09:18
@glbrntt
Copy link
Collaborator Author

glbrntt commented Jul 3, 2020

cc @Lutzifer

@Lutzifer
Copy link
Contributor

Lutzifer commented Jul 3, 2020

Should this be another Makefile target?

@glbrntt
Copy link
Collaborator Author

glbrntt commented Jul 3, 2020

Should this be another Makefile target?

A script is fine; it'll only be run by me or @MrMage when we tag a release.

@Lutzifer
Copy link
Contributor

Lutzifer commented Jul 3, 2020

Ah, thanks to the new Github design, I thought this was only an issue, not a PR 🙈

Good job!

Copy link
Collaborator

@MrMage MrMage left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@glbrntt glbrntt merged commit 4e91a40 into grpc:master Jul 3, 2020
@glbrntt glbrntt deleted the gb-add-plugin-bundling-script branch July 3, 2020 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants