You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by Christian Trutz <ch...@smilebase.org> on 2003/08/11 21:42:48 UTC

Re: cvs commit: incubator-geronimo maven.xml

hi commiters,

thanx for the "run" goal!

the "modules/core/src/conf" path should be also added to the test classpath,
i am writing some junit test cases and log4j.properties is not loaded 
automatically.


chris

AW: AW: AW: cvs commit: incubator-geronimo maven.xml

Posted by Steffen Schluff <sc...@oio.de>.
Hello James,

allright, obviously not ;-)

Will do that right after lunch,

thanks again,

Steffen


-----Ursprungliche Nachricht-----
Von: James Strachan [mailto:james_strachan@yahoo.co.uk]
Gesendet: Dienstag, 12. August 2003 13:10
An: geronimo-dev@incubator.apache.org
Betreff: Re: AW: AW: cvs commit: incubator-geronimo maven.xml



On Tuesday, August 12, 2003, at 12:15  pm, Steffen Schluff wrote:

> Hello James,
>
> the suggestion was based on the idea that some people might build 
> geronimo
> in an IDE which would most likely force them to setup an IDE specific
> classpath and in that respect explicitly stating every dependency 
> (including
> junit and the respective version in that special case) seems like a 
> good
> idea to me.
>
> But as I was saying just a suggestion ;-)

Have you tried using the Maven plugins for your IDE which 
auto-generates the project definition & fills in all the dependent jars 
for you?

James
-------
http://radio.weblogs.com/0112098/


Re: AW: AW: cvs commit: incubator-geronimo maven.xml

Posted by James Strachan <ja...@yahoo.co.uk>.
On Tuesday, August 12, 2003, at 12:15  pm, Steffen Schluff wrote:

> Hello James,
>
> the suggestion was based on the idea that some people might build 
> geronimo
> in an IDE which would most likely force them to setup an IDE specific
> classpath and in that respect explicitly stating every dependency 
> (including
> junit and the respective version in that special case) seems like a 
> good
> idea to me.
>
> But as I was saying just a suggestion ;-)

Have you tried using the Maven plugins for your IDE which 
auto-generates the project definition & fills in all the dependent jars 
for you?

James
-------
http://radio.weblogs.com/0112098/


AW: AW: cvs commit: incubator-geronimo maven.xml

Posted by Steffen Schluff <sc...@oio.de>.
Hello James,

the suggestion was based on the idea that some people might build geronimo
in an IDE which would most likely force them to setup an IDE specific
classpath and in that respect explicitly stating every dependency (including
junit and the respective version in that special case) seems like a good
idea to me.

But as I was saying just a suggestion ;-)

Cheers,

Steffen

*snip*

> - Wouldn't it be nice to list Junit as dependency (even though not
> really
> required for maven) ?

Why?



Re: AW: cvs commit: incubator-geronimo maven.xml

Posted by James Strachan <ja...@yahoo.co.uk>.
On Tuesday, August 12, 2003, at 09:26  am, Steffen Schluff wrote:

> Hello James, hello all,
>
> I have been playing with the project.xml again and noticed two things:
>
> - Shouldn't the unitTests also include **/*Test.java instead of only
> **/*TestCase.java ?

Have used the more common notation of **/*Test.java to follow all the 
existing test cases. Now fixed, thanks.


> - Wouldn't it be nice to list Junit as dependency (even though not 
> really
> required for maven) ?

Why?

>
> Patch is attached.

Thanks - patches always welcome! :)

James
-------
http://radio.weblogs.com/0112098/


AW: cvs commit: incubator-geronimo maven.xml

Posted by Steffen Schluff <sc...@oio.de>.
Hello James, hello all,

I have been playing with the project.xml again and noticed two things:

- Shouldn't the unitTests also include **/*Test.java instead of only
**/*TestCase.java ?
- Wouldn't it be nice to list Junit as dependency (even though not really
required for maven) ?

Patch is attached.

Thanks,

Steffen Schluff

-----Ursprungliche Nachricht-----
Von: James Strachan [mailto:james_strachan@yahoo.co.uk]
Gesendet: Dienstag, 12. August 2003 10:01
An: geronimo-dev@incubator.apache.org
Betreff: Re: cvs commit: incubator-geronimo maven.xml



On Monday, August 11, 2003, at 08:42  pm, Christian Trutz wrote:

> hi commiters,
>
> thanx for the "run" goal!
>
> the "modules/core/src/conf" path should be also added to the test
> classpath,
> i am writing some junit test cases and log4j.properties is not loaded
> automatically.

Its fixed now - the new Main doesn't require any command line arguments.

James
-------
http://radio.weblogs.com/0112098/

Re: cvs commit: incubator-geronimo maven.xml

Posted by James Strachan <ja...@yahoo.co.uk>.
On Monday, August 11, 2003, at 08:42  pm, Christian Trutz wrote:

> hi commiters,
>
> thanx for the "run" goal!
>
> the "modules/core/src/conf" path should be also added to the test 
> classpath,
> i am writing some junit test cases and log4j.properties is not loaded
> automatically.

Its fixed now - the new Main doesn't require any command line arguments.

James
-------
http://radio.weblogs.com/0112098/