check for junit in build.xml busted

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

check for junit in build.xml busted

Charlie Groves
Hi,

Glazedlists' build.xml is good enough to check for junit.jar in
ant.home/lib before running the tests and even has a nice error
message about its necessity.  Unfortunately compiletests runs before
test and fails equally badly if junit.jar isn't present.  The attached
patch moves the check into compiletests.

Thanks,
Charlie

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

testForJunitInBuildxml.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: check for junit in build.xml busted

Jesse Wilson
Hey Charles ---

Your patch looks good. It's been applied.
I love getting good patches like this one, thanks!

Cheers,
Jesse


On 7/7/06, Charlie Groves <[hidden email]> wrote:

> Hi,
>
> Glazedlists' build.xml is good enough to check for junit.jar in
> ant.home/lib before running the tests and even has a nice error
> message about its necessity.  Unfortunately compiletests runs before
> test and fails equally badly if junit.jar isn't present.  The attached
> patch moves the check into compiletests.
>
> Thanks,
> Charlie
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]