> -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]] > Sent: 24 May 2002 03:22 > To: Turbine Maven Developers List > Subject: Re: Checkstyle problem > > > How about we just make the path -> filelist happen in the maven classes > that set the properties up, and add this as a new property. This bit one > of > my guys yesterday, so I agree this needs to be done.
Yes, good idea. I'm still not familiar with the way Maven works I guess. Thanks. I'll try to do this today and send a patch. Cheers, -Vincent > -- > dIon Gillard, Multitask Consulting > Work: http://www.multitask.com.au > Developers: http://adslgateway.multitask.com.au/developers > > > > "Vincent > Massol" To: <turbine-maven- > [EMAIL PROTECTED]> > <vmassol@octo. cc: > com> Subject: Checkstyle problem > > 05/23/02 08:58 > PM > Please respond > to "Turbine > Maven > Developers > List" > > > > > > > Hi, > > The checkstyle task uses ${maven.src.dir} as its fileset. The problem > with this are : > > 1/ If you have the following directory structure : > > src > |_ java > |_ test > > then the checkstyle reports give file name prefixed with java/ or test/ > > but more importantly > > 2/ it breaks the xref link (on the line number) as xref uses the > maven.src.set sourcepathref (which I imagine come from the POM) to > generate its html files. > > Solution: > > Use maven.src.set in the checkstyle task. However, we need to transform > maven.src.set into a patternset to be able to use it in a fileset ... > > A solution I can see is to write an Ant task that does this. Something > like : > > <path2filelist patternsetid="new pattern set id"> > <path ...> > </path2filelist> > > where <path...> accepts a pathid attribute referring to an already > defined path set. > > I'm sure there are other ways (especially with Ant 1.5). Do you know any > other ? > > Thanks > -Vincent > > > -- > To unsubscribe, e-mail: < > mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: < > mailto:[EMAIL PROTECTED]> > > > > > > > -- > To unsubscribe, e-mail: <mailto:turbine-maven-dev- > [EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:turbine-maven-dev- > [EMAIL PROTECTED]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
