I believe it's... mvn -X
Sent from tablet device. Please excuse typos and brevity. On Oct 23, 2011 4:43 PM, "Phil Steitz" <phil.ste...@gmail.com> wrote: > On 10/23/11 12:42 PM, Oliver Heger wrote: > > Am 23.10.2011 20:16, schrieb Simone Tripodi: > >> Hi Oliver!! > >> I now realize that the checkstyle plugin is not smart as we > >> expect :) > >> > >> I guess you launched `mvn checkstyle:checkstyle`, if you do a > >> > >> `svn up&& mvn clean site&& open target/site/checkstyle.html` > >> > >> you can see totally different result!!! > >> Please let me know, many thanks in advance! > >> All the best, > >> Simo > > > > Hm, actually I did a mvn clean site:site. > > Beats me. Looks like you are still just not picking up the > suppressions file. Try running mvn with the -d or whatever the > debug thingy is and see what file it is trying to pick up. Normally > the / will get correctly changed to a \ for Windows. I thought that > it might be the suppressionsFileExpression that needed to be > changed, since what is in the [digester] pom is different from the > example in the docs; but it doesn't seem to matter what you put > there, as long as it is non-empty. The docs say nothing about what > that property should be set to and the code seems to ignore it, > other than checking that its not null. > > Another thing that I noticed running the build is > [WARNING] > > /Users/philsteitz/digester/trunk/src/main/java/org/apache/commons/digester3/CallMethodRule.java:85: > warning - Tag @link: reference not found: MethodUtils#invokeMethod > [WARNING] > > /Users/philsteitz/digester/trunk/src/main/java/org/apache/commons/digester3/CallMethodRule.java:85: > warning - Tag @link: reference not found: MethodUtils#invokeExactMethod > > These are because the BeanUtils MethodUtils class is not imported > fully any more. Either you need to just import the class or fully > qualify the names in the links. Are we sure that the class javadoc > here and elsewhere is correct for 3.0? > > Phil > > > > Oliver > > > >> > >> http://people.apache.org/~simonetripodi/ > >> http://simonetripodi.livejournal.com/ > >> http://twitter.com/simonetripodi > >> http://www.99soft.org/ > >> > >> > >> > >> On Sun, Oct 23, 2011 at 8:06 PM, Oliver Heger > >> <oliver.he...@oliver-heger.de> wrote: > >>> Am 23.10.2011 18:37, schrieb Simone Tripodi: > >>>> > >>>> Nice hint, thanks Phil!!! > >>>> all the best, > >>>> Simo > >>> > >>> It's really strange. I locally changed the digester pom to use > >>> version 2.7 > >>> of checkstyle, but again I see the same errors. I uploaded > >>> checkstyle.html > >>> to [1] so you can have a look at the output. > >>> > >>> Are the warnings really meaningful and do they correspond to > >>> your checkstyle > >>> configuration? > >>> > >>> Oliver > >>> > >>> [1] http://people.apache.org/~oheger/checkstyle.html > >>> > >>>> > >>>> http://people.apache.org/~simonetripodi/ > >>>> http://simonetripodi.livejournal.com/ > >>>> http://twitter.com/simonetripodi > >>>> http://www.99soft.org/ > >>>> > >>>> > >>>> > >>>> On Sun, Oct 23, 2011 at 6:10 PM, Phil > >>>> Steitz<phil.ste...@gmail.com> > >>>> wrote: > >>>>> > >>>>> On 10/22/11 8:02 AM, Oliver Heger wrote: > >>>>>> > >>>>>> Am 21.10.2011 23:19, schrieb Simone Tripodi: > >>>>>>> > >>>>>>> Forgot to mention about checkstyle: no idea. If you built the > >>>>>>> Digester > >>>>>>> using the provided pom, there shouldn't be ambiguity... any > >>>>>>> hint? > >>>>>> > >>>>>> Being no maven guru, I haven't got a clue either. I also would > >>>>>> expect that the pom contains sufficient information to have > >>>>>> reproducible build results. I ran a mvn help:effective-pom, and > >>>>>> here is the part I got for the configuration of the checkstyle > >>>>>> plug-in: > >>>>>> > >>>>>> <plugin> > >>>>>> <artifactId>maven-checkstyle-plugin</artifactId> > >>>>>> <version>2.6</version> > >>>>>> <configuration> > >>>>>> > >>>>>> <configLocation>config/maven_checks.xml</configLocation> > >>>>>> > >>>>>> <headerLocation>config/maven-header.txt</headerLocation> > >>>>>> > >>>>>> > >>>>>> > <suppressionsLocation>D:\data\projects\OpenSource\digester\commons-digester3-3.1-src/checkstyle-suppressions.xml</suppressionsLocation> > >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> > <suppressionsFileExpression>D:\data\projects\OpenSource\digester\commons-digester3-3.1-src/checkstyle-suppressions.xml</suppressionsFileExpression> > >>>>>> > >>>>>> > >>>>>> </configuration> > >>>>>> </plugin> > >>>>>> > >>>>>> Do you get a similar result? > >>>>> > >>>>> Looks like the plugin version is 2.6. Could be the failure on > >>>>> Windows is due to [1] which is fixed in 2.7. Try changing the > >>>>> plugin version to 2.7 and see if that fixes it. > >>>>> > >>>>> Phil > >>>>> [1] http://jira.codehaus.org/browse/MCHECKSTYLE-149 > >>>>>> > >>>>>> Oliver > >>>>>> > >>>>>>> TIA!!! > >>>>>>> Simo > >>>>>>> > >>>>>>> http://people.apache.org/~simonetripodi/ > >>>>>>> http://simonetripodi.livejournal.com/ > >>>>>>> http://twitter.com/simonetripodi > >>>>>>> http://www.99soft.org/ > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> On Fri, Oct 21, 2011 at 11:14 PM, Simone Tripodi > >>>>>>> <simonetrip...@apache.org> wrote: > >>>>>>>> > >>>>>>>> Hi Oliver, > >>>>>>>> thanks for the deep review!! > >>>>>>>> > >>>>>>>> I think that under a legal point of view - NOTICE file is > >>>>>>>> included in > >>>>>>>> the artifact - it would be better give another RC and take > >>>>>>>> advantage > >>>>>>>> to fix also PMD violations. > >>>>>>>> > >>>>>>>> Let's wait anyway for the end of the vote and see how thing > >>>>>>>> go, > >>>>>>>> in the > >>>>>>>> meanwhile I fix what you noticed. > >>>>>>>> > >>>>>>>> Thanks a lot, alles gute! > >>>>>>>> Simo > >>>>>>>> > >>>>>>>> http://people.apache.org/~simonetripodi/ > >>>>>>>> http://simonetripodi.livejournal.com/ > >>>>>>>> http://twitter.com/simonetripodi > >>>>>>>> http://www.99soft.org/ > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> On Fri, Oct 21, 2011 at 10:08 PM, Oliver Heger > >>>>>>>> <oliver.he...@oliver-heger.de> wrote: > >>>>>>>>> > >>>>>>>>> Build works fine with Java 1.5 on Windows 7. Artifacts > >>>>>>>>> look good. > >>>>>>>>> > >>>>>>>>> I found the following problems: > >>>>>>>>> - The Copyright in NOTICE.txt says still 2010. > >>>>>>>>> - The release notes claim to be for version 3.0 in the > >>>>>>>>> header. > >>>>>>>>> - There are some PMD errors which could be addressed (not > >>>>>>>>> blocking). > >>>>>>>>> - A bit strange: When I build the site locally I get 75 > >>>>>>>>> Checkstyle errors. > >>>>>>>>> Maybe this depends on the version of the checkstyle plug-in? > >>>>>>>>> > >>>>>>>>> Oliver > >>>>>>>>> > >>>>>>>>> Am 19.10.2011 23:38, schrieb Simone Tripodi: > >>>>>>>>>> > >>>>>>>>>> Hi all guys, > >>>>>>>>>> I'm writing to call for a vote to release apache > >>>>>>>>>> commons-digester-3.1 > >>>>>>>>>> based on RC1. > >>>>>>>>>> > >>>>>>>>>> This is the 3.0 release with the new async feature - not > >>>>>>>>>> very > >>>>>>>>>> much but > >>>>>>>>>> enough to justify a new release. > >>>>>>>>>> Please take in consideration that broken 3.1 links will be > >>>>>>>>>> fixed once > >>>>>>>>>> the site will be deployed. > >>>>>>>>>> > >>>>>>>>>> The vote will stay open for 72 hours anc closes on Saturday > >>>>>>>>>> 22th, at > >>>>>>>>>> 9:40pm CET. > >>>>>>>>>> > >>>>>>>>>> Many thanks in advance for reviewing, have a nice day! > >>>>>>>>>> All the best, > >>>>>>>>>> Simo > >>>>>>>>>> > >>>>>>>>>> Release notes: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > http://people.apache.org/builds/commons/digester/3.1/RC1/RELEASE-NOTES.txt > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> Tag: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > https://svn.apache.org/repos/asf/commons/proper/digester/tags/DIGESTER3_3_1_RC1/ > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> Site: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> http://people.apache.org/builds/commons/digester/3.1/RC1/site/ > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> Binaries: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > http://people.apache.org/builds/commons/digester/3.1/RC1/binaries/ > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> Maven Artifacts (staged on Nexus) > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > https://repository.apache.org/content/repositories/orgapachecommons-080/org/apache/commons/commons-digester3/ > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> [ ] +1 release it > >>>>>>>>>> [ ] +0 go ahead I don't care > >>>>>>>>>> [ ] -1 no, do not release it because... (please explain why) > >>>>>>>>>> > >>>>>>>>>> http://people.apache.org/~simonetripodi/ > >>>>>>>>>> http://simonetripodi.livejournal.com/ > >>>>>>>>>> http://twitter.com/simonetripodi > >>>>>>>>>> http://www.99soft.org/ > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > --------------------------------------------------------------------- > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>>>>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > --------------------------------------------------------------------- > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>>>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>>> > --------------------------------------------------------------------- > >>>>>>> > >>>>>>> > >>>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>>>> > >>>>>> > >>>>>> > >>>>>> > --------------------------------------------------------------------- > >>>>>> > >>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>>> > >>>>>> > >>>>> > >>>>> > >>>>> --------------------------------------------------------------------- > >>>>> > >>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>> > >>>>> > >>>> > >>>> --------------------------------------------------------------------- > >>>> > >>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>> > >>> > >>> > >>> --------------------------------------------------------------------- > >>> > >>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>> For additional commands, e-mail: dev-h...@commons.apache.org > >>> > >>> > >> > >> --------------------------------------------------------------------- > >> > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >> For additional commands, e-mail: dev-h...@commons.apache.org > >> > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
