Re: [xwiki-devs] Problems building xwiki-core (checkstyle violations)

Hi All, I'm really sorry about having to trouble like this. I was going to fix the checkstyle violations in xwiki-core but now i get the following error, [asiri@asiri-desktop xwiki-core]$ mvn checkstyle:check [INFO] Scanning for projects... [INFO] ------------------------------------------------------------------------ [INFO] Building XWiki Platform - Core - Core [INFO] task-segment: [checkstyle:check] [INFO] ------------------------------------------------------------------------ [INFO] [checkstyle:check] [INFO] Unable to perform checkstyle:check, unable to find checkstyle:checkstyle outputFile. [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESSFUL [INFO] ------------------------------------------------------------------------ [INFO] Total time: 1 second [INFO] Finished at: Thu Apr 03 21:35:47 LKT 2008 [INFO] Final Memory: 2M/9M [INFO] ------------------------------------------------------------------------ [asiri@asiri-desktop xwiki-core]$ What do i have to do so that checkstyle will function again ?

Thanks. - Asiri PS: I have verified that i'm using the correct version of maven. On Thu, Apr 3, 2008 at 8:46 PM, sachin mittal &lt;sjmittal(a)gmail.com&gt; wrote: > Use -fae flag. There is no direct command to skip checkstyle, but > using > -fae > the build would in the end only so you would have got your jars and > wars > ready. > > Thanks > Sachin > > >> ------------------------------ >> >> Message: 2 >> Date: Thu, 3 Apr 2008 20:13:26 +0530 >> From: "Asiri Rathnayake" &lt;asiri.rathnayake(a)gmail.com&gt; >> Subject: Re: [xwiki-devs] Problems building xwiki-core (checkstyle >> violations) >> To: "XWiki Developers" &lt;devs(a)xwiki.org&gt; >> Message-ID: >> &lt;9fd36c290804030743y426e9999mdfa5b788c8f1f418(a)mail.gmail.com&gt; >> Content-Type: text/plain; charset=ISO-8859-1 >> >> On Thu, Apr 3, 2008 at 8:07 PM, Vincent Massol &lt;vincent(a)massol.net&gt; > wrote: >> >>> >>> On Apr 3, 2008, at 4:31 PM, Asiri Rathnayake wrote: >>> >>>> On Thu, Apr 3, 2008 at 7:52 PM, Vincent Massol >>>> &lt;vincent(a)massol.net&gt; >>>> wrote: >>>> >>>>> >>>>> On Apr 3, 2008, at 4:11 PM, Asiri Rathnayake wrote: >>>>> >>>>>> I'm having a real hard-time with these check-style issues. Can >>>>>> someone fix >>>>>> these issues ? Or perhaps tell me how to disable check-style in >>>>>> maven ? I >>>>>> just need to do a simple test, now i'm spending more time on > getting >>>>>> over >>>>>> these check-style issues :( >>>>> >>>>> I can see 3 solutions: >>>>> >>>>> 1) you can google for it... (to learn how to skip checkstyle) >>>> >>>> >>>> I did, no luck .. :) >>> >>> try with "skip checkstyle maven".... The first link is good... >>> >> >> Ok, my bad. :) >> >> Thanks. >> >> - Asiri >> >> >>> >>>>> >>>>> 2) you could use Maven 2.1 snapshot (which si the RECOMMENDED >>>>> version) >>>> >>>> >>>> I'm using it. Still get the same problem. >>> >>> I don't see how this is possible since: >>> 1) I don't get the pb >>> 2) Continuum doesn't get the problem >>> 3) nobody in the dev team is getting the problem >>> >>> To verify , just type mvn --help >>> >>>>> 3) you could fix the checkstyle errors >>>>> >>>> >>>> Ok, I'll work on this ;) >>> >>> cool >>> >>>>> If you're spending time on this it's because you haven't read the >>>>> Building page which says to use Maven 2.1 snapshot attached. So >>>>> all >>>>> your fault... :) >>>>> >>>> >>>> This is what i don't understand, I am using this version >>>> *doubtful*. >>>> >>>> >>>>> >>>>> Anyway if you want to play nice/team spirit, finding how to make >>>>> checkstyle work in 2.1-snapshot would be the best. >>>>> >>>> >>>> I don't understand what you meant here. Fix the check-style >>>> mistakes >>>> or ... >>>> ? >>> >>> Best is to make checkstyle work in Maven 2.1-snapshot. To do >>> this, you >>> need to ping the maven dev team on their mailing list, on IRC, >>> search >>> the Maven JIRA, etc. >>> >>> Thanks >>> -Vincent >>> >>>> Anyway, I got through the problem in following manner, >>>> >>>> do `mvn package` and then copy the created jar file into local >>>> repository >>>> ... smart eh ? ;-) >>>> >>>> Maven executes check-style verification after building the jar, so > no >>>> problems .. >>>> >>>> Anyway, I'll work on fixing the check-style violations. >>>> >>>> Thanks all for the replies. >>>> >>>> - Asiri >>>> >>>> >>>>> >>>>> Thanks >>>>> -Vincent >>>>> >>>>>> On Thu, Apr 3, 2008 at 7:30 PM, Thomas Mortagne < >>>>> thomas.mortagne(a)xwiki.com >>>>>>> >>>>>> wrote: >>>>>> >>>>>>> On Thu, Apr 3, 2008 at 3:55 PM, Asiri Rathnayake >>>>>>> &lt;asiri.rathnayake(a)gmail.com&gt; wrote: >>>>>>>> I thought there was a way to fix these check-style violations >>>>>>>> within >>>>>>>> eclipse, Do we have that eclipse check-style definition file ? >>>>>>>> Anyway, >>>>>>> I'll >>>>>>>> try to find it out. >>>>>>> >>>>>>> See >>>>>>> >>>>> >>> >> >

>>>>>>> >>>>>>>> >>>>>>>> Thanks. >>>>>>>> >>>>>>>> - Asiti >>>>>>>> >>>>>>>> On Thu, Apr 3, 2008 at 7:06 PM, Vincent Massol >>>>>>>> &lt;vincent(a)massol.net&gt; >>>>>>> wrote: >>>>>>>> >>>>>>>>> >>>>>>>>> On Apr 3, 2008, at 3:23 PM, Asiri Rathnayake wrote: >>>>>>>>> >>>>>>>>>> Some complaints seems to be somewhat pointless; like the > string >>>>>>> "date" >>>>>>>>>> appearing twice in the file. >>>>>>>>> >>>>>>>>> None is pointless. See >>>>>>>>> http://checkstyle.sourceforge.net/availablechecks.html >>>>>>>>> >>>>>>>>>> Is it suggesting that we should use a publlic static final >>>>>>>>>> String >>>>>>>>>> field >>>>>>>>>> instead ? >>>>>>>>> >>>>>>>>> yes or some other solution depending on the case at hand >>>>>>>>> (like >>>>>>>>> some >>>>>>>>> resource bundles, etc). >>>>>>>>> >>>>>>>>> -Vincent >>>>>>>>> >>>>>>>>>> On Thu, Apr 3, 2008 at 6:34 PM, Vincent Massol >>>>>>>>>> &lt;vincent(a)massol.net&gt; >>>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>>> You're using Maven 2.0.x (which works). The problem is that >>>>>>>>>>> Maven >>>>>>>>>>> 2.1.x has a bug where the checkstyle plugin is NOT working > and >>>>>>> thus >>>>>>>>>>> these checkstyle errors have been introduced in the >>>>>>>>>>> meantime. >>>>>>>>>>> >>>>>>>>>>> We need to: >>>>>>>>>>> 1) fix them >>>>>>>>>>> 2) probably use version 2.0.9 as our official version (at > least >>>>>>> make >>>>>>>>>>> sure our C instance is using it), unless we can find a >>>>>>>>>>> way to >>>>>>>>>>> get >>>>>>>>>>> checkstyle working in 2.1. >>>>>>>>>>> >>>>>>>>>>> I've known this for some time now but haven't had the >>>>>>>>>>> time to >>>>>>>>>>> fix >>>>>>> it. >>>>>>>>>>> If anyone wants to fix it that'd help. >>>>>>>>>>> >>>>>>>>>>> Thanks >>>>>>>>>>> -Vincent >>>>>>>>>>> >>>>>>>>>>> On Apr 3, 2008, at 2:57 PM, Asiri Rathnayake wrote: >>>>>>>>>>> >>>>>>>>>>>> To reproduce, goto /trunks/xwiki-platform-core/xwiki-core > and >>>>>>>>>>>> execute `mvn >>>>>>>>>>>> install` >>>>>>>>>>>> >>>>>>>>>>>> i get the following, >>>>>>>>>>>> >>>>>>>>>>>> <DUMP> >>>>>>>>>>>> >>>>>>>>>>>> [INFO] Starting audit... >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:110: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:256: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> > src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:258:27: >>>>>>>>>>>> The String "Failed to load internationalization document >>>>>>>>>>>> bundle >>>>>>> [" >>>>>>>>>>>> appears 2 >>>>>>>>>>>> times in the file. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> > src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:259:23: >>>>>>>>>>>> The String "]." appears 2 times in the file. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> > src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:276:64: >>>>>>>>>>>> Expected @param tag for 'defaultLanguage'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java: >>>>>>>>>>>> 278:9: >>>>>>>>>>>> Declaring variables, return values or parameters of type >>>>>>> 'ArrayList' >>>>>>>>>>>> is not >>>>>>>>>>>> allowed. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java:297: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/web/XWikiMessageTool.java: >>>>>>>>>>>> 307:5: >>>>>>>>>>>> Missing a Javadoc comment. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> >>>>>>> src/main/java/com/xpn/xwiki/plugin/fileupload/ >>>>>>> FileUploadPlugin.java: >>>>>>>>>>>> 236: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/plugin/zipexplorer/ >>>>>>>>>>>> ZipExplorerPlugin.java:289: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/plugin/zipexplorer/ >>>>>>>>>>>> ZipExplorerPlugin.java:315: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/plugin/zipexplorer/ >>>>>>>>>>>> ZipExplorerPlugin.java:319: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/plugin/zipexplorer/ >>>>>>>>>>>> ZipExplorerPlugin.java:325: >>>>>>>>>>>> warning: Comment matches to-do format 'TODO:'. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/store/hibernate/ >>>>>>>>>>>> HibernateAttachmentRecycleBinStore.java:32:8: >>>>>>>>>>>> Unused import - com.xpn.xwiki.XWiki. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/store/hibernate/ >>>>>>>>>>>> HibernateAttachmentRecycleBinStore.java:125:31: >>>>>>>>>>>> The String "unchecked" appears 2 times in the file. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/store/hibernate/ >>>>>>>>>>>> HibernateAttachmentRecycleBinStore.java:131:43: >>>>>>>>>>>> The String "docId" appears 2 times in the file. >>>>>>>>>>>> /home/asiri/projects/xwiki-dev/trunks/xwiki-platform-core/ >>>>>>>>>>>> xwiki- >>>>>>>>>>>> core/ >>>>>>>>>>>> src/main/java/com/xpn/xwiki/store/hibernate/ >>>>>>>>>>>> HibernateAttachmentRecycleBinStore.java:136:69: >>>>>>>>>>>> The String "date" appears 2 times in the file. >>>>>>>>>>>> Audit done. >>>>>>>>>>>> >>>>>>>>>>>> [INFO] There are 9 checkstyle errors. >>>>>>>>>>>> [WARNING] Unable to locate Source XRef to link to - >>>>>>>>>>>> DISABLED >>>>>>>>>>>> [INFO] Ending forked execution [fork id: 341048484] >>>>>>>>>>>> [ERROR] >>>>>>>>>>>> >>>>>>>>>>>> Mojo: >>>>>>>>>>>> >>>>>>>>>>>> org.apache.maven.plugins:maven-checkstyle-plugin:2.1:check >>>>>>>>>>>> >>>>>>>>>>>> FAILED for project: >>>>>>>>>>>> >>>>>>>>>>>> com.xpn.xwiki.platform:xwiki-core:jar:1.4-SNAPSHOT >>>>>>>>>>>> >>>>>>>>>>>> Reason: >>>>>>>>>>>> >>>>>>>>>>>> You have 9 checkstyle violations. >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> [INFO] >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>>> >>> >

>>>>>>>>>>>> [INFO] For more information, run with the -e flag >>>>>>>>>>>> [INFO] >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>>> >>> >

>>>>>>>>>>>> [INFO] BUILD FAILED >>>>>>>>>>>> [INFO] >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>>> >>> >

>>>>>>>>>>>> [INFO] Total time: 1 minute 6 seconds >>>>>>>>>>>> [INFO] Finished at: Thu Apr 03 18:18:24 LKT 2008 >>>>>>>>>>>> [INFO] Final Memory: 9M/24M >>>>>>>>>>>> [INFO] >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>>> >>> >

_______________________________________________ devs mailing list devs(a)xwiki.org http://lists.xwiki.org/mailman/listinfo/devs