Return-Path: 
 <axis-dev-return-11617-apmail-xml-axis-dev-archive=xml.apache.org@xml.apache.org>
Delivered-To: apmail-xml-axis-dev-archive@xml.apache.org
Received: (qmail 58539 invoked by uid 500); 22 Jul 2002 18:12:44 -0000
Mailing-List: contact axis-dev-help@xml.apache.org; run by ezmlm
Precedence: bulk
Reply-To: axis-dev@xml.apache.org
list-help: <mailto:axis-dev-help@xml.apache.org>
list-unsubscribe: <mailto:axis-dev-unsubscribe@xml.apache.org>
list-post: <mailto:axis-dev@xml.apache.org>
Delivered-To: mailing list axis-dev@xml.apache.org
Received: (qmail 58530 invoked from network); 22 Jul 2002 18:12:43 -0000
Subject: Re: [axis] VOTE: drop redesigned test and samples process
To: axis-dev@xml.apache.org
X-Mailer: Lotus Notes Build V60_M13_04302002 Pre-release 2 April 30, 2002
Message-ID: 
 <OFC1C4C38B.A00ECE05-ON85256BFE.0063FB33-86256BFE.0063F1D6@us.ibm.com>
From: mseibert@us.ibm.com
Date: Mon, 22 Jul 2002 13:12:46 -0500
X-MIMETrack: Serialize by Router on D04NM206/04/M/IBM(Build M13TT_07122002
 Pre-release 2|July
 12, 2002) at 07/22/2002 14:12:46
MIME-Version: 1.0
Content-type: multipart/related;
	Boundary="0__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3"
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

--0__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3
Content-type: multipart/alternative;
	Boundary="1__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3"

--1__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3
Content-type: text/plain; charset=US-ASCII

Steve,

This is great feedback.  I will start on most of these suggestions as pre-
PHASE2 work, so that when we get to that point in the next week or so,
these suggestions can be realized.

Thanks again!

Matt Seibert                                           mseibert@us.ibm.com
IBM        External:    (512) 838-3656      Internal:   678-3656


                                                                                                                             
                      "Steve Loughran"                                                                                       
                      <steve_l@iseran.         To:       "axis-dev" <axis-dev@xml.apache.org>                                
                      com>                     cc:                                                                           
                                               Subject:  Re: [axis] VOTE: drop redesigned test and samples process           
                      07/22/2002 12:26                                                                                       
                      Please respond to                                                                                      
                      axis-dev                                                                                               
                                                                                                                             




----- Original Message -----
From: <butek@us.ibm.com>
To: <axis-dev@xml.apache.org>
Sent: Monday, July 22, 2002 8:18 AM
Subject: Re: [axis] VOTE: drop redesigned test and samples process


> Folks, for those interested, I've dropped Matt's work onto http://xml.
> apache.org/~butek/test.tar.  Once you grab and untar that file, take a
look
> at AxisTestRedesign.html.  We'll commit his phase 1 stuff today.
>
> Russell Butek
> butek@us.ibm.com


I like ot; it is much cleaner and much lower maintenance. I think it should
go through one more iteration, for even better isolation and scaling. but
what is there right now is a lot better than before.

1. properties.xml
I'd use <property location> a lot more often over <property value>, as it
does make a difference on big build files. In particular a lot of stuff
seems to be defined relative to axis/java, but when included in some sub
build file, they will be coming in relative to tests/something. So where
there is something like

<property name="build.webapp" value="build/webapps/axis" />

it really will translate to

test/something/build/webapps/axis

This only happens if you call <ant> with the build directory set to the
directory of the test
    <ant dir="test/something" />
rather than the directory of axis base
    <ant antfile="test/something" />

Historically in axis the build files have all done the latter, but IMO it
isnt the right approach for a very large build as it confuses all the code
in that build directory as to where things like the src dir is, and it
stops
you running the builds standalone.

Proposed change:

**we get used to running build files in the directory they live, and modify
the properties files and build files accordingly. **

The changes will be in buildcompile and buildtest.xml

Things like
  <target name="RFCDispatch" depends="setenv">
    <ant inheritAll="true" antfile="test/RFCDispatch/buildComponent.xml"/>
  </target>

would become

  <target name="RFCDispatch" depends="setenv">
    <ant inheritAll="true" antfile="buildComponent.xml"
dir="test/RFCDispatch" />
  </target>

to carry the same trick off in the <foreach> task would need use of the
<dirname> and <basename> tasks that split a filename up.

------
Returning to the properties file I'd define all these things as locations
relative to a base directory

  <property name="masterbuild.dir" location=".."/>

pull in stuff from that directly
  <property file="${masterbuild.dir}/build.properties"/>

or from dirs beneath it

<property name="lib.dir" location="${masterbuild.dir}/lib"/>
<property name="build.dir" location="${masterbuild.dir}/build"/>
<property name="junit.jar" value="${lib.dir}/junit.jar" />
<property name="build.webapp" value="${build.dir}/webapps/axis" />

the use of explicit properties everywhere makes it much easier to build
something out the axis tree altogether, or in a different place in the
tree.

This change will work regardless of how the sub projects are being called;
indeed, with masterbuild.dir defined to a location in the toplevel file,
all
the references will fall out automatically.

--------

Also in path_refs.xml; use of env.ANT_HOME
 -it is only optional to have ANT_HOME defined as an env variable; if it
aint set ant.{bat,sh,pl} sets it to the exe dir/.. , and passes it down as
the property ant.home
 -but IDEs (like intellij) dont set ant.home

so really properties.xml should have the line
 <property name="ant.home" value="env.ANT_HOME" /> to overwrite ant.home
only when it is not defined already




--1__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3
Content-type: text/html; charset=US-ASCII
Content-Disposition: inline

<html><body>Steve,<br>
<br>
This is great feedback.  I will start on most of these suggestions as pre-PHASE2 work, so that when we get to that point in the next week or so, these suggestions can be realized.<br>
<br>
Thanks again!<br>
<br>
Matt Seibert                                           mseibert@us.ibm.com<br>
IBM        External:    (512) 838-3656      Internal:   678-3656<br>
<img src="cid:10__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com" width="16" height="16" alt="">&quot;Steve Loughran&quot; &lt;steve_l@iseran.com&gt;<br>
<br>
<br>

<table V5DOTBL=true width="100%" border="0" cellspacing="0" cellpadding="0">
<tr valign="top"><td width="1%"><img src="cid:20__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com" border="0" height="1" width="72" alt=""><br>
</td><td style="background-image:url(cid:30__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com); background-repeat: no-repeat; " width="1%"><img src="cid:20__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com" border="0" height="1" width="225" alt=""><br>

<ul>
<ul>
<ul>
<ul><b><font size="2">&quot;Steve Loughran&quot; &lt;steve_l@iseran.com&gt;</font></b>
<p><font size="2">07/22/2002 12:26</font><br>
<font size="2">Please respond to axis-dev</font></ul>
</ul>
</ul>
</ul>
</td><td width="100%"><img src="cid:20__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com" border="0" height="1" width="1" alt=""><br>
<font size="1" face="Arial">	</font><br>
<font size="2">	To:	</font><font size="2">&quot;axis-dev&quot; &lt;axis-dev@xml.apache.org&gt;</font><br>
<font size="2">	cc:	</font><br>
<font size="2">	Subject:	</font><font size="2">Re: [axis] VOTE: drop redesigned test and samples process</font><br>
<br>
<font size="1" face="Arial">       </font></td></tr>
</table>
<br>
<tt><br>
----- Original Message -----<br>
From: &lt;butek@us.ibm.com&gt;<br>
To: &lt;axis-dev@xml.apache.org&gt;<br>
Sent: Monday, July 22, 2002 8:18 AM<br>
Subject: Re: [axis] VOTE: drop redesigned test and samples process<br>
<br>
<br>
&gt; Folks, for those interested, I've dropped Matt's work onto </tt><tt>http://xml</tt><tt>.<br>
&gt; apache.org/~butek/test.tar. &nbsp;Once you grab and untar that file, take a<br>
look<br>
&gt; at AxisTestRedesign.html. &nbsp;We'll commit his phase 1 stuff today.<br>
&gt;<br>
&gt; Russell Butek<br>
&gt; butek@us.ibm.com<br>
<br>
<br>
I like ot; it is much cleaner and much lower maintenance. I think it should<br>
go through one more iteration, for even better isolation and scaling. but<br>
what is there right now is a lot better than before.<br>
<br>
1. properties.xml<br>
I'd use &lt;property location&gt; a lot more often over &lt;property value&gt;, as it<br>
does make a difference on big build files. In particular a lot of stuff<br>
seems to be defined relative to axis/java, but when included in some sub<br>
build file, they will be coming in relative to tests/something. So where<br>
there is something like<br>
<br>
&lt;property name=&quot;build.webapp&quot; value=&quot;build/webapps/axis&quot; /&gt;<br>
<br>
it really will translate to<br>
<br>
test/something/build/webapps/axis<br>
<br>
This only happens if you call &lt;ant&gt; with the build directory set to the<br>
directory of the test<br>
 &nbsp; &nbsp;&lt;ant dir=&quot;test/something&quot; /&gt;<br>
rather than the directory of axis base<br>
 &nbsp; &nbsp;&lt;ant antfile=&quot;test/something&quot; /&gt;<br>
<br>
Historically in axis the build files have all done the latter, but IMO it<br>
isnt the right approach for a very large build as it confuses all the code<br>
in that build directory as to where things like the src dir is, and it stops<br>
you running the builds standalone.<br>
<br>
Proposed change:<br>
<br>
**we get used to running build files in the directory they live, and modify<br>
the properties files and build files accordingly. **<br>
<br>
The changes will be in buildcompile and buildtest.xml<br>
<br>
Things like<br>
 &nbsp;&lt;target name=&quot;RFCDispatch&quot; depends=&quot;setenv&quot;&gt;<br>
 &nbsp; &nbsp;&lt;ant inheritAll=&quot;true&quot; antfile=&quot;test/RFCDispatch/buildComponent.xml&quot;/&gt;<br>
 &nbsp;&lt;/target&gt;<br>
<br>
would become<br>
<br>
 &nbsp;&lt;target name=&quot;RFCDispatch&quot; depends=&quot;setenv&quot;&gt;<br>
 &nbsp; &nbsp;&lt;ant inheritAll=&quot;true&quot; antfile=&quot;buildComponent.xml&quot;<br>
dir=&quot;test/RFCDispatch&quot; /&gt;<br>
 &nbsp;&lt;/target&gt;<br>
<br>
to carry the same trick off in the &lt;foreach&gt; task would need use of the<br>
&lt;dirname&gt; and &lt;basename&gt; tasks that split a filename up.<br>
<br>
------<br>
Returning to the properties file I'd define all these things as locations<br>
relative to a base directory<br>
<br>
 &nbsp;&lt;property name=&quot;masterbuild.dir&quot; location=&quot;..&quot;/&gt;<br>
<br>
pull in stuff from that directly<br>
 &nbsp;&lt;property file=&quot;${masterbuild.dir}/build.properties&quot;/&gt;<br>
<br>
or from dirs beneath it<br>
<br>
&lt;property name=&quot;lib.dir&quot; location=&quot;${masterbuild.dir}/lib&quot;/&gt;<br>
&lt;property name=&quot;build.dir&quot; location=&quot;${masterbuild.dir}/build&quot;/&gt;<br>
&lt;property name=&quot;junit.jar&quot; value=&quot;${lib.dir}/junit.jar&quot; /&gt;<br>
&lt;property name=&quot;build.webapp&quot; value=&quot;${build.dir}/webapps/axis&quot; /&gt;<br>
<br>
the use of explicit properties everywhere makes it much easier to build<br>
something out the axis tree altogether, or in a different place in the tree.<br>
<br>
This change will work regardless of how the sub projects are being called;<br>
indeed, with masterbuild.dir defined to a location in the toplevel file, all<br>
the references will fall out automatically.<br>
<br>
--------<br>
<br>
Also in path_refs.xml; use of env.ANT_HOME<br>
 -it is only optional to have ANT_HOME defined as an env variable; if it<br>
aint set ant.{bat,sh,pl} sets it to the exe dir/.. , and passes it down as<br>
the property ant.home<br>
 -but IDEs (like intellij) dont set ant.home<br>
<br>
so really properties.xml should have the line<br>
 &lt;property name=&quot;ant.home&quot; value=&quot;env.ANT_HOME&quot; /&gt; to overwrite ant.home<br>
only when it is not defined already<br>
<br>
<br>
<br>
</tt></body></html>

--1__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3--


--0__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3
Content-type: image/gif;
	name="graycol.gif"
Content-Disposition: inline; filename="graycol.gif"
Content-ID: <10__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com>
Content-transfer-encoding: base64

R0lGODlhEAAQAKECAMzMzAAAAP///wAAACH5BAEAAAIALAAAAAAQABAAAAIXlI+py+0PopwxUbpu
ZRfKZ2zgSJbmSRYAIf4fT3B0aW1pemVkIGJ5IFVsZWFkIFNtYXJ0U2F2ZXIhAAA7

--0__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3
Content-type: image/gif;
	name="ecblank.gif"
Content-Disposition: inline; filename="ecblank.gif"
Content-ID: <20__=0ABBE16DDFF07DA38f9e8a93df938@us.ibm.com>
Content-transfer-encoding: base64

R0lGODlhEAABAIAAAAAAAP///yH5BAEAAAEALAAAAAAQAAEAAAIEjI8ZBQA7

--0__=0ABBE16DDFF07DA38f9e8a93df938690918c0ABBE16DDFF07DA3--