> On Dec. 8, 2016, 8:59 p.m., Alex Clemmer wrote: > > 3rdparty/CMakeLists.txt, line 311 > > <https://reviews.apache.org/r/50415/diff/6/?file=1579858#file1579858line311> > > > > I must be missing something, because I don't quite see why it's > > necessary to copy `protoc` over to the Java root?
the build step in the source code requires protoc at relative location ../src/protoc > On Dec. 8, 2016, 8:59 p.m., Alex Clemmer wrote: > > 3rdparty/CMakeLists.txt, line 347 > > <https://reviews.apache.org/r/50415/diff/6/?file=1579858#file1579858line347> > > > > If I'm understanding this correctly, this will download a fresh copy of > > the protobuf tarball, and then build only the Java parts of it. > > > > Since we're already downloading this for the normal Protobuf build, it > > seems like it would be easier to get rid of the `ExternalProject_Add` call > > and simply append the java config, build, and install commands to the > > protobuf config, build, and install commands. For example: > > > > ``` > > set(PROTOBUF_BUILD_CMD ${PROTOBUF_BUILD_CMD} ${PROTOBUF_JAVA_BUILD_CMD}) > > ``` > > > > This should cause the normal protobuf build to build the Java stuff as > > well, no? I prefer to build the jar as a step for Java and I will add another step for Python. Hope this works. - Srinivas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50415/#review158565 ----------------------------------------------------------- On Dec. 8, 2016, 11:04 p.m., Srinivas Brahmaroutu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50415/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2016, 11:04 p.m.) > > > Review request for mesos, Alex Clemmer and Joseph Wu. > > > Bugs: MESOS-5902 > https://issues.apache.org/jira/browse/MESOS-5902 > > > Repository: mesos > > > Description > ------- > > Added config, build and install steps for the External project > protobuf_java. > 'make probuf_java' produces the > ${PROTOBUF_JAVA_ROOT}/java/target/protobuf-java-2.6.1.jar > > > Diffs > ----- > > 3rdparty/CMakeLists.txt a340b29fc1cc8b68bf74bfc0c9c2274f0af84eee > > Diff: https://reviews.apache.org/r/50415/diff/ > > > Testing > ------- > > cmake && make protobuf-2.6.1-java > check if > 3rdparty/protobuf-2.6.1/src/protobuf-2.6.1-java/java/target/protobuf-java-2.6.1.jar > > > Thanks, > > Srinivas Brahmaroutu > >
