Skip to content

Commit

Permalink
Update maven compiler plugin to newest version (apache#1042)
Browse files Browse the repository at this point in the history
* Update maven compiler plugin to newest version

The previously used version doesn't handle the errors from newer jdk
versions, so if there's a error you can't see what's actually wrong.

* Let aspectJ bind to correct phase

It was binding to process-sources before, which makes no sense since
it works on the class files. Removed the phase specification, which
allows it to bind to compile (as it does by default).
  • Loading branch information
ivankelly authored and merlimat committed Jan 30, 2018
1 parent 53fa9d0 commit 0439a1f
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -742,7 +742,7 @@ flexible messaging model and an intuitive client API.</description>
</plugin>
<plugin>
<artifactId>maven-compiler-plugin</artifactId>
<version>2.3.2</version>
<version>3.7.0</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
Expand Down
1 change: 0 additions & 1 deletion pulsar-broker/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,6 @@
</configuration>
<executions>
<execution>
<phase>process-sources</phase>
<goals>
<goal>compile</goal>
</goals>
Expand Down
1 change: 0 additions & 1 deletion pulsar-zookeeper/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@
</configuration>
<executions>
<execution>
<phase>process-sources</phase>
<goals>
<goal>compile</goal>
</goals>
Expand Down

0 comments on commit 0439a1f

Please sign in to comment.