harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Petrenko" <alexey.a.petre...@gmail.com>
Subject Re: [testing permission to commit changes to Struts scenario
Date Mon, 18 Feb 2008 13:33:05 GMT
I can not find the patch you are going to commit in the issue...

Thanks in advance.

SY, Alexey

2008/2/18, Alexei Zakharov <alexei.zakharov@gmail.com>:
> All,
>
> As it was agreed I'm going to commit the changes to the Struts
> scenario to let it follow the current Harmony behavior - we may use it
> for the snapshot testing after fixing it. See HARMONY-5498 for
> details. I'm waiting for other committer's vote.
>
> Thanks,
> Alexei
>
> 2008/2/14, Alexei Zakharov <alexei.zakharov@gmail.com>:
> > >  > The senario test is writtern by ourselves. My question is, will this
> > >  > difference block the struts or not?
> >
> > I can't say that Struts is *blocked* by this difference. It just uses
> > it in one of its standard examples. And our struts scenario checks
> > that example. In respect that the example itself is not a production
> > application and RI's behavior looks strange indeed I agree we may mark
> > it as a non-bug difference and fix Struts scenario to follow the
> > current Harmony behavior.
> >
> > Alexei
> >
> > 2008/2/13, Tim Ellison <t.p.ellison@gmail.com>:
> > > Tony Wu wrote:
> > >  > Tim, Alexei
> > >  > Seems it is locale dependent. I tried several locales, there is a
> > >  > empty string in the output. I can not tell why RI behaves like this.
> > >  >
> > >  > The senario test is writtern by ourselves. My question is, will this
> > >  > difference block the struts or not? If it blocks struts we have to
> > >  > work around even if the fix does harm to performance. If not, let's
> > >  > keep it as non-bug difference and find out the reason.
> > >
> > >  My inclination is to leave the current behavior as-is, a non-bug
> > >  difference until we get a compelling reason to duplicate this 'bug'.
> > >
> > >  Regards,
> > >
> > > Tim
> >
>

Mime
View raw message