I can revert the findbugs part of my changes if that's what the team
wants. Seems Simon wants that also, but I prefer my approach, since I
don't have to drop to the command line and start typing commands to run
Findbugs. With my approach I can run it from the IDE, and if I have to
drop to the command line to run it then I will likely forget. I'm not
sure why Simon is so opposed to my change, he just says he doesn't like
it (despite the fact he doesn't even use Maven), but if you don't like
it too then I will revert
I'm still going to want access to CI to remove the explicit call to the
checkstyle target though. Since my changes its currently being called twice.
Chris
On 31/03/2016 08:18, Glenn Adams wrote:
I wonder if it is actually worth creating a phase for findbugs which
invocation still requires one to specify something like -P
runFindbugs. It is just as easy to add a goal findbugs:check on the
command line, isn't it? That's what I've been using. For the CI
configuration, I had used:
mvn clean install checkstyle:checkstyle findbugs:findbugs -B -U -e -fae -V
On Thu, Mar 31, 2016 at 12:47 AM, Chris Bowditch
<bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com>> wrote:
Hi Glenn,
As mentioned in another thread. I can't quite complete the changes
until I'm given access to CI by being added to the group
hudson-jobadmin. Also, I've only done fop-core so far. Once I've
got it working and the team are happy I will propagate my changes
to the other modules.
Thanks,
Chris
On 30/03/2016 16:38, Glenn Adams wrote:
Thanks for fixing. I haven't had a chance to work on the site
phase, but will do so ASAP.
On Wed, Mar 30, 2016 at 1:56 AM, Chris Bowditch
<bowditch_ch...@hotmail.com
<mailto:bowditch_ch...@hotmail.com>
<mailto:bowditch_ch...@hotmail.com
<mailto:bowditch_ch...@hotmail.com>>> wrote:
The site phase currently appears to be broken. Rather than
investigate that, I moved findBugs into a profile. Which
should
suit the requirements of some people wanting findBugs run
as part
of install and others not wanting it run. Its off by default.
Chris
On 29/03/2016 16:33, Chris Bowditch wrote:
I prefer to know ASAP whether checkstyle or findbugs is
broken, and having them in validate/verify phases supports
this. Moving them to manual scripts (as was the case
previously) is different to other Maven projects I've
worked
with and seems to encourage introducing new issues.
Moving the
phase of Findbugs seems like a fair compromise, I'll
have a
look into that tomorrow. Alternatively I can move FindBugs
into its own profile which you have to explicitly enable.
Thanks,
Chris
On 29/03/2016 16:10, Glenn Adams wrote:
On Tue, Mar 29, 2016 at 1:59 AM, Simon Steiner
<simonsteiner1...@gmail.com
<mailto:simonsteiner1...@gmail.com>
<mailto:simonsteiner1...@gmail.com
<mailto:simonsteiner1...@gmail.com>>
<mailto:simonsteiner1...@gmail.com
<mailto:simonsteiner1...@gmail.com>
<mailto:simonsteiner1...@gmail.com
<mailto:simonsteiner1...@gmail.com>>>> wrote:
Hi,
After this change findbugs is always run for
example:
mvn clean
install -DskipTests
This doesn’t work on java 8 and takes much
longer to run.
I agree with Simon. It is preferable to explicitly
specify
a findbugs goal when performing compile or
install. The
more standard process is to have findbugs run
automatically when running the site phase.
Thanks
-----Original Message-----
From: cbowdi...@apache.org
<mailto:cbowdi...@apache.org>
<mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>> <mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>
<mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>>>
[mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>
<mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>> <mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>
<mailto:cbowdi...@apache.org
<mailto:cbowdi...@apache.org>>>]
Sent: 24 March 2016 16:17
To: fop-comm...@xmlgraphics.apache.org
<mailto:fop-comm...@xmlgraphics.apache.org>
<mailto:fop-comm...@xmlgraphics.apache.org
<mailto:fop-comm...@xmlgraphics.apache.org>>
<mailto:fop-comm...@xmlgraphics.apache.org
<mailto:fop-comm...@xmlgraphics.apache.org>
<mailto:fop-comm...@xmlgraphics.apache.org
<mailto:fop-comm...@xmlgraphics.apache.org>>>
Subject: svn commit: r1736460 -
/xmlgraphics/fop/trunk/fop-core/pom.xml
Author: cbowditch
Date: Thu Mar 24 16:16:47 2016
New Revision: 1736460
URL:
http://svn.apache.org/viewvc?rev=1736460&view=rev
Log:
Avoid the need to run checkstyle and findbugs
separately and
integrate them into the build process;
checkstyle runs
in validate
(pre-compile) phase and findbugs runs in verify
(post-compile) phase
Modified:
xmlgraphics/fop/trunk/fop-core/pom.xml
Modified: xmlgraphics/fop/trunk/fop-core/pom.xml
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/pom.xml?rev=1736460&r1=1736459&r2=1736460&view=diff
==============================================================================
--- xmlgraphics/fop/trunk/fop-core/pom.xml
(original)
+++ xmlgraphics/fop/trunk/fop-core/pom.xml Thu
Mar 24
16:16:47 2016
@@ -300,6 +300,7 @@
</plugin>
<!-- code analysis - checkstyle -->
<plugin>
+ <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<configuration>
<configLocation>${project.baseUri}src/tools/resources/checkstyle/checkstyle.xml</configLocation>
@@ -311,6 +312,15 @@
<suppressionsLocation>${project.baseUri}src/tools/resources/checkstyle/suppressions.xml</suppressionsLocation>
<violationSeverity>warning</violationSeverity>
</configuration>
+ <executions>
+ <execution>
+ <id>validate</id>
+ <phase>validate</phase>
+ <goals>
+ <goal>check</goal>
+ </goals>
+ </execution>
+ </executions>
</plugin>
<!-- code analysis - findbugs -->
<plugin>
@@ -322,6 +332,15 @@
<effort>Max</effort>
<threshold>Low</threshold>
</configuration>
+ <executions>
+ <execution>
+ <id>verify</id>
+ <phase>verify</phase>
+ <goals>
+ <goal>check</goal>
+ </goals>
+ </execution>
+ </executions>
</plugin>
</plugins>
<resources>
---------------------------------------------------------------------
To unsubscribe, e-mail:
fop-commits-unsubscr...@xmlgraphics.apache.org
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org>
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org>>
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org>
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org
<mailto:fop-commits-unsubscr...@xmlgraphics.apache.org>>>
For additional commands, e-mail:
fop-commits-h...@xmlgraphics.apache.org
<mailto:fop-commits-h...@xmlgraphics.apache.org>
<mailto:fop-commits-h...@xmlgraphics.apache.org
<mailto:fop-commits-h...@xmlgraphics.apache.org>>
<mailto:fop-commits-h...@xmlgraphics.apache.org
<mailto:fop-commits-h...@xmlgraphics.apache.org>
<mailto:fop-commits-h...@xmlgraphics.apache.org
<mailto:fop-commits-h...@xmlgraphics.apache.org>>>