Re: svn commit: r432773 - /geronimo/trunk/modules/geronimo-testsupport/src/main/java/org/apache/geronimo/testsupport/TestSupport.java

[Jason Dillon]

Ya, I will add that as soon as I can verify that it works in IDEA an  
Eclipse... probably sometime next week.

--jason

On Aug 18, 2006, at 4:31 PM, Gianny Damour wrote:

On 19/08/2006, at 9:15 AM, [EMAIL PROTECTED] wrote:

Author: jdillon
Date: Fri Aug 18 16:15:53 2006
New Revision: 432773

URL: http://svn.apache.org/viewvc?rev=432773&view=rev
Log:
Comments about BASEDIR and IDE magic

Modified:
    geronimo/trunk/modules/geronimo-testsupport/src/main/java/org/ 
apache/geronimo/testsupport/TestSupport.java

Modified: geronimo/trunk/modules/geronimo-testsupport/src/main/ 
java/org/apache/geronimo/testsupport/TestSupport.java
URL: http://svn.apache.org/viewvc/geronimo/trunk/modules/geronimo- 
testsupport/src/main/java/org/apache/geronimo/testsupport/ 
TestSupport.java?rev=432773&r1=432772&r2=432773&view=diff
===================================================================== 
=========
--- geronimo/trunk/modules/geronimo-testsupport/src/main/java/org/ 
apache/geronimo/testsupport/TestSupport.java (original)
+++ geronimo/trunk/modules/geronimo-testsupport/src/main/java/org/ 
apache/geronimo/testsupport/TestSupport.java Fri Aug 18 16:15:53 2006
@@ -37,6 +37,12 @@
     private static final File BASEDIR;

     static {
+        //
+        // TODO: Add some special magic here to figure this out  
when we are running from
+        //       and IDE, like IDEA or Eclipse.  user.dir/target  
might work... but need
+        //       to validate what env each IDE has this set to.
+        //
+
         String tmp = System.getProperty("basedir");

Hi Jason,

This is the trick that Dain has suggested:

    private static final tmp = System.getProperty("basedir",  
System.getProperty("user.dir"));


Thanks,
Gianny



         if (tmp == null) {
             throw new Error("Missing 'basedir' property; tests  
need this property set to run correctly");