[ 
https://issues.apache.org/jira/browse/SLIDER-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15052232#comment-15052232
 ] 

Josh Elser commented on SLIDER-1005:
------------------------------------

bq.  I have attached a patch for this, move the plugin from profile to build. 

Don't we still want this to be in a profile, per Steve's earlier remark?

One other question from your patch:

{code}
<output>${project.build.directory}/generated-sources/java</output>
{code}

Is Maven smart enough to find the sources here and include them in the normal 
jar? Some sort of convention?

> 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)

Reply via email to