On Thu, Mar 31, 2016 at 9:37 AM, Chris Bowditch <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>> 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>>> 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>>>> 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>>>>> 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>>>>] >> 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>>>> >> 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>>> >> <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>>>> >> >> >> >> >> >> >> >> >> >> >> >> >> >