harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nathan Beyer" <nbe...@gmail.com>
Subject Re: [classlib][archive] Parsing invalid manifest (was: Re: classlib archive changed broke EUT/update)
Date Thu, 24 Apr 2008 01:07:26 GMT
What's the "Harmony documentation system"?

-Nathan

On Wed, Apr 23, 2008 at 2:08 AM, Alexei Fedotov <alexei.fedotov@gmail.com>
wrote:

> Hello Nathan,
> What else is blocked by the fact that we mandate manifest versions in
> manifests? I was asking myself the same question. This night I found
> out that Harmony documentation system depends on it. More precisely,
> Anakia refuses to load Apache commons. This means we have to set up a
> workaround until "commoners" fix their manifest.
>
> Thanks.
>
>
>
> On Wed, Apr 23, 2008 at 7:17 AM, Nathan Beyer <ndbeyer@apache.org<https://mail.google.com/mail?view=cm&tf=0&to=ndbeyer@apache.org>>
> wrote:
> > I just got through getting the Eclipse guys to fix their test artifacts.
> >  Argh...
> >
> >  Keep in mind - this isn't a manifest in some long-used application,
> this is
> >  a mocked up manifest used in a test case. I'd like to get a better
> feeling
> >  for how many real invalid manifest files are out there. So far it's
> just
> >  three files in some test cases, which have already been fixed. What
> else is
> >  out there?
> >
> >  -Nathan
> >
> >
> >  On Tue, Apr 22, 2008 at 5:29 AM, Tim Ellison <t.p.ellison@gmail.com<https://mail.google.com/mail?view=cm&tf=0&to=t.p.ellison@gmail.com>>
> wrote:
> >
> >  > Even though the manifest is invalid according to the spec, shouldn't
> we be
> >  > parsing it the same way as the RI for compatibility reasons?
> >  >
> >  > I wonder how many apps have invalid manifests.
> >  >
> >  > Regards,
> >  > Tim
> >  >
> >  > Nathan Beyer wrote:
> >  >
> >  > > Careful, the test isn't necessarily invalid - that's quick
> assumption.
> >  > > It
> >  > > needs to be verified that the test isn't dyanamically generating a
> >  > > manifest
> >  > > with Harmony's classlib; it's possible that Harmony's classlib is
> >  > > creating
> >  > > the invalid manifest, which later causes a failure when reading the
> >  > > manifest.
> >  > >
> >  > > -Nathan
> >  > >
> >  > > On Fri, Apr 18, 2008 at 3:45 AM, Vladimir Beliaev <
> >  > > vladimir.k.beliaev@gmail.com<https://mail.google.com/mail?view=cm&tf=0&to=vladimir.k.beliaev@gmail.com>
> <https://mail.google.com/mail?view=cm&tf=0&to=vladimir.k.beliaev@gmail.com
> >>
> >
> >
> > > > wrote:
> >  > >
> >  > >  According to the issue quick evaluation the tests are invalid, so
> noted
> >  > > > revision helped to detect them. The tests are to be excluded now.
> >  > > >
> >  > > > Tim, Alexei, thank you for your input.
> >  > > >
> >  > > > Thanks
> >  > > > Vladimir
> >  > > >
> >  > > > 2008/4/18, Alexei Fedotov <alexei.fedotov@gmail.com<https://mail.google.com/mail?view=cm&tf=0&to=alexei.fedotov@gmail.com>
> <https://mail.google.com/mail?view=cm&tf=0&to=alexei.fedotov@gmail.com>
> >  > > > <
> https://mail.google.com/mail?view=cm&tf=0&to=alexei.fedotov@gmail.com
> >  > > > >
> >  > > >
> >  > > > > :
> >  > > > >
> >  > > > > Tim,
> >  > > > > We are sorting that out.
> >  > > > >
> >  > > > > Thanks.
> >  > > > >
> >  > > > > On Fri, Apr 18, 2008 at 12:21 PM, Tim Ellison <
> t.p.ellison@gmail.com<https://mail.google.com/mail?view=cm&tf=0&to=t.p.ellison@gmail.com>
> <https://mail.google.com/mail?view=cm&tf=0&to=t.p.ellison@gmail.com>
> >  > > > > <
> https://mail.google.com/mail?view=cm&tf=0&to=t.p.ellison@gmail.com>
> >  > > > >
> >  > > > > wrote:
> >  > > > >
> >  > > > > > It'll be a couple of days before I can look at it as I'm
> going
> >  > > > > > away.
> >  > > > > >
> >  > > > > I'll
> >  > > > >
> >  > > > > > take a look when I get back if nobody beats me to it.
> >  > > > > >
> >  > > > > >  Regards,
> >  > > > > >  Tim
> >  > > > > >
> >  > > > > >
> >  > > > > >
> >  > > > > >  Vladimir Beliaev wrote:
> >  > > > > >
> >  > > > > >  Tim,
> >  > > > > > >
> >  > > > > > > I've opened
> >  > > > > > >
> >  > > > > > HARMONY-5747<
> https://issues.apache.org/jira/browse/HARMONY-5747>
> >  > > > > >
> >  > > > > > > which
> >  > > > > > > reports regression in EUT/update suite due to r641928
> >  > > > > > >
> >  > > > > > >      [apply patch
> >  > > > > > > HARMONY-4569<
> https://issues.apache.org/jira/browse/HARMONY-4569>
> >  > > > > > > ([classlib][performance]
> >  > > > > > > Ineffecient manifest parsing results in slowdown
when
> debugging
> >  > > > > > > java
> >  > > > > > >
> >  > > > > > code)]
> >  > > > > >
> >  > > > > > > Could you take a look at regression & get it
resolved,
> please?
> >  > > > > > >
> >  > > > > > > Thanks
> >  > > > > > > Vladimir
> >  > > > > > >
> >  > > > > > >
> >  > > > > > >
> >  > > > >
> >  > > > > --
> >  > > > > With best regards,
> >  > > > > Alexei
> >  > > > >
> >  > > > >
> >  > >
> >
>
>
>
> --
> With best regards,
> Alexei
>

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