[
https://issues.apache.org/jira/browse/SLIDER-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15053606#comment-15053606
]
Steve Loughran commented on SLIDER-1005:
----------------------------------------
I'd still like to have that generated-sources under src/main and src/test, with
maven set up (a few extra lines) to grab the files, and the build to put it in
there a profile. Same for the avro bits.
why?
# protoc is an extra pain in the build, especially on windows
# IDEs can't find the files unless you do the build/rebuild, and it gets
complex when switching branches
# we don't change the build much
having them all in a separate dir marked generated should be enough to warn
people not to edit
> The java code generated by protobuf should not be placed in src directory
> -------------------------------------------------------------------------
>
> Key: SLIDER-1005
> URL: https://issues.apache.org/jira/browse/SLIDER-1005
> Project: Slider
> Issue Type: Improvement
> Components: build, core
> Affects Versions: Slider 0.90
> Environment: Centos 6.5, Maven 3.2.5
> Reporter: Pan Yuxuan
> Assignee: Pan Yuxuan
> Priority: Minor
> Attachments: SLIDER-1005-001.patch
>
>
> Now slider using protobuf for RPC, but the output directory is
> <output>${basedir}/src/main/java</output> in slider-core/pom.xml. That means
> the generated java code was been placed in the src directory.
> I have seen the generated java code for protobuf in Hadoop, they put the code
> in build directory
> <output>${project.build.directory}/generated-sources/java</output>. And they
> put the compile-protoc in <plugin> not <profile> in the pom.xml.
> I think the way Hadoop used is more reasonable. The code is generated by
> protobuf, not by ourselves.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
