Hi,
Not sure mvn checkstyle is working anymore? Thanks From: Glenn Adams [mailto:gl...@skynav.com] Sent: 31 March 2016 17:51 To: FOP Developers <fop-dev@xmlgraphics.apache.org> Subject: Re: svn commit: r1736460 - /xmlgraphics/fop/trunk/fop-core/pom.xml I have temporarily reverted your changes to the build/plugins configs for CS/FB plugins, while leaving your runFindbugs profile in place. Keep in mind that there are different CS/FB goals that are used. We have to make sure the configs work for: * checkstyle:check * checkstyle:checkstyle * findbugs:check * findbugs:findbugs The 1st and 3rd of these, which do not generate reports, are generally going to be used by devs when performing day to day dev work; while the 2nd and 4th of these, which do generate reports, are going to be used by CI and site goals. It would be best if we keep non-profile/execution plugin configs that support these use cases, then add profiles as needed on top of these configs. Note that CI needs to run checkstyle:checkstyle and findbugs:findbugs in order to generate reports used by CI to populate the project stats, etc. On Thu, Mar 31, 2016 at 9:47 AM, Glenn Adams <gl...@skynav.com <mailto:gl...@skynav.com> > wrote: On Thu, Mar 31, 2016 at 9:37 AM, Chris Bowditch <bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> > wrote: Hi Glenn, On 31/03/2016 16:20, Glenn Adams wrote: On Thu, Mar 31, 2016 at 1:59 AM, Chris Bowditch <bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> >> wrote: 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 When you run from IDE, can't you specify explicit goals? Or are you limited to specifying a list of profiles to apply? 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. It looks like I may need to ask INFRA to add the access, since I don't see any way to do it myself. I found the command to run on minotaur, but got an access denied error. The instructions say you need to be a PMC chair to run the script; https://wiki.apache.org/general/Jenkins#How_do_I_get_an_account thanks; i've added you, so go ahead with your tweaks to the CI In any case, it looks like the problem with the current build failure is that the exclusions file is not being found. I'm working on the pom and config now, so give me until the end of the day to see if I can arrive at a solution. The exclusions file isn't found because the plugin configuration has been moved under a profile. Therefore maven is falling back to a default findbugs configuration. Specifying the profile will resolve this 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> <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> > <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> <mailto: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> > <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> >> <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> <mailto:bowditch_ch...@hotmail.com <mailto:bowditch_ch...@hotmail.com> > <mailto: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> >> <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 <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> <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> <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> >>> <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 <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> <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 <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> >> <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 <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> <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 <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 <http://svn.apache.org/viewvc?rev=1736460&view=rev> &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 <http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/pom.xml?rev=1736460&r1=1736459&r2=1736460&view=diff> &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> >> <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 <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> <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 <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> >> <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 <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> <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 <mailto:fop-commits-h...@xmlgraphics.apache.org> <mailto:fop-commits-h...@xmlgraphics.apache.org <mailto:fop-commits-h...@xmlgraphics.apache.org> >>>>