-
Notifications
You must be signed in to change notification settings - Fork 15.5k
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
Google Bazel support? #311
Comments
Seems it's the time to open-source the BUILD files we have been using internally for building protobufs :) And just for the reference, an ex-googler has already written some BUILD files for protobuf here: https://github.com/mzhaom/trunk/tree/master/third_party/protobuf |
The bazel-examples repository has rules for building protoc: https://github.com/google/bazel-examples/tree/master/protobuf-2.5.0. |
There is a feature request for proto_library rule in bazel: bazelbuild/bazel#52 Before it's implemented in bazel, we can probably provide it as a wrapper script first. |
There is an internal type to Google called RawMessage that is similar to json.RawMessage. Since there is no proper proto reflection API, we special-cased the Bytes method of RawMessage to access the raw bytes. This is a gross hack since Bytes() []byte is such a common method signature. Remove this hack. Fixes protocolbuffers#311
The newly released bazel seems to be a very promising build tool. Can we add Google Bazel build for protobuf?
http://bazel.io/
Thanks,
Huahang
The text was updated successfully, but these errors were encountered: