basho-labs / rebar3_protobuf_plugin

Rebar 3 Plugin to compile protobuffs as part of a build process.
Apache License 2.0
2 stars 3 forks source link

protobuf plugin #1

Open cmkarlsson opened 8 years ago

cmkarlsson commented 8 years ago

I wasn't aware of this repository before. I had to create a rebar3 protobuf plugin to be able to compile the riak client (which has riak_pb as dependency). You can find it under https://github.com/cmkarlsson/rebar3_protobuffs_plugin if there is something you want to reuse here.

Cheers

binarytemple-external commented 7 years ago

@lukebakken @cuyler I've tested @cmkarlsson plugin against riak-erlang-client (master 13 Dec 2016) and it builds. This project seems abandoned. Perhaps it would be good to adopt @cmkarlsson project as offered?

lukebakken commented 7 years ago

I'm looking into why this repository exists.

lukebakken commented 7 years ago

For what it's worth, I have switched the erlang client to use gpb instead of erlang_protobuffs and modified the build process so that a protobuf compiler is not run on every build, making these plugins a moot point "very soon now" 😄

See basho/riak-erlang-client#339

cuyler commented 7 years ago

@lukebakken It was an abandoned attempt at rebar3 conversion two summers ago.

binarytemple-external commented 7 years ago

@lukebakken not sure what the implications of gpb will be, will it build with rebar3 ?

lukebakken commented 7 years ago

Yes, because the .proto files are parsed only when changes to them are made, and the generated files are commited to the repo (erl_protogen target). Prior to my change, the .proto files are parsed on every build which requires a rebar plugin.

binarytemple-external commented 7 years ago

@lukebakken I mean, will it still be possible to build riak-erlang-client using rebar3 with @cmkarlsson plugin. Or will the changes break that?

lukebakken commented 7 years ago

As of commit adc603b in riak_pb and c203271b in riak-erlang-client, you do not need any protobuf plugins for rebar to build either project. Go ahead and give it a try - the develop branch in both projects should work.

binarytemple-external commented 7 years ago

@lukebakken thanks, will give it a try.