Skip to content
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.

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

remove every language we don't care about #15

Merged
merged 2 commits into from
Sep 20, 2019
Merged

Conversation

evie404
Copy link

@evie404 evie404 commented Sep 20, 2019

I got sick of trying to make all the languages we don't actually use work... so I deleted everything except for closure, go, and gogo (maybe we might experiment with it one day!?).

This should make the footprint a lot smaller and maintenance a lot easier in the future.

Should we ever care for using one of those languages for protobuf, we shouldn't use rules_protobuf anyways in favor of whatever is the native Bazel rules.

build passed in: https://travis-ci.org/vsco/rules_protobuf/jobs/587324241

@evie404
Copy link
Author

evie404 commented Sep 20, 2019

this actually ends up making our upgrade story significantly easier. i already have a branch on Bazel 0.26.x


load("//ruby:rules.bzl", "ruby_proto_repositories")

ruby_proto_repositories()

Choose a reason for hiding this comment

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

😢

Copy link
Author

Choose a reason for hiding this comment

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

i know :'(

Copy link

@akakitani akakitani left a comment

Choose a reason for hiding this comment

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

👍 just curious, do you know what the blockers have been for us using go_proto_library from rules_go?

@evie404
Copy link
Author

evie404 commented Sep 20, 2019

@akakitani iirc, rules_go uses protoc-gen-go from golang/protobuf whereas rules_prorobuf uses protoc from protobuf. the latter has better plugin support which we need (or rather, i haven't been able to disentangle successfully)

relevant issue: golang/protobuf#547

Copy link

@promiseofcake promiseofcake left a comment

Choose a reason for hiding this comment

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

Wonderful, thanks!

@evie404 evie404 merged commit c66bd69 into master Sep 20, 2019
@evie404 evie404 deleted the rpai/remove_everything branch September 20, 2019 17:06
@evie404 evie404 mentioned this pull request Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants