[
https://issues.apache.org/jira/browse/RATIS-132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16240908#comment-16240908
]
Tsz Wo Nicholas Sze commented on RATIS-132:
-------------------------------------------
The generated proto java files are in {{target/generated-sources/protobuf/java}}
{code}
$ls -l
ratis-proto-shaded/target/generated-sources/protobuf/java/org/apache/ratis/shaded/
total 0
drwxr-xr-x 8 szetszwo staff 272 Nov 6 13:05 proto
{code}
Could we move them to {{src/main/java}}?
> Support simple 'mvn clean compile' command (move shading before the compile
> phase)
> -----------------------------------------------------------------------------------
>
> Key: RATIS-132
> URL: https://issues.apache.org/jira/browse/RATIS-132
> Project: Ratis
> Issue Type: Bug
> Reporter: Elek, Marton
> Assignee: Elek, Marton
> Priority: Trivial
> Attachments: RATIS-132.001.patch
>
>
> The current situation is described in the BUILDING.md:
> {code}
> When building Ratis the first time, shaded files need to be generated by the
> following command:
> ```
> $ mvn package -DskipTests
> ```
> After that, `mvn compile` or `mvn test` can be used as normal.
> For example, we may run the basic tests by
> ```
> {code}
> In short: the shading phase is bound to the 'package' maven phase, therefore
> we can't use 'mvn clean install' just 'mvn clean package' because the package
> phase should be done before the compilation of ratis-proto-shaded.
> This blocks the nightly build as yetus uses one 'mvn clean compile' or 'mvn
> clean test-compile' without invoking the package phase.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
