From: J. R. <jjb...@st...> - 2005-06-07 07:22:46
|
I don't agree. If we are going to implement this solution we will have to document it and people will start using it. It will be harder to change it later because people will already be using the quick fix solution. I think it would be better to take some time to create it the good way (nested filesets). If you support nested filesets you also support references to predefined filesets which was mentioned earlyer. The quick fix for now can be to copy the src folders to one directory and reference this directory in the report task. I have used this solution in one of our projects. Example code: <mkdir dir="${src.all}"/> <copy todir="${src.all}"> <fileset dir="PROJECT1/src"> <include name="**/*.java" /> </fileset> <fileset dir="PROJECT2/src"> <include name="**/*.java" /> </fileset> </copy> <cobertura-report srcdir="${src.all}" destdir="report"/> Best regards, Julien Quoting "Sinke, Mark (GE Healthcare)" <mar...@me...>: > For now, I'd just go with source folders; the way <javac> support > <includes> or <excludes> would be a bonus, but not necessary in a first > iteration, I think. > > Mark. > > -----Original Message----- > From: cob...@li... > [mailto:cob...@li...] On Behalf Of Mark > Doliner > Sent: Monday, June 06, 2005 17:02 > To: Seigel, James; cob...@li... > Subject: RE: [Cobertura-devel] Multiple Source directories > > It seems like it would make the most sense to mimic the javac task. > Then maybe people could just copy and paste parts of javac into > cobertura.report? This would allow you to do something like the > following: > > <cobertura-report destdir="${classes}"> > <src path="${basedir}/src"/> > <src path="${basedir}/examples"/> > <include name="net/sourceforge/cobertura/**"/>> > <exclude name="**/DebuggingHelper*"/> > <exclude name="**/Test*"/> > </javac> > > Is there a reason we would want to do this differently than the javac > task? Would we even need to support the include and exclude elements? > > -Mark > > > -----Original Message----- > > From: cob...@li... > > [mailto:cob...@li...] On Behalf Of J. > > Rentrop > > Sent: Sunday, June 05, 2005 11:49 AM > > To: Seigel, James > > Cc: cob...@li... > > Subject: Re: [Cobertura-devel] Multiple Source directories > > > > Hi, > > > > I think nested filesets would probally be the best. This is similiair > > to the instrument task. > > > > Example: > > <cobertura-report destdir="${coverage.html.dir}"> > > <fileset dir="${src.dir}"> > > <include name="**/*.java" /> > > </fileset> > > <fileset dir="${test.dir}"> > > <include name="**/*.java" /> > > </fileset> > > </cobertura-report> > > > > Best regards, > > Julien > > > > Quoting "Seigel, James" <Jam...@av...>: > > > > > > > > Hello, > > > > > > Sorry for the hiatus. I was wondering if anyone had any > > thoughts on the > > > format to support multiple source directories on the > > coverage-report task? > > > > > > What do you think of this kind of structure? > > > > > > <cobertura-report destdir="${coverage.html.dir}"> > > > <sourcepath location="${src.dir}"/> > > > <sourcepath location="${test.dir}"/> > > > </cobertura-report> > > > > > > > > > Cheers > > > James. > > > > > > > > > ------------------------------------------------------- > > > This SF.Net email is sponsored by: NEC IT Guy Games. How > > far can you > > > shotput > > > a projector? How fast can you ride your desk chair down the > > office luge > > > track? > > > If you want to score the big prize, get to know the little guy. > > > Play to win an NEC 61" plasma display: http://www.necitguy.com/?r > > > _______________________________________________ > > > Cobertura-devel mailing list > > > Cob...@li... > > > https://lists.sourceforge.net/lists/listinfo/cobertura-devel > > > > > > > > > > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: NEC IT Guy Games. How far can you > > shotput a projector? How fast can you ride your desk chair down the > > office luge track? > > If you want to score the big prize, get to know the little guy. > > Play to win an NEC 61" plasma display: http://www.necitguy.com/?r=20 > > _______________________________________________ > > Cobertura-devel mailing list > > Cob...@li... > > https://lists.sourceforge.net/lists/listinfo/cobertura-devel > > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: NEC IT Guy Games. How far can you > shotput > a projector? How fast can you ride your desk chair down the office luge > track? > If you want to score the big prize, get to know the little guy. > Play to win an NEC 61" plasma display: http://www.necitguy.com/?r > _______________________________________________ > Cobertura-devel mailing list > Cob...@li... > https://lists.sourceforge.net/lists/listinfo/cobertura-devel > > > ------------------------------------------------------- > This SF.Net email is sponsored by: NEC IT Guy Games. How far can you > shotput > a projector? How fast can you ride your desk chair down the office luge > track? > If you want to score the big prize, get to know the little guy. > Play to win an NEC 61" plasma display: http://www.necitguy.com/?r > _______________________________________________ > Cobertura-devel mailing list > Cob...@li... > https://lists.sourceforge.net/lists/listinfo/cobertura-devel > |