FYI - the test VM has been incorrectly identified as IBM for a while;
probably since the new version implementation was put in. It should be
fixed now, but if any test excludes have been tweaked lately, you may
want to double-check them.

-Nathan

On Tue, Apr 14, 2009 at 9:08 PM,  <[email protected]> wrote:
> Author: ndbeyer
> Date: Wed Apr 15 02:07:59 2009
> New Revision: 765035
>
> URL: http://svn.apache.org/viewvc?rev=765035&view=rev
> Log:
> make vm info test case insensitive
>
> Modified:
>    harmony/enhanced/classlib/trunk/make/properties.xml
>
> Modified: harmony/enhanced/classlib/trunk/make/properties.xml
> URL: 
> http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/make/properties.xml?rev=765035&r1=765034&r2=765035&view=diff
> ==============================================================================
> --- harmony/enhanced/classlib/trunk/make/properties.xml (original)
> +++ harmony/enhanced/classlib/trunk/make/properties.xml Wed Apr 15 02:07:59 
> 2009
> @@ -442,6 +442,7 @@
>     </target>
>
>     <target name="test-jre-vm-info" depends="-get-test-vm-output">
> +        <echo level="info" message="java 
> -version${line.separator}${test.vm.info.tmp}" />
>         <!-- Add other runtime names here -->
>         <condition property="hy.test.vm.name" value="ri" >
>             <contains string="${test.vm.info.tmp}" substring="HotSpot"/>
> @@ -452,7 +453,7 @@
>         <!-- FIXME current behavior relies on the fact that J9 VM doesn't
>              support -version parameter -->
>         <condition property="hy.test.vm.name" value="drl" else="ibm" >
> -            <contains string="${test.vm.info.tmp}" substring="harmony" />
> +            <contains string="${test.vm.info.tmp}" substring="harmony" 
> casesensitive="false" />
>         </condition>
>         <echo level="info" message="hy.test.vm.name = ${hy.test.vm.name}" />
>     </target>
>
>
>

Reply via email to