harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Beliaev" <vladimir.k.beli...@gmail.com>
Subject Re: [jira] Created: (HARMONY-4832) [buildtest][eut] text.xml should be patched with J2SE-5.0 to run EUT3.3 on Harmony
Date Fri, 21 Sep 2007 10:32:27 GMT
Sure, I'm working on patch for #2.

Thanks
Vladimir Beliaev


2007/9/21, Stepan Mishura <stepan.mishura@gmail.com>:
>
> On 9/21/07, Vladimir Beliaev wrote:
> > Hello, Stepan,
> >
> > good notes, thanks!
> >
> > I've implement the patching of test.xml follow the practice of eut3.2
> /test.xml
> > patching. As far as I wrote in JIRA description I do not like this
> solution
> > a lot since it produces an ugly looking test.xml (although a working
> one).
> >
> > As far as I know the base model of EUT is run both Eclipse-ant-runner
> and
> > Eclipe JUnit Tests on the version of VM (still different instances) -
> and
> > this VM is taken from classpath. There is a trick with J2SE-5.0 settings
> (I
> > wrote in JIRA about it) and now we are using it to run JUnit Tests on
> > Harmony while Eclipse-ant-runner is started on RI.
> >
> > Ok, your proposals seem to be correct although they relay on EUT
> > undocumented internals.
> >
> > Let me summarize the possible solutions (we need to run EUT on Harmony)
> and
> > pick the most appropriate one (note - all solutions require
> > J2SE-5.0property set from our
> > build.xml):
> >
> > 1. *sef-hosting *(i.e. Harmony is in path and in J2SE-5.0). No patching
> is
> > required. No additional setting is required.
> >
> > 2. *set 'jvm' property* from out build.xml. No patching is required.
> Still
> > 'jvm' setting is undocumented.
> >
> > 3. *patch 'runtests.bat' to set 'vmcmd' *and keep Harmony in PATH and
> > JAVA_HOME. No patching is required. A bit unclear way of running.
> >
> > I would pick the #2 from above. The #1 requires an accurate testing in
> > advance - I believe such a testing ccan be done after EUT regular
> testing is
> > stabilized (i.e no regressions allowed).
> >
> > I'll update the patch for HARMONY-4832 but please pick the solution.
>
> OK. If there are no other ideas let's choose the option #2.
> I assume we also will remove patching build.xml from 3.2 tests too.
>
> Could you update the patch?
>
> Thanks,
> Stepan.
>
> >
> > Thanks
> > Vladimir Beliaev
> >
> > 2007/9/21, Stepan Mishura <stepan.mishura@gmail.com>:
> > >
> > > Hi Vladimir,
> > >
> > > May be I'm missing something (I didn't look into the build script and
> > > tests deeply. I only quickly reviewed test.xml for EUT 3.3) but my
> > > impression Eclipse unit tests have concept of "running VM" and "tested
> > > VM".
> > >
> > > As I see there is a shell script ('runtests') that picks up java
> > > ('vmcmd' property ) from PATH to run launcher. So we can call  'vmcmd'
> > > property  "running VM".
> > >
> > > Also in the eclipse-testing/test.xml I see
> > > <target name="setJVMfromUserSpecified" if="jvm">
> > >
> > > And the property 'jvm' is set up before launching test suite, for
> example
> > >
> > > <target name="jdtcorebuilder" depends="setJVMProperties">
> > >    <!--Run with 1.5 vm if it is available-->
> > >    <condition property="jvm" value="${J2SE-5.0}">
> > >        <isset property="J2SE-5.0"/>
> > >    </condition>
> > >    <runTests testPlugin="org.eclipse.jdt.core.tests.builder" />
> > > </target>
> > >
> > > So I assume that 'jvm' value is used to run tests. Am I correct?
> > > If yes then we don't need to patch test.xml. IMO it is enough to set
> > > jvm in 'properties.file'
> > >
> > > Also from the first glance it should work for EUT 3.2 too. Correct?
> > >
> > > Thanks,
> > > Stepan.
> > >
> > > -----Original Message-----
> > > From: Vladimir Beliaev (JIRA) [mailto:jira@apache.org]
> > > Sent: Friday, September 21, 2007 1:08 AM
> > > To: commits@harmony.apache.org
> > > Subject: [jira] Created: (HARMONY-4832) [buildtest][eut] text.xml
> > > should be patched with J2SE-5.0 to run EUT3.3 on Harmony
> > >
> > > [buildtest][eut] text.xml should be patched with J2SE-5.0 to run
> > > EUT3.3 on Harmony
> > >
> > >
> ----------------------------------------------------------------------------------
> > >
> > >                 Key: HARMONY-4832
> > >                 URL:
> https://issues.apache.org/jira/browse/HARMONY-4832
> > >             Project: Harmony
> > >          Issue Type: Bug
> > >          Components: build - test - ci
> > >         Environment: any
> > >            Reporter: Vladimir Beliaev
> > >
> > >
> > > Ok, the less magic the better. There is no concept of "running VM" and
> > > "tested VM" for EUT. Instead all EUT are run on the same java taken
> > > from PATH. Still some tests requires Java5.0, so if J2SE-5.0 property
> > > is specified then these tests are run on java pointed by this
> > > property.
> > >
> > > This trick is used to "point" EUT to Harmony, i.e each EUT test suite
> > > (in eclipse-testing/test.xml) must have J2SE-5.0 property specified.
> > >
> > > So while BTI installation (executing "setup" tag from
> > > tests/eut/build.xml) the eclispe-testing/test.xml must be patched.
> > >
> > > Such a patching was done for EUT3.2. One need to add it for EUT3.3.
> > >
> > > --
> > > This message is automatically generated by JIRA.
> > > -
> > > You can reply to this email to add a comment to the issue online.
> > >
> >
> >
> >
> > --
> > Vladimir Beliaev
> > Intel Middleware Products Division
>



-- 
Vladimir Beliaev
Intel Middleware Products Division

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message