Without the classpath modifications then it works for me on linux.
-Mark.
On 4/20/06, Stepan Mishura <[EMAIL PROTECTED]> wrote:
> On 4/20/06, Mark Hindess wrote:
> >
> > Stepan,
> >
> > In your original commit for HARMONY-315, you made this change:
> >
> > <classpath>
> > <pathelement path="${hy.crypto.src.test.resources}"/>
> > </classpath>
> >
> > I'm not sure how this relates to this JIRA
>
>
> Right, it was better to separate updates.
>
> but it is this that breaks
> > the test. Why not remove/fix this unrelated change rather than
> > comment out the otherwise working test?
>
>
> It is not quite correct. The tests passes on Windows but fails on Linux -
> I'm going to investigate the difference. But if I remove these lines then it
> will fail on Windows too.
>
> IMHO, we should avoid making unrelated changes in a single commit.
>
>
> Agreed.
>
> Thanks,
> Stepan.
>
> Regards,
> > Mark.
> >
> > On 4/20/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> > > Author: smishura
> > > Date: Thu Apr 20 02:05:28 2006
> > > New Revision: 395541
> > >
> > > URL: http://svn.apache.org/viewcvs?rev=395541&view=rev
> > > Log:
> > > tests/api/javax/crypto/CipherTest.java back to exclude test to make test
> > suite pass on Linux
> > >
> > > Modified:
> > >
> > incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml
> > >
> > > Modified:
> > incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml
> > > URL:
> > http://svn.apache.org/viewcvs/incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml?rev=395541&r1=395540&r2=395541&view=diff
> > >
> > ==============================================================================
> > > ---
> > incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml
> > (original)
> > > +++
> > incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml
> > Thu Apr 20 02:05:28 2006
> > > @@ -118,6 +118,9 @@
> > > <batchtest todir="${hy.tests.reports}" haltonfailure="no">
> > > <fileset dir="${hy.crypto.src.test.java}">
> > > <include name="**/*Test.java"/>
> > > +
> > > + <!-- Harmony exclude list -->
> > > + <exclude
> > name="tests/api/javax/crypto/CipherTest.java"/>
> > > </fileset>
> > > </batchtest>
> > > </junit>
> > >
> > >
> > >
> >
> >
> > --
> > Mark Hindess <[EMAIL PROTECTED]>
> > IBM Java Technology Centre, UK.
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
> --
> Thanks,
> Stepan Mishura
> Intel Middleware Products Division
>
> ------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
--
Mark Hindess <[EMAIL PROTECTED]>
IBM Java Technology Centre, UK.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
