Sorry about my english, what you mean is that I had to write a test to verify that the bug won't arise again, isn't it? If I'm right I've just committed a test for that.
BTW the assert jelly tab library throws an exception when it's declared in maven.xml <project [...] xmlns:assert="assert"> That's the case of the plugin-test in the ant plugin > -----Original Message----- > From: Dion Gillard [mailto:[EMAIL PROTECTED] > Sent: Sunday, July 04, 2004 2:41 PM > To: Maven Developers List > Subject: Re: cvs commit: maven-plugins/ant/xdocs changes.xml > > No, i meant that you'd 'fixed' something, but we didn't have > a test case to check it against. > There are test cases for the ant plugin, and as someone once > said, "Fix a bug without a test case and you're asking > someone to break it again". > > On Sun, 4 Jul 2004 13:41:53 +0200, Carlos Sanchez > <[EMAIL PROTECTED]> wrote: > > Sorry, do you mean if I've wrotten a test for the changes I'd made? > > > > If that's the question I'm afraid the answer is no. I've > only tested > > it with one of my projects Tell me if test is needed and > I'll do after > > seeing Fernando Alonso win the Magny Cours Formula1 Grand Prix ;) > > > > Carlos Sanchez > > A Coruņa, Spain > > > > Oness Project > > http://oness.sourceforge.net > > > > > > > > > > > -----Original Message----- > > > From: Dion Gillard [mailto:[EMAIL PROTECTED] > > > Sent: Sunday, July 04, 2004 11:42 AM > > > To: Maven Developers List > > > Subject: Re: cvs commit: maven-plugins/ant/xdocs changes.xml > > > > > > Was there a failing plugin test for this? > > > > > > On 3 Jul 2004 17:49:14 -0000, [EMAIL PROTECTED] > <[EMAIL PROTECTED]> > > > wrote: > > > > carlos 2004/07/03 10:49:14 > > > > > > > > Modified: ant/src/plugin-resources/templates build.jelly > > > > ant/xdocs changes.xml > > > > Log: > > > > Use relative paths in directory properties. Issue MPANT-16 > > > > > > > > Revision Changes Path > > > > 1.16 +12 -6 > > > maven-plugins/ant/src/plugin-resources/templates/build.jelly > > > > > > > > Index: build.jelly > > > > > > > > =================================================================== > > > > RCS file: > > > /home/cvs/maven-plugins/ant/src/plugin-resources/templates/bui > > > ld.jelly,v > > > > retrieving revision 1.15 > > > > retrieving revision 1.16 > > > > diff -u -r1.15 -r1.16 > > > > --- build.jelly 18 May 2004 21:16:04 -0000 1.15 > > > > +++ build.jelly 3 Jul 2004 17:49:14 -0000 1.16 > > > > @@ -42,15 +42,21 @@ > > > > > > > > <project name="${pom.artifactId}" default="jar" basedir="."> > > > > > > > > - <property name="defaulttargetdir" > value="${maven.build.dir}"/> > > > > - <property name="libdir" value="${maven.build.dir}/lib" /> > > > > - <property name="classesdir" value="${maven.build.dest}" /> > > > > - <property name="testclassesdir" > value="${maven.test.dest}" /> > > > > + <j:set var="baseDirFile" > > > value="${pom.file.canonicalFile.parentFile}"/> > > > > + <maven:makeRelativePath basedir="${baseDirFile}" > > > path="${maven.build.dir}" separator="/" var="defaulttargetdir"/> > > > > + <property name="defaulttargetdir" > value="${defaulttargetdir}"/> > > > > + <property name="libdir" value="${defaulttargetdir}/lib" /> > > > > + > > > > + <maven:makeRelativePath basedir="${baseDirFile}" > > > path="${maven.build.dest}" separator="/" var="classesdir"/> > > > > + <property name="classesdir" value="${classesdir}" /> > > > > > > > > <j:if test="${unitTestSourcesPresent == 'true'}"> > > > > - <maven:pluginVar plugin="maven-test-plugin" > > > property="maven.test.dest" var="testclassesdir" /> > > > > + <maven:pluginVar plugin="maven-test-plugin" > > > property="maven.test.dest" var="abstestclassesdir" /> > > > > + <maven:makeRelativePath basedir="${baseDirFile}" > > > path="${abstestclassesdir}" separator="/" var="testclassesdir"/> > > > > <property name="testclassesdir" > value="${testclassesdir}" /> > > > > - <maven:pluginVar plugin="maven-test-plugin" > > > property="maven.test.reportsDirectory" var="testreportdir" /> > > > > + > > > > + <maven:pluginVar plugin="maven-test-plugin" > > > property="maven.test.reportsDirectory" var="abstestreportdir" /> > > > > + <maven:makeRelativePath basedir="${baseDirFile}" > > > path="${abstestreportdir}" separator="/" var="testreportdir"/> > > > > <property name="testreportdir" > value="${testreportdir}" /> > > > > </j:if> > > > > > > > > 1.23 +1 -0 maven-plugins/ant/xdocs/changes.xml > > > > > > > > Index: changes.xml > > > > > > > > =================================================================== > > > > RCS file: /home/cvs/maven-plugins/ant/xdocs/changes.xml,v > > > > retrieving revision 1.22 > > > > retrieving revision 1.23 > > > > diff -u -r1.22 -r1.23 > > > > --- changes.xml 18 May 2004 21:29:51 -0000 1.22 > > > > +++ changes.xml 3 Jul 2004 17:49:14 -0000 1.23 > > > > @@ -25,6 +25,7 @@ > > > > </properties> > > > > <body> > > > > <release version="1.8-SNAPSHOT" date="In CVS"> > > > > + <action dev="carlos" type="fix" issue="MPANT-16" > > > due-to="Brent Worden" due-to-email="[EMAIL PROTECTED]">Use > > > relative paths in directory properties.</action> > > > > <action dev="aheritier" type="update">Run Junit > > > only if present in ANT (display a warning otherwise).</action> > > > > </release> > > > > <release version="1.7" date="2004-05-15"> > > > > > > > > > > > > -------------------------------------------------------------------- > > > - > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] For > > > > additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > > -------------------------------------------------------------------- > > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For > > > additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] For > > additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] For > additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
