You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Adar Dembo (Code Review)" <ge...@cloudera.org> on 2016/02/26 03:32:15 UTC

[kudu-CR] java: fix fallback to system protoc

Hello Jean-Daniel Cryans,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2321

to review the following change.

Change subject: java: fix fallback to system protoc
......................................................................

java: fix fallback to system protoc

This used to work, but broke in commit 65b7e5b with the introduction of
additionalProtoPathelements. The new approach uses an automatically
activated maven profile which obviates the need for the protoc wrapper
script.

Change-Id: I9f307844b455667f87b56df4e12b27b39344aa71
---
D java/kudu-client/dev-support/build-proto.sh
M java/kudu-client/pom.xml
2 files changed, 34 insertions(+), 37 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/21/2321/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2321
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9f307844b455667f87b56df4e12b27b39344aa71
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans