Re: svn commit: r1133357 - /ofbiz/trunk/build.xml

[Scott Gray-2]

Hi Erwan,

I'm no ant expert but since run-tests already depends on build I'm not sure that your new target needs to as well (build might be run twice).

Regards
Scott

On 8/06/2011, at 11:52 PM, [hidden email] wrote:

> Author: erwan
> Date: Wed Jun  8 11:52:13 2011
> New Revision: 1133357
>
> URL: http://svn.apache.org/viewvc?rev=1133357&view=rev
> Log:
> Spelling mistake in run-tests-with-cobertura target. Thanks Adrian
>
> Modified:
>    ofbiz/trunk/build.xml
>
> Modified: ofbiz/trunk/build.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/build.xml?rev=1133357&r1=1133356&r2=1133357&view=diff
> ==============================================================================
> --- ofbiz/trunk/build.xml (original)
> +++ ofbiz/trunk/build.xml Wed Jun  8 11:52:13 2011
> @@ -705,7 +705,7 @@ under the License.
>         </fail>
>     </target>
>
> -   <target name="run-tests-with-corbertura" depends="download-cobertura, build, run-tests"
> +   <target name="run-tests-with-cobertura" depends="download-cobertura, build, run-tests"
>         description="Same as run-tests but downloads cobertura and performs code coverage. You will need a valid Internet connection to download cobertura"/>
>
>     <target name="_check-separated-tests-already-setup">
>
>